Don't attempt to add a zero length padding extension

The padding extension should always be at least 1 byte long

Reviewed-by: Paul Dale <paul.dale@oracle.com>
(Merged from https://github.com/openssl/openssl/pull/4331)
This commit is contained in:
Matt Caswell 2017-09-04 08:44:02 +01:00
parent 45fd6a59fa
commit 3d85c7f408

View File

@ -843,7 +843,7 @@ EXT_RETURN tls_construct_ctos_padding(SSL *s, WPACKET *pkt,
* 1 byte long so as not to have an empty extension last (WebSphere 7.x,
* 8.x are intolerant of that condition)
*/
if (hlen >= 4)
if (hlen > 4)
hlen -= 4;
else
hlen = 1;