Comment #10 on issue 17101 by [email protected]: Review FORWARD_NULL
defects reported by Coverity in Chromium code base.
http://code.google.com/p/chromium/issues/detail?id=17101
The following revision refers to this bug:
http://src.chromium.org/viewvc/chrome?view=rev&revision=21702
------------------------------------------------------------------------
r21702 | [email protected] | 2009-07-27 13:58:07 -0700 (Mon, 27 Jul
2009) | 16 lines
Changed paths:
M
http://src.chromium.org/viewvc/chrome/branches/195/src/sandbox/src/sandbox_nt_util.cc?r1=21702&r2=21701
M
http://src.chromium.org/viewvc/chrome/branches/195/src/sandbox/src/sandbox_nt_util.h?r1=21702&r2=21701
Merge 21404 - Fix a FORWARD_NULL defect in ExtractModuleName reported by
Coverity.
If 'sep' is still NULL after the for loop, ix must be 1, so
ix == 0 cannot be true.
Update the comment for ExtractModuleName in the header to match
the implementation. I don't see any code that checks whether the
path is a full path.
R=rvargas
BUG=http://crbug.com/17101
TEST=none
Review URL: http://codereview.chromium.org/155979
[email protected]
Review URL: http://codereview.chromium.org/160196
------------------------------------------------------------------------
--
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
-~----------~----~----~----~------~----~------~--~---
