diff options
| author | Dr. Stephen Henson <steve@openssl.org> | 2014-01-28 15:10:27 +0000 |
|---|---|---|
| committer | Dr. Stephen Henson <steve@openssl.org> | 2014-01-28 15:15:58 +0000 |
| commit | 73c92dfa0c15d7932d86130a525d1a1bc43c312a (patch) | |
| tree | 87b37b382d187d2c98a8e96dab517034a5808782 /ssl | |
| parent | 717cc8589540b95122a652dee68e6a75b6262d93 (diff) | |
Check i before r[i].
PR#3244
(cherry picked from commit 9614d2c676ffe74ce0c919d9e5c0d622a011cbed)
Diffstat (limited to 'ssl')
| -rw-r--r-- | ssl/s3_srvr.c | 4 |
1 files changed, 2 insertions, 2 deletions
diff --git a/ssl/s3_srvr.c b/ssl/s3_srvr.c index 29459f0490..822d9b57ef 100644 --- a/ssl/s3_srvr.c +++ b/ssl/s3_srvr.c @@ -1896,7 +1896,7 @@ int ssl3_send_server_key_exchange(SSL *s) SSLerr(SSL_F_SSL3_SEND_SERVER_KEY_EXCHANGE,SSL_R_UNKNOWN_KEY_EXCHANGE_TYPE); goto f_err; } - for (i=0; r[i] != NULL && i<4; i++) + for (i=0; i < 4 && r[i] != NULL; i++) { nr[i]=BN_num_bytes(r[i]); #ifndef OPENSSL_NO_SRP @@ -1931,7 +1931,7 @@ int ssl3_send_server_key_exchange(SSL *s) } d = p = ssl_handshake_start(s); - for (i=0; r[i] != NULL && i<4; i++) + for (i=0; i < 4 && r[i] != NULL; i++) { #ifndef OPENSSL_NO_SRP if ((i == 2) && (type & SSL_kSRP)) |