diff options
| author | eroman@chromium.org <eroman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-08 19:03:29 +0000 |
|---|---|---|
| committer | eroman@chromium.org <eroman@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2009-07-08 19:03:29 +0000 |
| commit | 09f83fa89c9ab23b5f9623b06f1322850d21a15a (patch) | |
| tree | ff4c1eb9cef0509389974905e2eacc221d4d26c4 /tools/valgrind | |
| parent | 20b29783fd53c31aad7e362cbdedd5e13ac41f48 (diff) | |
| download | chromium_src-09f83fa89c9ab23b5f9623b06f1322850d21a15a.zip chromium_src-09f83fa89c9ab23b5f9623b06f1322850d21a15a.tar.gz chromium_src-09f83fa89c9ab23b5f9623b06f1322850d21a15a.tar.bz2 | |
Fix a leak of the new location GURL by NotificationTask when doing a redirect.
The problem is scoped_ptr<ResourceRequestDetails> is used to delete an instance of ResourceRedirectDetails, however the base class's destructor is non-virtual, so ResourceRedirectDetails extra field (GURL new_url_) does not get torn down properly.
Note that the blame callstacks in the bug report appear unrelated, because std::string is doing some fancy refcounting under the hood. So as GURLs get passed around, they take over the GURL::spec's memory that got allocated earlier by the network stack.
BUG=http://crbug.com/10873
TEST=The "RedirectTest.*" ui tests when run on linux valgrind bots should show no leaks
Review URL: http://codereview.chromium.org/155204
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@20179 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'tools/valgrind')
| -rw-r--r-- | tools/valgrind/suppressions.txt | 25 |
1 files changed, 0 insertions, 25 deletions
diff --git a/tools/valgrind/suppressions.txt b/tools/valgrind/suppressions.txt index 79e9cff..f0f98b4 100644 --- a/tools/valgrind/suppressions.txt +++ b/tools/valgrind/suppressions.txt @@ -494,31 +494,6 @@ fun:_ZN11MessageLoop7RunTaskEP4Task } { - # See http://crbug.com/10871 - bug_10871 - Memcheck:Leak - fun:_Znwj - fun:_ZNSs4_Rep9_S_createEjjRKSaIcE - fun:_ZNSs4_Rep8_M_cloneERKSaIcEj - fun:_ZNSs7reserveEj - fun:_ZNK4GURL27ResolveWithCharsetConverterERKSsPN9url_canon16CharsetConverterE - fun:_ZNK4GURL7ResolveERKSs - fun:_ZN17URLRequestHttpJob18IsRedirectResponseEP4GURLPi -} -{ - # See http://crbug.com/10873 - bug_10873 - Memcheck:Leak - fun:_Znwj - fun:_ZNSs4_Rep9_S_createEjjRKSaIcE - fun:_ZNSs4_Rep8_M_cloneERKSaIcEj - fun:_ZNSs7reserveEj - fun:_ZNK4GURL17ReplaceComponentsERKN9url_canon12ReplacementsIcEE - fun:_ZN13URLRequestJob21NotifyHeadersCompleteEv - fun:_ZN17URLRequestHttpJob21NotifyHeadersCompleteEv - fun:_ZN17URLRequestHttpJob16OnStartCompletedEi -} -{ # See http://crbug.com/11116 bug_11116a Memcheck:Leak |