Rietveld Code Review Tool
Help | Bug tracker | Discussion group | Source code | Sign in
(11)

Issue 10724003: Fix call to rt_sigaction

Can't Edit
Can't Publish+Mail
Start Review
Created:
1 year, 10 months ago by cjhopman
Modified:
1 year, 10 months ago
Reviewers:
Mark Mentovai
Base URL:
https://chromium.googlesource.com/external/google-breakpad/src.git@master
Visibility:
Public.

Description

Fix call to rt_sigaction

Despite the fact that many places imply that sigaction and rt_sigaction
are essentially the same, rt_sigaction's signature is actually
different-- it takes the size of the kernel's sigset_t as an extra argument.

This change uses the kernel_sigaction, kernel_sigset_t and sys_rt_sigaction
provided by linux_syscall_support.h instead of rolling our own.

BUG=473973

Patch Set 1 : #

Unified diffs Side-by-side diffs Delta from patch set Stats Patch
M client/linux/handler/exception_handler.cc View 1 chunk +4 lines, -4 lines 0 comments Download

Messages

Total messages: 2
cjhopman
mark: * One more for ya.
1 year, 10 months ago #1
Mark Mentovai
1 year, 10 months ago #2
LGTM. Good luck.
Sign in to reply to this message.

Powered by Google App Engine
RSS Feeds Recent Issues | This issue
This is Rietveld 1004:630ec63f810e-tainted