Fix FORWARD_NULL defects reported by Coverity....

Fix FORWARD_NULL defects reported by Coverity.  We should
also return an error if argv is NULL.

R=rvargas
BUG=http://crbug.com/17101
TEST=none

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

[rvargas (doing something else), 2009-07-22 02:56:51]

One of the changes is not correct. The other one LGTM.

Thanks.

http://codereview.chromium.org/155904/diff/1/2
File sandbox/src/process_policy_test.cc (right):

http://codereview.chromium.org/155904/diff/1/2#newcode41
Line 41: wchar_t *cmd_line = const_cast<wchar_t*>(command.c_str());
I don't think this is actually what we want. Passing NULL to CreateProcess is
OK; passing an empty string is not the same.

[cpu_(ooo_6.6-7.5), 2009-07-22 21:35:29]

LGTM