]> git.ipfire.org Git - thirdparty/squid.git/commit
Squid Assertion String.cc:221: "str"
authorChristos Tsantilas <chtsanti@users.sourceforge.net>
Wed, 3 Jun 2015 11:12:40 +0000 (14:12 +0300)
committerChristos Tsantilas <chtsanti@users.sourceforge.net>
Wed, 3 Jun 2015 11:12:40 +0000 (14:12 +0300)
commited58b5234e629f24dc617759b82bddaa53be2fb8
tree0b3e4eae43fb8c6fbe9a0de9bc82ed0326221f5c
parentb418d9c8cb5ea7b4cac2c7c4dd38c3075a1ad794
Squid Assertion String.cc:221: "str"

This bug can be caused by certificates does not contain a CN field. In this
case the Ssl::ErrorDetail::cn method may return NULL causing this assertion
somewhere inside Ssl::ErrorDetail::buildDetail method, which expects always
a non NULL value from Ssl::ErrorDetail::cn and similar methods.

This patch try to hardening the Ssl::ErrorDetail error formating functions to
avoid always check for NULL values and also avoid sending wrong information
for various certificate fields in the case of an error while extracting the
information from certificate..

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