Closed
Bug 743306
Opened 12 years ago
Closed 12 years ago
Fix MOZ_Assert and MOZ_Crash for Android
Categories
(Core :: MFBT, defect)
Tracking
()
RESOLVED
FIXED
mozilla14
People
(Reporter: cpeterson, Assigned: cpeterson)
Details
Attachments
(2 files)
1.18 KB,
patch
|
glandium
:
review+
|
Details | Diff | Splinter Review |
1.17 KB,
patch
|
glandium
:
review+
|
Details | Diff | Splinter Review |
No description provided.
Assignee | ||
Comment 1•12 years ago
|
||
Android's stdio and stderr are directed to /dev/null. On Android, log MOZ_Assert message to adb logcat instead of fprintf(stderr).
Attachment #612953 -
Flags: review?(mh+mozilla)
Assignee | ||
Comment 2•12 years ago
|
||
On Android, segfault in MOZ_Crash because raise(SIGABRT) is handled asynchronously. When MOZ_Crash raises SIGABRT, the process does not die immediately. MOZ_Crash returns and the thread continues running. When all threads in the process receives a SIBABRT, the current thread's call stack will not point to MOZ_Crash. If we segfault in MOZ_Crash, we crash immediately and capture the current call stack pointing to MOZ_Crash. Android's dalvikvm and abort() segfault for the same reasons.
Attachment #612955 -
Flags: review?(mh+mozilla)
Updated•12 years ago
|
Attachment #612953 -
Flags: review?(mh+mozilla) → review+
Updated•12 years ago
|
Attachment #612955 -
Flags: review?(mh+mozilla) → review+
Updated•12 years ago
|
Assignee: nobody → cpeterson
Assignee | ||
Comment 3•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/8a7965116a33 https://hg.mozilla.org/integration/mozilla-inbound/rev/42870fe1a4a3
Assignee | ||
Updated•12 years ago
|
Status: NEW → ASSIGNED
Comment 4•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/8a7965116a33 https://hg.mozilla.org/mozilla-central/rev/42870fe1a4a3
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla14
You need to log in
before you can comment on or make changes to this bug.
Description
•