summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorthestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-06-22 03:06:09 +0000
committerthestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98>2011-06-22 03:06:09 +0000
commit2f9acefea88a314685423340b0e81ac8c5b63982 (patch)
tree7e35c1d18e0669bd5d9bae982325a25f05b38256
parent48aa6c8da84a6dbe8ceb9f229c7185a2bd97db68 (diff)
downloadchromium_src-2f9acefea88a314685423340b0e81ac8c5b63982.zip
chromium_src-2f9acefea88a314685423340b0e81ac8c5b63982.tar.gz
chromium_src-2f9acefea88a314685423340b0e81ac8c5b63982.tar.bz2
Coverity: Check a return value in ProtocolHandlerRegistry.
BUG=none TEST=none CID=16865 Review URL: http://codereview.chromium.org/7217021 git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89969 0039d316-1c4b-4281-b951-d872f2087c98
-rw-r--r--chrome/browser/custom_handlers/protocol_handler_registry.cc3
1 files changed, 2 insertions, 1 deletions
diff --git a/chrome/browser/custom_handlers/protocol_handler_registry.cc b/chrome/browser/custom_handlers/protocol_handler_registry.cc
index 1fb661b..d8714f9 100644
--- a/chrome/browser/custom_handlers/protocol_handler_registry.cc
+++ b/chrome/browser/custom_handlers/protocol_handler_registry.cc
@@ -141,7 +141,8 @@ ProtocolHandlerRegistry::GetHandlersFromPref(const char* pref_name) const {
if (handlers) {
for (size_t i = 0; i < handlers->GetSize(); ++i) {
DictionaryValue* dict;
- handlers->GetDictionary(i, &dict);
+ if (!handlers->GetDictionary(i, &dict))
+ continue;
if (ProtocolHandler::IsValidDict(dict)) {
result.push_back(dict);
}