diff options
| author | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-20 22:16:38 +0000 |
|---|---|---|
| committer | darin@chromium.org <darin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-02-20 22:16:38 +0000 |
| commit | a3ba3289219898411b3f3d9aea4a316a0bea19d7 (patch) | |
| tree | f8c51e8c9ea86908f8611d359601292509a3f629 /chrome/browser/ssl | |
| parent | 084683940bef81aeeceb00db74f674422c831ff4 (diff) | |
| download | chromium_src-a3ba3289219898411b3f3d9aea4a316a0bea19d7.zip chromium_src-a3ba3289219898411b3f3d9aea4a316a0bea19d7.tar.gz chromium_src-a3ba3289219898411b3f3d9aea4a316a0bea19d7.tar.bz2 | |
Fix memory leak caused by bad reference counting. When the test server is
allocated it has a reference count of 0, so calling Release again, does not
free the memory. The fix is to use a scoped_refptr for both the return value
as well as for the local variable.
This failure case is triggered by the purify bot not having the test
certificate installed. We should also fix that so that we run more tests on
the purify bot. Testing this case is also interesting I guess :)
TBR=erikkay
Review URL: http://codereview.chromium.org/27013
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@10129 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/ssl')
0 files changed, 0 insertions, 0 deletions