diff options
| author | davidben@chromium.org <davidben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-05 23:12:19 +0000 |
|---|---|---|
| committer | davidben@chromium.org <davidben@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-08-05 23:12:19 +0000 |
| commit | 780d03b3bfdca6b6d372d59fcc932606aaf07c2a (patch) | |
| tree | 3e784745ea60f12aa6f9090ec789011065fadff0 /net/socket/ssl_client_socket_pool.cc | |
| parent | f6d7389ff93180213b0828b702bee0163807cb49 (diff) | |
| download | chromium_src-780d03b3bfdca6b6d372d59fcc932606aaf07c2a.zip chromium_src-780d03b3bfdca6b6d372d59fcc932606aaf07c2a.tar.gz chromium_src-780d03b3bfdca6b6d372d59fcc932606aaf07c2a.tar.bz2 | |
Fix crash during SSL client auth
We did not set ssl_error_response_info in all cases, so later error-handling
code was missing objects which it needed.
BUG=51127
TEST=see bug report
Review URL: http://codereview.chromium.org/3015057
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@55154 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/socket/ssl_client_socket_pool.cc')
| -rw-r--r-- | net/socket/ssl_client_socket_pool.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/net/socket/ssl_client_socket_pool.cc b/net/socket/ssl_client_socket_pool.cc index e0f04af..255a486 100644 --- a/net/socket/ssl_client_socket_pool.cc +++ b/net/socket/ssl_client_socket_pool.cc @@ -240,11 +240,13 @@ int SSLConnectJob::DoTunnelConnectComplete(int result) { } void SSLConnectJob::GetAdditionalErrorState(ClientSocketHandle * handle) { + // Headers in |error_response_info_| indicate a proxy tunnel setup + // problem. See DoTunnelConnectComplete. if (error_response_info_.headers) { - handle->set_ssl_error_response_info(error_response_info_); handle->set_pending_http_proxy_connection( transport_socket_handle_.release()); } + handle->set_ssl_error_response_info(error_response_info_); if (!ssl_connect_start_time_.is_null()) handle->set_is_ssl_error(true); } |