summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authordoug <>2015-07-20 23:15:28 +0000
committerdoug <>2015-07-20 23:15:28 +0000
commit1f3595b71a0bc1f032a89cb05bfc63999046bca1 (patch)
treeb3d9963e191447c8c845e597bf240a3549840a5d /src
parent4daa9c93fc6f86cf389f1a696e38894bb40e4161 (diff)
downloadopenbsd-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.c6
-rw-r--r--src/lib/libssl/src/crypto/bio/bss_dgram.c6
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);