]> 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>
Thu, 17 Sep 2015 05:58:53 +0000 (22:58 -0700)
committerAmos Jeffries <squid3@treenet.co.nz>
Thu, 17 Sep 2015 05:58:53 +0000 (22:58 -0700)
commit336bac61258fd59e7973568882b77bc33fb766a4
tree2bcd095771beabac9c52cf87e50ff25689b410a6
parentb8c5570b0e582a4dc137f92ab376a5da5d8fa461
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 tries to improve the related code to
avoid related problems in the future.

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