Check DTLS_BAD_VER for version number.
The version check for DTLS1_VERSION was redundant as DTLS1_VERSION > TLS1_1_VERSION, however we do need to check for DTLS1_BAD_VER for compatibility. PR:2984 (cherry picked from commit d980abb22e22661e98e5cee33d760ab0c7584ecc)
This commit is contained in:
parent
0ced72c608
commit
3a3a1af1da
@ -148,7 +148,7 @@ int tls1_cbc_remove_padding(const SSL* s,
|
|||||||
unsigned padding_length, good, to_check, i;
|
unsigned padding_length, good, to_check, i;
|
||||||
const unsigned overhead = 1 /* padding length byte */ + mac_size;
|
const unsigned overhead = 1 /* padding length byte */ + mac_size;
|
||||||
/* Check if version requires explicit IV */
|
/* Check if version requires explicit IV */
|
||||||
if (s->version >= TLS1_1_VERSION || s->version == DTLS1_VERSION)
|
if (s->version >= TLS1_1_VERSION || s->version == DTLS1_BAD_VER)
|
||||||
{
|
{
|
||||||
/* These lengths are all public so we can test them in
|
/* These lengths are all public so we can test them in
|
||||||
* non-constant time.
|
* non-constant time.
|
||||||
|
Loading…
x
Reference in New Issue
Block a user