diff options
| author | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-22 03:05:50 +0000 |
|---|---|---|
| committer | thestig@chromium.org <thestig@chromium.org@0039d316-1c4b-4281-b951-d872f2087c98> | 2011-06-22 03:05:50 +0000 |
| commit | 48aa6c8da84a6dbe8ceb9f229c7185a2bd97db68 (patch) | |
| tree | 69a630013eb206aac6985c8b2970de2d84a6294b | |
| parent | 98d87fb010ceed978ddd94abd465399f1413539c (diff) | |
| download | chromium_src-48aa6c8da84a6dbe8ceb9f229c7185a2bd97db68.zip chromium_src-48aa6c8da84a6dbe8ceb9f229c7185a2bd97db68.tar.gz chromium_src-48aa6c8da84a6dbe8ceb9f229c7185a2bd97db68.tar.bz2 | |
Coverity: Check a return value.
BUG=none
TEST=none
CID=13577
Review URL: http://codereview.chromium.org/7215036
git-svn-id: svn://svn.chromium.org/chrome/trunk/src@89968 0039d316-1c4b-4281-b951-d872f2087c98
| -rw-r--r-- | chrome/browser/browser_signin.cc | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/chrome/browser/browser_signin.cc b/chrome/browser/browser_signin.cc index f30c227..6c99f65 100644 --- a/chrome/browser/browser_signin.cc +++ b/chrome/browser/browser_signin.cc @@ -4,6 +4,7 @@ #include "chrome/browser/browser_signin.h" +#include <algorithm> #include <string> #include <vector> @@ -181,7 +182,8 @@ void BrowserSigninHtml::HandleSigninInit(const ListValue* args) { // particular value. bool is_editable; std::string user; - json_args.GetBoolean("editable_user", &is_editable); + if (!json_args.GetBoolean("editable_user", &is_editable)) + is_editable = false; json_args.GetString("user", &user); if (is_editable && user.empty() && !suggested_email_.empty()) json_args.SetString("user", suggested_email_); |