diff options
| author | shess@chromium.org <shess@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-21 03:57:47 +0000 |
|---|---|---|
| committer | shess@chromium.org <shess@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2010-10-21 03:57:47 +0000 |
| commit | 041fdee3a75759df5a36623a7cda1811f2dc578f (patch) | |
| tree | 032d4d9377b97d6c148a1fe05064a0ca49a6fcd6 /chrome/browser/safe_browsing | |
| parent | 947446bd2c19777f7f20b9b14085dc31cdb73d1a (diff) | |
| download | chromium_src-041fdee3a75759df5a36623a7cda1811f2dc578f.zip chromium_src-041fdee3a75759df5a36623a7cda1811f2dc578f.tar.gz chromium_src-041fdee3a75759df5a36623a7cda1811f2dc578f.tar.bz2 | |
Incorrect error return loading bloom filter.
http://codereview.chromium.org/3860003 mentioned an error I introduced
after the fact. Add the NULL return back where indicated.
TBR=jar@chromium.org
BUG=58401
TEST=none
Review URL: http://codereview.chromium.org/3963001
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@63326 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/safe_browsing')
| -rw-r--r-- | chrome/browser/safe_browsing/bloom_filter.cc | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/chrome/browser/safe_browsing/bloom_filter.cc b/chrome/browser/safe_browsing/bloom_filter.cc index a69873f..83d3e96 100644 --- a/chrome/browser/safe_browsing/bloom_filter.cc +++ b/chrome/browser/safe_browsing/bloom_filter.cc @@ -138,6 +138,7 @@ BloomFilter* BloomFilter::LoadFile(const FilePath& filter_name) { bytes_read = filter.Read(data.get(), byte_size, NULL); if (bytes_read < byte_size) { RecordFailure(FAILURE_FILTER_READ_DATA_SHORT); + return NULL; } else if (bytes_read != byte_size) { RecordFailure(FAILURE_FILTER_READ_DATA); return NULL; |