diff options
author | mbelshe@chromium.org <mbelshe@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-21 18:43:21 +0000 |
---|---|---|
committer | mbelshe@chromium.org <mbelshe@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-04-21 18:43:21 +0000 |
commit | 69d717bd97afda07c4f87d50559bdbff5b8018e4 (patch) | |
tree | 497908d17d44ca8778fb77bd8e4dcb139d9d507e /net/spdy/spdy_session_pool.cc | |
parent | 6ade700106737e9218f5a97be11ac0eb6a90b1e1 (diff) | |
download | chromium_src-69d717bd97afda07c4f87d50559bdbff5b8018e4.zip chromium_src-69d717bd97afda07c4f87d50559bdbff5b8018e4.tar.gz chromium_src-69d717bd97afda07c4f87d50559bdbff5b8018e4.tar.bz2 |
Fix crasher on SpdySession teardown. When we close the session on error,
we can get into a case where we delete the session itself while cleaning
out the active streams. This ends up with a re-entrant call into
CloseAllStreams.
Fix is to self-hold a reference during teardown.
The test case crashes without the fix, and works fine with the fix.
BUG=42215
TEST=SpdyNetworkTransactionTest.GoAwayWithActiveStream
Review URL: http://codereview.chromium.org/1711008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@45210 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'net/spdy/spdy_session_pool.cc')
0 files changed, 0 insertions, 0 deletions