diff options
| author | dpolukhin@chromium.org <dpolukhin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-26 13:04:47 +0000 |
|---|---|---|
| committer | dpolukhin@chromium.org <dpolukhin@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2014-06-26 13:04:47 +0000 |
| commit | 07a20e1eced09635f0ce0b80cbf927870700f177 (patch) | |
| tree | 8242ef493d0d710f61e19390059521ccb5fedc8f /extensions/browser | |
| parent | a20a31ccc9c1fff1700110ad03206c50a54fe686 (diff) | |
| download | chromium_src-07a20e1eced09635f0ce0b80cbf927870700f177.zip chromium_src-07a20e1eced09635f0ce0b80cbf927870700f177.tar.gz chromium_src-07a20e1eced09635f0ce0b80cbf927870700f177.tar.bz2 | |
Fix crash in ContentVerifier::ShouldVerifyAnyPaths when deleagte_ is NULL
My previous patch removed this check because I thought that all code paths have
such check but it is not the case so I returned this check.
BUG=388797
TBR= asargent@chromium.org
TEST=N/A
Review URL: https://codereview.chromium.org/352633008
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@279992 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'extensions/browser')
| -rw-r--r-- | extensions/browser/content_verifier.cc | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/extensions/browser/content_verifier.cc b/extensions/browser/content_verifier.cc index 97e2de2..9653435 100644 --- a/extensions/browser/content_verifier.cc +++ b/extensions/browser/content_verifier.cc @@ -124,7 +124,7 @@ void ContentVerifier::OnFetchComplete( bool ContentVerifier::ShouldVerifyAnyPaths( const Extension* extension, const std::set<base::FilePath>& relative_paths) { - if (!extension || !extension->version()) + if (!delegate_ || !extension || !extension->version()) return false; ContentVerifierDelegate::Mode mode = delegate_->ShouldBeVerified(*extension); |