diff options
| author | Moritz Buhl <mbuhl@cvs.openbsd.org> | 2022-11-11 16:17:17 +0000 |
|---|---|---|
| committer | Moritz Buhl <mbuhl@cvs.openbsd.org> | 2022-11-11 16:17:17 +0000 |
| commit | 0c55f5eb12fd42a25f3e954f364c13befead629a (patch) | |
| tree | ae2793a18d417bef6ff59187b308fb729ea78817 /sbin | |
| parent | 1422f74e44fc345907b988bcebf4f9f670395bb7 (diff) | |
In case of an invalid SA resp is passed to ikev2_msg_cleanup without
initialization where the msg_parent field is accessed.
ok tobhe
Diffstat (limited to 'sbin')
| -rw-r--r-- | sbin/iked/ikev2_msg.c | 5 |
1 files changed, 3 insertions, 2 deletions
diff --git a/sbin/iked/ikev2_msg.c b/sbin/iked/ikev2_msg.c index 96d13026757..05e26af331e 100644 --- a/sbin/iked/ikev2_msg.c +++ b/sbin/iked/ikev2_msg.c @@ -1,4 +1,4 @@ -/* $OpenBSD: ikev2_msg.c,v 1.87 2022/09/21 22:32:11 tobhe Exp $ */ +/* $OpenBSD: ikev2_msg.c,v 1.88 2022/11/11 16:17:16 mbuhl Exp $ */ /* * Copyright (c) 2019 Tobias Heider <tobias.heider@stusta.de> @@ -811,7 +811,8 @@ ikev2_send_encrypted_fragments(struct iked *env, struct iked_sa *sa, sa->sa_encr == NULL || sa->sa_integr == NULL) { log_debug("%s: invalid SA", __func__); - goto done; + ikestat_inc(env, ikes_frag_send_failures); + return ret; } sa_fam = ((struct sockaddr *)&sa->sa_local.addr)->sa_family; |