Fix a FORWARD_NULL defect reported by Coverity....

Fix a FORWARD_NULL defect reported by Coverity.

If 'profile' is NULL, we should return
ResultCodes::NORMAL_EXIT so that we won't go on to
dereference 'profile'.

R=brettw,estade
BUG=http://crbug.com/17101
TEST=none

Committed: http://src.chromium.org/viewvc/chrome?view=rev&revision=24469

[wtc, 2009-08-15 00:22:38]

http://codereview.chromium.org/171007/diff/1/2
File chrome/browser/browser_main.cc (left):

http://codereview.chromium.org/171007/diff/1/2#oldcode542
Line 542: #endif  // defined(OS_WIN) || defined(OS_LINUX)
An alternative is to add

+#else
+    return ResultCode::MISSING_DATA;  // or some other result code
 #endif  // defined(OS_WIN) || defined(OS_LINUX)

I don't know the code well enough to know what's the right
return code for Mac here.

[Evan Stade, 2009-08-15 00:35:00]

I also don't know what's right for mac but this seems fine to me.

[brettw, 2009-08-15 17:54:42]

LGTM 2

On Fri, Aug 14, 2009 at 5:35 PM, <estade@chromium.org> wrote:
> I also don't know what's right for mac but this seems fine to me.
>
> http://codereview.chromium.org/171007
>