]> git.ipfire.org Git - thirdparty/squid.git/commit
Bug Fix: Squid may crash, when accessing an SSL certificate with errors
authorChristos Tsantilas <chtsanti@users.sourceforge.net>
Wed, 16 Mar 2011 09:29:40 +0000 (11:29 +0200)
committerChristos Tsantilas <chtsanti@users.sourceforge.net>
Wed, 16 Mar 2011 09:29:40 +0000 (11:29 +0200)
commit8211c31447f36587e644e43d34d427b5af85273f
tree922c92ec0d73c39f3f140127f869bf20a52b11a7
parent01494d71b4ddedcf84704a3a15e6951de907f32f
Bug Fix: Squid may crash, when accessing an SSL certificate with errors

This is a security bug.
The bug report is:
   When accessing a revoked certificate (i.e., X509_V_ERR_CERT_REVOKED) Squid
crashes. In ssl/ErrorDetail.cc:223 the detailed message is left blank if the
error is not specifically handled by Squid and the errorpage.cc:1193 assertion
fails while trying to convert the message.

This patch:
 - Handle inside ErrorState::Convert the cases where the ErrorDetail return
   blank error detail string
 - Use a default  detail error message in ssl::ErrorDetail, for the cases
   where the detail error is not defined in TheSslDetailMap.
 - If a name for the ssl::ErrorDetail error code is not defined return the
   numeric error code when the "%err_name" formating code used
src/errorpage.cc
src/ssl/ErrorDetail.cc