diff options
author | doug <> | 2015-07-20 23:15:28 +0000 |
---|---|---|
committer | doug <> | 2015-07-20 23:15:28 +0000 |
commit | 1f3595b71a0bc1f032a89cb05bfc63999046bca1 (patch) | |
tree | b3d9963e191447c8c845e597bf240a3549840a5d /src | |
parent | 4daa9c93fc6f86cf389f1a696e38894bb40e4161 (diff) | |
download | openbsd-1f3595b71a0bc1f032a89cb05bfc63999046bca1.tar.gz openbsd-1f3595b71a0bc1f032a89cb05bfc63999046bca1.tar.bz2 openbsd-1f3595b71a0bc1f032a89cb05bfc63999046bca1.zip |
Correct #if/else logic in BIO's dgram_ctrl.
Coverity issue 72741 noticed that ret is being overwritten before use.
The actual issue is that the #if/else logic is guarding the wrong
lines. Besides impacting ret, this also made the case's break logic
wrong because it was in the wrong location.
ok bcook@ beck@
Diffstat (limited to 'src')
-rw-r--r-- | src/lib/libcrypto/bio/bss_dgram.c | 6 | ||||
-rw-r--r-- | src/lib/libssl/src/crypto/bio/bss_dgram.c | 6 |
2 files changed, 6 insertions, 6 deletions
diff --git a/src/lib/libcrypto/bio/bss_dgram.c b/src/lib/libcrypto/bio/bss_dgram.c index a07834b62c..a9657402ce 100644 --- a/src/lib/libcrypto/bio/bss_dgram.c +++ b/src/lib/libcrypto/bio/bss_dgram.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: bss_dgram.c,v 1.40 2015/02/09 10:55:33 jsing Exp $ */ | 1 | /* $OpenBSD: bss_dgram.c,v 1.41 2015/07/20 23:15:28 doug Exp $ */ |
2 | /* | 2 | /* |
3 | * DTLS implementation written by Nagendra Modadugu | 3 | * DTLS implementation written by Nagendra Modadugu |
4 | * (nagendra@cs.stanford.edu) for the OpenSSL project 2005. | 4 | * (nagendra@cs.stanford.edu) for the OpenSSL project 2005. |
@@ -423,10 +423,10 @@ dgram_ctrl(BIO *b, int cmd, long num, void *ptr) | |||
423 | ret = -1; | 423 | ret = -1; |
424 | break; | 424 | break; |
425 | } | 425 | } |
426 | ret = -1; | ||
427 | #else | 426 | #else |
428 | break; | 427 | ret = -1; |
429 | #endif | 428 | #endif |
429 | break; | ||
430 | case BIO_CTRL_DGRAM_QUERY_MTU: | 430 | case BIO_CTRL_DGRAM_QUERY_MTU: |
431 | #if defined(IP_MTU) | 431 | #if defined(IP_MTU) |
432 | addr_len = (socklen_t)sizeof(addr); | 432 | addr_len = (socklen_t)sizeof(addr); |
diff --git a/src/lib/libssl/src/crypto/bio/bss_dgram.c b/src/lib/libssl/src/crypto/bio/bss_dgram.c index a07834b62c..a9657402ce 100644 --- a/src/lib/libssl/src/crypto/bio/bss_dgram.c +++ b/src/lib/libssl/src/crypto/bio/bss_dgram.c | |||
@@ -1,4 +1,4 @@ | |||
1 | /* $OpenBSD: bss_dgram.c,v 1.40 2015/02/09 10:55:33 jsing Exp $ */ | 1 | /* $OpenBSD: bss_dgram.c,v 1.41 2015/07/20 23:15:28 doug Exp $ */ |
2 | /* | 2 | /* |
3 | * DTLS implementation written by Nagendra Modadugu | 3 | * DTLS implementation written by Nagendra Modadugu |
4 | * (nagendra@cs.stanford.edu) for the OpenSSL project 2005. | 4 | * (nagendra@cs.stanford.edu) for the OpenSSL project 2005. |
@@ -423,10 +423,10 @@ dgram_ctrl(BIO *b, int cmd, long num, void *ptr) | |||
423 | ret = -1; | 423 | ret = -1; |
424 | break; | 424 | break; |
425 | } | 425 | } |
426 | ret = -1; | ||
427 | #else | 426 | #else |
428 | break; | 427 | ret = -1; |
429 | #endif | 428 | #endif |
429 | break; | ||
430 | case BIO_CTRL_DGRAM_QUERY_MTU: | 430 | case BIO_CTRL_DGRAM_QUERY_MTU: |
431 | #if defined(IP_MTU) | 431 | #if defined(IP_MTU) |
432 | addr_len = (socklen_t)sizeof(addr); | 432 | addr_len = (socklen_t)sizeof(addr); |