Comment #2 on issue 15350 by bugdro...@chromium.org: Review RESOURCE_LEAK
Coverity defects in Chromium
http://code.google.com/p/chromium/issues/detail?id=15350
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=22379
------------------------------------------------------------------------
r22379 | fin...@chromium.org | 2009-08-04 08:20:34 -0700 (Tue, 04 Aug 2009)
| 8 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_codec.cc?r1=22379&r2=22378
M
http://src.chromium.org/viewvc/chrome/trunk/src/chrome/browser/bookmarks/bookmark_codec.h?r1=22379&r2=22378
Fixing potential memory leak in BookmarkCodec::DecodeNode.
Coverity finds a condition where this leaks. We are at the moment not using
it in such a way that it leaks, but that might of course change. It looks
to me like the API is not intended to be used in such a way anyway, so we
can just check for it and return false.
BUG=15350
TEST=Decoding bookmarks should work as before.
Review URL: http://codereview.chromium.org/159075
------------------------------------------------------------------------
--
You received this message because you are listed in the owner
or CC fields of this issue, or because you starred this issue.
You may adjust your issue notification preferences at:
http://code.google.com/hosting/settings
--~--~---------~--~----~------------~-------~--~----~
Automated mail from issue updates at http://crbug.com/
Subscription options: http://groups.google.com/group/chromium-bugs
-~----------~----~----~----~------~----~------~--~---
