diff options
author | Bodo Möller <bodo@openssl.org> | 2001-06-15 18:05:09 +0000 |
---|---|---|
committer | Bodo Möller <bodo@openssl.org> | 2001-06-15 18:05:09 +0000 |
commit | 285b42756ae3023d885461d819fad14fc1e363e3 (patch) | |
tree | 47e0c5ac6b8f4b9b596e72b28a08a5cc416aadde /ssl/t1_enc.c | |
parent | 508f15cdabe9c66db694efb24028cc38b9bee261 (diff) |
pay attention to blocksize before attempting decryption
Diffstat (limited to 'ssl/t1_enc.c')
-rw-r--r-- | ssl/t1_enc.c | 14 |
1 files changed, 13 insertions, 1 deletions
diff --git a/ssl/t1_enc.c b/ssl/t1_enc.c index 5f0976f9e7..d3a15e3441 100644 --- a/ssl/t1_enc.c +++ b/ssl/t1_enc.c @@ -509,6 +509,16 @@ int tls1_enc(SSL *s, int send) } #endif /* KSSL_DEBUG */ + if (!send) + { + if (l == 0 || l%bs != 0) + { + SSLerr(SSL_F_TLS1_ENC,SSL_R_BLOCK_CIPHER_PAD_IS_WRONG); + ssl3_send_alert(s,SSL3_AL_FATAL,SSL_AD_DECRYPT_ERROR); + return(0); + } + } + EVP_Cipher(ds,rec->data,rec->input,l); #ifdef KSSL_DEBUG @@ -522,7 +532,7 @@ int tls1_enc(SSL *s, int send) if ((bs != 1) && !send) { - ii=i=rec->data[l-1]; + ii=i=rec->data[l-1]; /* padding_length */ i++; if (s->options&SSL_OP_TLS_BLOCK_PADDING_BUG) { @@ -533,6 +543,8 @@ int tls1_enc(SSL *s, int send) if (s->s3->flags & TLS1_FLAGS_TLS_PADDING_BUG) i--; } + /* TLS 1.0 does not bound the number of padding bytes by the block size. + * All of them must have value 'padding_length'. */ if (i > (int)rec->length) { SSLerr(SSL_F_TLS1_ENC,SSL_R_BLOCK_CIPHER_PAD_IS_WRONG); |