Closed Bug 346843 Opened 18 years ago Closed 18 years ago

enable nsSigHandlers trapping for x86_64

Categories

(Core :: XUL, defect)

x86
Linux
defect
Not set
normal

Tracking

()

RESOLVED FIXED

People

(Reporter: sylvain.pasche, Assigned: jag+mozilla)

References

Details

Attachments

(2 files)

User-Agent:       Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.8.1b1) Gecko/20060801 BonEcho/2.0b1
Build Identifier: Mozilla/5.0 (X11; U; Linux x86_64; en-US; rv:1.8.1b1) Gecko/20060801 BonEcho/2.0b1

patch coming

Reproducible: Always
Attached patch patch, v1Splinter Review
That is working fine for me. But I don't know it was not enabled for other more complex reasons.
Attachment #231575 - Flags: review?(bryner)
Status: UNCONFIRMED → NEW
Ever confirmed: true
s/know it was/know if it was/
Attachment #231575 - Flags: review?(bryner) → review+
Comment on attachment 231575 [details] [diff] [review]
patch, v1

I think I forgot to ask for super-review (I took bug 333538 as example). Can you commit that afterwards?
Attachment #231575 - Flags: superreview?(bryner)
Attachment #231575 - Flags: superreview?(bryner) → superreview+
Checked in, thanks.
Status: NEW → RESOLVED
Closed: 18 years ago
Resolution: --- → FIXED
It's probably severely broken in debug+opt builds on newer gcc's -- at least it was the last time I tested.
(In reply to comment #5)
> It's probably severely broken in debug+opt builds on newer gcc's -- at least it
> was the last time I tested.
> 

I have been testing this with gcc 4.0. Maybe I will try with gcc 4.1.

By "severly broken", you mean that the adresses reported by the stacktrace are not correct?
I can attach some output there if helpful to diagnose the issues.
I mean that not only are large numbers of stack frames missing, but it often crashed somewhere in the middle of the stack when EBP was used for something other than the frame pointer, especially when the stack went through system libraries which often ship without the frame pointer.
*** Bug 347287 has been marked as a duplicate of this bug. ***
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Creator:
Created:
Updated:
Size: