]> git.ipfire.org Git - thirdparty/squid.git/commit
Bug 4309: Fix the presence of extensions detection in SSL Hello messages
authorChristos Tsantilas <chtsanti@users.sourceforge.net>
Wed, 16 Sep 2015 08:43:18 +0000 (11:43 +0300)
committerChristos Tsantilas <chtsanti@users.sourceforge.net>
Wed, 16 Sep 2015 08:43:18 +0000 (11:43 +0300)
commit458fd4703e2b539401fab128557792a938944aa8
treeb0d041844f23ba1a95bd67cca53ddc56c9504b64
parent09c931e6d9a292322d3212ce5a3733a380bc2a2c
Bug 4309: Fix the presence of extensions detection in SSL Hello messages

RFC5246 section 7.4.1.3 (Server Hello) says:

   The presence of extensions can be detected by determining whether
   there are bytes following the compression_method field at the end of
   the ServerHello.

Current parsing Hello code checks whether there are bytes in the whole SSL
message. It does not account for the fact that the message may contain more than
just ServerHello.
This patch fixes this issue and try to improve the related code to avoid related
problems in the future.

This is a Measurement Factory project
src/ssl/bio.cc