Fix a stack overflow in the windows sandbox SpawnTarget function.

Fix a stack overflow in the windows sandbox SpawnTarget function.

This was caused by my recent change to allow handles other than STDOUT and STDERR to be shared
with the target. Reason for the crash was copying additional handles to the HANDLE array which had
space for 2 handles only.

Fix is to use scoped_ptr instead and allocate appropriate space for all handles being shared.

BUG=486434
R=cpu

Committed: https://crrev.com/a6ddf97af304f1aa7e88e28473e593e2c08c9157
Cr-Commit-Position: refs/heads/master@{#329276}

[cpu_(ooo_6.6-7.5), 2015-05-11 22:35:45]

https://codereview.chromium.org/1128903006/diff/1/sandbox/win/src/broker_serv...
File sandbox/win/src/broker_services.cc (right):

https://codereview.chromium.org/1128903006/diff/1/sandbox/win/src/broker_serv...
sandbox/win/src/broker_services.cc:419: new HANDLE[policy_handle_list.size() +
2]);
this is too strange now use
std::vector<HANDLE> inherit_handle_list 

and instead of inherit_handle_count use inherit_handle_list.push_back()

[ananta, 2015-05-11 22:42:24]

https://codereview.chromium.org/1128903006/diff/1/sandbox/win/src/broker_serv...
File sandbox/win/src/broker_services.cc (right):

https://codereview.chromium.org/1128903006/diff/1/sandbox/win/src/broker_serv...
sandbox/win/src/broker_services.cc:419: new HANDLE[policy_handle_list.size() +
2]);
On 2015/05/11 22:35:45, cpu wrote:
> this is too strange now use
> std::vector<HANDLE> inherit_handle_list 
> 
> and instead of inherit_handle_count use inherit_handle_list.push_back()
> 

Done.

[cpu_(ooo_6.6-7.5), 2015-05-11 23:33:14]

lgtm

[cpu_(ooo_6.6-7.5), 2015-05-11 23:33:20]

The CQ bit was checked by cpu@chromium.org

[commit-bot: I haz the power, 2015-05-11 23:35:43]

CQ is trying da patch. Follow status at
 https://chromium-cq-status.appspot.com/patch-status/1128903006/20001

[commit-bot: I haz the power, 2015-05-11 23:46:05]

Committed patchset #2 (id:20001)

[commit-bot: I haz the power, 2015-05-11 23:47:50]

Patchset 2 (id:??) landed as
https://crrev.com/a6ddf97af304f1aa7e88e28473e593e2c08c9157
Cr-Commit-Position: refs/heads/master@{#329276}