Closed
Bug 1317802
Opened 7 years ago
Closed 7 years ago
don't stop for SIGSYS in .gdbinit
Categories
(Core :: General, defect)
Core
General
Tracking
()
RESOLVED
FIXED
mozilla53
Tracking | Status | |
---|---|---|
firefox53 | --- | fixed |
People
(Reporter: froydnj, Assigned: froydnj)
Details
Attachments
(1 file)
858 bytes,
patch
|
jld
:
review+
|
Details | Diff | Splinter Review |
The sandboxing code generates this signal nowadays, which makes debugging with tools like rr quite frustrating.
Assignee | ||
Comment 1•7 years ago
|
||
Picking jld because he probably understands the sandboxing code and how this would interact with rr? Feel free to redirect.
Attachment #8811024 -
Flags: review?(jld)
Comment 2•7 years ago
|
||
Comment on attachment 8811024 [details] [diff] [review] don't stop for SIGSYS in .gdbinit Seems like a good idea — for all practical purposes, SIGSYS is now an implementation detail / expected side effect of calls like open(), and unless you're working on sandboxing or debugging a sandboxing issue you won't want to stop on it. Printing the signal could have some value, but usually it will just be confusing and annoying, especially because gdb will stop and prompt after every screenful of messages, so turning that off is fine too.
Attachment #8811024 -
Flags: review?(jld) → review+
Comment 3•7 years ago
|
||
As for rr, there were some issues with using the SIGSYS handler to polyfill syscalls with other syscalls but they've been fixed, so as far as I know it should now be faithfully reproducing the SIGSYS-related irritation level of a regular debugging session (modulo sometimes changing the direction of time).
Pushed by nfroyd@mozilla.com: https://hg.mozilla.org/integration/mozilla-inbound/rev/fa4c2f76351d don't stop for SIGSYS in .gdbinit; r=jld
Comment 5•7 years ago
|
||
bugherder |
https://hg.mozilla.org/mozilla-central/rev/fa4c2f76351d
Status: NEW → RESOLVED
Closed: 7 years ago
status-firefox53:
--- → fixed
Resolution: --- → FIXED
Target Milestone: --- → mozilla53
You need to log in
before you can comment on or make changes to this bug.
Description
•