Set siginfo.si_pid to getpid() in SimulateSignalDelivery()

ExceptionHandler::HandleSignal (https://code.google.com/searchframe#OAMlx_jo-ck/src/breakpad/src/client/linux/handler/exception_handler.cc&exact_package=chromium&q=SI_USER&type=cs&l=389) requires si_pid to be equal to getpid() for SI_USER signals.

This is a patch by Lei Zhang (thestig@chromium.org) that fixes the issue with Chrome+ASan being unable to upload the crash dump to the crash server when the seccomp sandbox is on.

[glider1, 2013-01-10 08:57:10]

PTAL

[thestig, 2013-01-10 23:33:42]

On 2013/01/10 08:57:10, glider1 wrote:
> PTAL

Deferring to Mark since this is what I suggested.

[Mark Mentovai, 2013-01-14 17:33:22]

https://codereview.appspot.com/7066068/diff/1/breakpad/src/client/linux/handl...
File breakpad/src/client/linux/handler/exception_handler.cc (right):

https://codereview.appspot.com/7066068/diff/1/breakpad/src/client/linux/handl...
breakpad/src/client/linux/handler/exception_handler.cc:419: siginfo.si_code =
SI_USER;
Comment-worthy.

[glider1, 2013-01-14 18:02:11]

https://codereview.appspot.com/7066068/diff/1/breakpad/src/client/linux/handl...
File breakpad/src/client/linux/handler/exception_handler.cc (right):

https://codereview.appspot.com/7066068/diff/1/breakpad/src/client/linux/handl...
breakpad/src/client/linux/handler/exception_handler.cc:419: siginfo.si_code =
SI_USER;
On 2013/01/14 17:33:22, Mark Mentovai wrote:
> Comment-worthy.

Done.

[Mark Mentovai, 2013-01-14 18:03:14]

LGTM

[glider1, 2013-01-14 18:07:39]

On 2013/01/14 18:03:14, Mark Mentovai wrote:
> LGTM

Can you please also land this?

[Mark Mentovai, 2013-01-14 18:53:27]

Committed Breakpad r1095

[glider, 2013-01-14 18:56:03]

Thanks a lot!
On Jan 14, 2013 10:53 PM, <mark@chromium.org> wrote:

> Committed Breakpad r1095
>
>
https://codereview.appspot.**com/7066068/<https://codereview.appspot.com/7066...
>