diff options
| author | jsing <> | 2014-11-27 16:07:33 +0000 |
|---|---|---|
| committer | jsing <> | 2014-11-27 16:07:33 +0000 |
| commit | 592644a0e4a109bebb8c932afbc4ca360039d3f9 (patch) | |
| tree | c9c7516112d9b77b9f111d920e6280490dc320e9 | |
| parent | 07409820dc766c10b948b458d7af9315231002f1 (diff) | |
| download | openbsd-592644a0e4a109bebb8c932afbc4ca360039d3f9.tar.gz openbsd-592644a0e4a109bebb8c932afbc4ca360039d3f9.tar.bz2 openbsd-592644a0e4a109bebb8c932afbc4ca360039d3f9.zip | |
Avoid a double-free in an error path.
Reported by Felix Groebert of the Google Security Team.
ok beck@ miod@
Diffstat (limited to '')
| -rw-r--r-- | src/lib/libssl/d1_srtp.c | 3 | ||||
| -rw-r--r-- | src/lib/libssl/src/ssl/d1_srtp.c | 3 |
2 files changed, 2 insertions, 4 deletions
diff --git a/src/lib/libssl/d1_srtp.c b/src/lib/libssl/d1_srtp.c index 1443964776..eeb19a6183 100644 --- a/src/lib/libssl/d1_srtp.c +++ b/src/lib/libssl/d1_srtp.c | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: d1_srtp.c,v 1.9 2014/11/16 14:12:47 jsing Exp $ */ | 1 | /* $OpenBSD: d1_srtp.c,v 1.10 2014/11/27 16:07:33 jsing Exp $ */ |
| 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) | 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * | 4 | * |
| @@ -355,7 +355,6 @@ ssl_parse_clienthello_use_srtp_ext(SSL *s, unsigned char *d, int len, int *al) | |||
| 355 | SSLerr(SSL_F_SSL_PARSE_CLIENTHELLO_USE_SRTP_EXT, | 355 | SSLerr(SSL_F_SSL_PARSE_CLIENTHELLO_USE_SRTP_EXT, |
| 356 | SSL_R_BAD_SRTP_MKI_VALUE); | 356 | SSL_R_BAD_SRTP_MKI_VALUE); |
| 357 | *al = SSL_AD_DECODE_ERROR; | 357 | *al = SSL_AD_DECODE_ERROR; |
| 358 | sk_SRTP_PROTECTION_PROFILE_free(clnt); | ||
| 359 | goto done; | 358 | goto done; |
| 360 | } | 359 | } |
| 361 | 360 | ||
diff --git a/src/lib/libssl/src/ssl/d1_srtp.c b/src/lib/libssl/src/ssl/d1_srtp.c index 1443964776..eeb19a6183 100644 --- a/src/lib/libssl/src/ssl/d1_srtp.c +++ b/src/lib/libssl/src/ssl/d1_srtp.c | |||
| @@ -1,4 +1,4 @@ | |||
| 1 | /* $OpenBSD: d1_srtp.c,v 1.9 2014/11/16 14:12:47 jsing Exp $ */ | 1 | /* $OpenBSD: d1_srtp.c,v 1.10 2014/11/27 16:07:33 jsing Exp $ */ |
| 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) | 2 | /* Copyright (C) 1995-1998 Eric Young (eay@cryptsoft.com) |
| 3 | * All rights reserved. | 3 | * All rights reserved. |
| 4 | * | 4 | * |
| @@ -355,7 +355,6 @@ ssl_parse_clienthello_use_srtp_ext(SSL *s, unsigned char *d, int len, int *al) | |||
| 355 | SSLerr(SSL_F_SSL_PARSE_CLIENTHELLO_USE_SRTP_EXT, | 355 | SSLerr(SSL_F_SSL_PARSE_CLIENTHELLO_USE_SRTP_EXT, |
| 356 | SSL_R_BAD_SRTP_MKI_VALUE); | 356 | SSL_R_BAD_SRTP_MKI_VALUE); |
| 357 | *al = SSL_AD_DECODE_ERROR; | 357 | *al = SSL_AD_DECODE_ERROR; |
| 358 | sk_SRTP_PROTECTION_PROFILE_free(clnt); | ||
| 359 | goto done; | 358 | goto done; |
| 360 | } | 359 | } |
| 361 | 360 | ||
