diff options
author | csilv@chromium.org <csilv@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-03-11 19:44:08 +0000 |
---|---|---|
committer | csilv@chromium.org <csilv@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-03-11 19:44:08 +0000 |
commit | 8874e02178746958ffdf674866bc86087f7dea46 (patch) | |
tree | f92c85b60b51c922f348c28853ffcdfccef9f80b /chrome/browser/prefs | |
parent | 23db9f7756b518c5655e44e3280f9af7df91879a (diff) | |
download | chromium_src-8874e02178746958ffdf674866bc86087f7dea46.zip chromium_src-8874e02178746958ffdf674866bc86087f7dea46.tar.gz chromium_src-8874e02178746958ffdf674866bc86087f7dea46.tar.bz2 |
Coverity: Fix various NULL_RETURNS defects.
CID=8699, 13592, 14444, 14510, 14658, 14805, 14897
BUG=none
Review URL: http://codereview.chromium.org/6672016
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@77845 0039d316-1c4b-4281-b951-d872f2087c98
Diffstat (limited to 'chrome/browser/prefs')
-rw-r--r-- | chrome/browser/prefs/browser_prefs.cc | 7 | ||||
-rw-r--r-- | chrome/browser/prefs/pref_service.cc | 5 |
2 files changed, 8 insertions, 4 deletions
diff --git a/chrome/browser/prefs/browser_prefs.cc b/chrome/browser/prefs/browser_prefs.cc index b48945b..9200d5d 100644 --- a/chrome/browser/prefs/browser_prefs.cc +++ b/chrome/browser/prefs/browser_prefs.cc @@ -176,9 +176,10 @@ void MigrateBrowserPrefs(PrefService* user_prefs, PrefService* local_state) { local_state->RegisterDictionaryPref(prefs::kBrowserWindowPlacement); DCHECK(user_prefs->FindPreference(prefs::kBrowserWindowPlacement)); if (local_state->HasPrefPath(prefs::kBrowserWindowPlacement)) { - user_prefs->Set(prefs::kBrowserWindowPlacement, - *(local_state->FindPreference(prefs::kBrowserWindowPlacement)-> - GetValue())); + const PrefService::Preference* pref = + local_state->FindPreference(prefs::kBrowserWindowPlacement); + DCHECK(pref); + user_prefs->Set(prefs::kBrowserWindowPlacement, *(pref->GetValue())); } local_state->ClearPref(prefs::kBrowserWindowPlacement); diff --git a/chrome/browser/prefs/pref_service.cc b/chrome/browser/prefs/pref_service.cc index 7c17a6f..5afb08f 100644 --- a/chrome/browser/prefs/pref_service.cc +++ b/chrome/browser/prefs/pref_service.cc @@ -377,7 +377,10 @@ DictionaryValue* PrefService::GetPreferenceValues() const { DictionaryValue* out = new DictionaryValue; DefaultPrefStore::const_iterator i = default_store_->begin(); for (; i != default_store_->end(); ++i) { - const Value* value = FindPreference(i->first.c_str())->GetValue(); + const Preference* pref = FindPreference(i->first.c_str()); + DCHECK(pref); + const Value* value = pref->GetValue(); + DCHECK(value); out->Set(i->first, value->DeepCopy()); } return out; |