Closed
Bug 1193069
Opened 9 years ago
Closed 9 years ago
Crash on update to Build 30
Categories
(Firefox for iOS :: Data Storage, defect)
Tracking
()
RESOLVED
FIXED
Tracking | Status | |
---|---|---|
fxios | + | --- |
People
(Reporter: rnewman, Assigned: rnewman)
Details
(Keywords: crash)
Attachments
(1 file)
https://gist.github.com/sleroux/968b706df81ac45fb42a#file-testflight-crash Storage!_TFFC7Storage12BrowserTable11updateTableFS0_FTCS_18SQLiteDBConnection4fromSi2toSi_SbU_FCS_5SDRowSS + 0x108 fp = 0x000000016fdfc850 lr = 0x00000001005b11f4 sp = 0x000000016fdfc840 pc = 0x00000001005c4c14 Found by: previous frame's frame pointer
Assignee | ||
Comment 1•9 years ago
|
||
let urls = db.executeQuery("SELECT DISTINCT url FROM \(TableHistory)", factory: { $0["url"] as! String }) versus "url TEXT UNIQUE, " + // May only be null for deleted records. That looks like one possible culprit.
Assignee | ||
Comment 2•9 years ago
|
||
I can repro with a SIGABRT at that line with a similar stack if I null out a URL in the DB. The log shows Could not cast value of type '__NSCFNumber' (0x10cd8ccf0) to 'NSString' (0x10bebd8e0). which is presumably a null -> 0 sqlite/SwiftData confusion.
Assignee | ||
Comment 3•9 years ago
|
||
Attachment #8646082 -
Flags: review?(sleroux)
Attachment #8646082 -
Flags: review?(sarentz)
Attachment #8646082 -
Flags: review?(bnicholson)
Comment 4•9 years ago
|
||
Comment on attachment 8646082 [details] [review] Pull req. The code looks good. What scenario would we have a null url? Would this happen if we try restoring a saved 'home panel' tab?
Attachment #8646082 -
Flags: review?(sleroux) → review+
Updated•9 years ago
|
Attachment #8646082 -
Flags: review?(bnicholson) → review+
Assignee | ||
Comment 5•9 years ago
|
||
This would happen if you have a deleted history item that hasn't yet synced to the server. There might be other root causes that I haven't yet identified.
Assignee | ||
Comment 6•9 years ago
|
||
b48f840
Status: ASSIGNED → RESOLVED
Closed: 9 years ago
Resolution: --- → FIXED
Assignee | ||
Updated•9 years ago
|
Attachment #8646082 -
Flags: review?(sarentz)
You need to log in
before you can comment on or make changes to this bug.
Description
•