diff options
| author | kmadhusu@chromium.org <kmadhusu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-12 17:00:52 +0000 |
|---|---|---|
| committer | kmadhusu@chromium.org <kmadhusu@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-01-12 17:00:52 +0000 |
| commit | 72f891f407cc2ccd132328e97f1b4dd5eaab72b2 (patch) | |
| tree | 92d8cc652d284024de6ea0a50e16888e03babaef /chrome/browser/diagnostics | |
| parent | b2b0fceea58bf7e13cc73020133aff6d6b4f4640 (diff) | |
| download | chromium_src-72f891f407cc2ccd132328e97f1b4dd5eaab72b2.zip chromium_src-72f891f407cc2ccd132328e97f1b4dd5eaab72b2.tar.gz chromium_src-72f891f407cc2ccd132328e97f1b4dd5eaab72b2.tar.bz2 | |
Coverity: Fix unchecked return values.
CID=13943, 13746, 13749, 13597, 13249, 13145, 12875, 12874, 12871, 12438, 12436, 12431
BUG=none
TEST=none
Review URL: http://codereview.chromium.org/6076006
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@71179 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/diagnostics')
| -rw-r--r-- | chrome/browser/diagnostics/recon_diagnostics.cc | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/chrome/browser/diagnostics/recon_diagnostics.cc b/chrome/browser/diagnostics/recon_diagnostics.cc index 5752014..ac0548b 100644 --- a/chrome/browser/diagnostics/recon_diagnostics.cc +++ b/chrome/browser/diagnostics/recon_diagnostics.cc @@ -328,7 +328,11 @@ class JSONTest : public DiagnosticTest { return true; } int64 file_size; - file_util::GetFileSize(path_, &file_size); + if (!file_util::GetFileSize(path_, &file_size)) { + RecordFailure(ASCIIToUTF16("Cannot obtain file size")); + return true; + } + if (file_size > max_file_size_) { RecordFailure(ASCIIToUTF16("File too big")); return true; |