diff options
| author | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-11 09:33:42 +0000 |
|---|---|---|
| committer | finnur@chromium.org <finnur@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-11 09:33:42 +0000 |
| commit | fb8df11f00fd768825238f630dc1b6fa97e9844e (patch) | |
| tree | e2e01f921199787af3b2af80b4f069cc68d531e7 /chrome/browser/diagnostics/recon_diagnostics.cc | |
| parent | 039370f20f575b722c625cea6c927fb964c059e5 (diff) | |
| download | chromium_src-fb8df11f00fd768825238f630dc1b6fa97e9844e.zip chromium_src-fb8df11f00fd768825238f630dc1b6fa97e9844e.tar.gz chromium_src-fb8df11f00fd768825238f630dc1b6fa97e9844e.tar.bz2 | |
Fix minor CHECKED_RETURN Coverity complaint in code I added to the --diagnostics code.
BUG=None
TEST=None
CID=14387
Review URL: http://codereview.chromium.org/6171004
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71027 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/diagnostics/recon_diagnostics.cc')
| -rw-r--r-- | chrome/browser/diagnostics/recon_diagnostics.cc | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/diagnostics/recon_diagnostics.cc b/chrome/browser/diagnostics/recon_diagnostics.cc index 2cacd86..5752014 100644 --- a/chrome/browser/diagnostics/recon_diagnostics.cc +++ b/chrome/browser/diagnostics/recon_diagnostics.cc @@ -97,7 +97,8 @@ class ConflictingDllsTest : public DiagnosticTest { string16 failures = ASCIIToUTF16("Possibly conflicting modules:"); DictionaryValue* dictionary; for (size_t i = 0; i < list->GetSize(); ++i) { - list->GetDictionary(i, &dictionary); + if (!list->GetDictionary(i, &dictionary)) + RecordFailure(ASCIIToUTF16("Dictionary lookup failed")); int status; string16 location; string16 name; |