Closed
Bug 967553
Opened 11 years ago
Closed 11 years ago
crash in throwSqliteException
Categories
(Firefox for Android Graveyard :: General, defect)
Tracking
(firefox29+ fixed, firefox30 fixed)
RESOLVED
FIXED
Firefox 30
People
(Reporter: aaronmt, Assigned: jchen)
References
Details
(Keywords: crash, regression)
Crash Data
Attachments
(1 file)
2.65 KB,
patch
|
gcp
:
review+
Sylvestre
:
approval-mozilla-aurora+
|
Details | Diff | Splinter Review |
This bug was filed from the Socorro interface and is
report bp-65315147-df79-4aa3-a1d2-ea9d32140203.
=============================================================
Assignee | ||
Comment 1•11 years ago
|
||
Looks like bug 964859 didn't fix the real issue. The real problem is a regression from bug 801571.
Assignee | ||
Comment 2•11 years ago
|
||
Assignee | ||
Comment 3•11 years ago
|
||
Comment on attachment 8370149 [details] [diff] [review]
Restore to using __wrap_free
Changing back to free() will give us back our wrapped free().
Attachment #8370149 -
Attachment description: Bug 967553 - Restore to using __wrap_free; r= → Restore to using __wrap_free
Attachment #8370149 -
Flags: review?(gpascutto)
Comment 4•11 years ago
|
||
Comment on attachment 8370149 [details] [diff] [review]
Restore to using __wrap_free
Review of attachment 8370149 [details] [diff] [review]:
-----------------------------------------------------------------
Well at least that made things simpler on our side.
Attachment #8370149 -
Flags: review?(gpascutto) → review+
Assignee | ||
Updated•11 years ago
|
Keywords: checkin-needed
Comment 5•11 years ago
|
||
Keywords: checkin-needed
Whiteboard: [fixed-in-fx-team]
Comment 6•11 years ago
|
||
Status: ASSIGNED → RESOLVED
Closed: 11 years ago
Resolution: --- → FIXED
Whiteboard: [fixed-in-fx-team]
Target Milestone: --- → Firefox 30
Updated•11 years ago
|
Assignee | ||
Comment 7•11 years ago
|
||
Comment on attachment 8370149 [details] [diff] [review]
Restore to using __wrap_free
[Approval Request Comment]
Bug caused by (feature/regressing bug #): Bug 801571
User impact if declined: Crashes
Testing completed (on m-c, etc.): Locally, m-c
Risk to taking this patch (and alternatives if risky): Very small; patch just fixes crash
String or IDL/UUID changes made by this patch: None
Attachment #8370149 -
Flags: approval-mozilla-aurora?
Updated•11 years ago
|
Attachment #8370149 -
Flags: approval-mozilla-aurora? → approval-mozilla-aurora+
Updated•11 years ago
|
Comment 8•11 years ago
|
||
Updated•4 years ago
|
Product: Firefox for Android → Firefox for Android Graveyard
You need to log in
before you can comment on or make changes to this bug.
Description
•