]> granicus.if.org Git - openssl/commit
Fix a mem leak in NPN handling
authorMatt Caswell <matt@openssl.org>
Fri, 9 Sep 2016 09:53:39 +0000 (10:53 +0100)
committerMatt Caswell <matt@openssl.org>
Thu, 22 Sep 2016 08:28:07 +0000 (09:28 +0100)
commit6d32c2ae28952b5c1d7a24968e488532fcadc51a
treeeeae76c033d526b3d05415d9ce441eb9ec721eb1
parentf6a7505e64d06f9d41e01b763b684e4e2df34922
Fix a mem leak in NPN handling

If a server sent multiple NPN extensions in a single ClientHello then a
mem leak can occur. This will only happen where the client has requested
NPN in the first place. It does not occur during renegotiation. Therefore
the maximum that could be leaked in a single connection with a malicious
server is 64k (the maximum size of the ServerHello extensions section). As
this is client side, only occurs if NPN has been requested and does not
occur during renegotiation this is unlikely to be exploitable.

Issue reported by Shi Lei.

Reviewed-by: Rich Salz <rsalz@openssl.org>
ssl/t1_lib.c