Closed
Bug 723029
Opened 12 years ago
Closed 12 years ago
Crash in mozilla::storage::Service::UpdateQutoaInformationForFile @ `anonymous namespace''::xOpen
Categories
(Toolkit :: Telemetry, defect)
Tracking
()
VERIFIED
FIXED
mozilla13
People
(Reporter: scoobidiver, Assigned: khuey)
Details
(Keywords: crash, topcrash, Whiteboard: [qa!])
Crash Data
Attachments
(1 file)
772 bytes,
patch
|
khuey
:
review+
akeybl
:
approval-mozilla-aurora+
akeybl
:
approval-mozilla-beta+
lsblakk
:
approval-mozilla-esr10+
|
Details | Diff | Splinter Review |
70% of crashes happen within one minute. Signature `anonymous namespace''::xOpen(sqlite3_vfs*, char const*, sqlite3_file*, int, int*) More Reports Search UUID 33991ed1-d358-445e-a265-775392120201 Date Processed 2012-02-01 09:22:13 Uptime 3 Last Crash 7 seconds before submission Install Age 14.5 hours since version was first installed. Install Time 2012-01-31 19:51:04 Product Firefox Version 10.0 Build ID 20120129021758 Release Channel release OS Windows NT OS Version 5.1.2600 Service Pack 3 Build Architecture x86 Build Architecture Info GenuineIntel family 6 model 37 stepping 2 Crash Reason EXCEPTION_ACCESS_VIOLATION_READ Crash Address 0xe App Notes AdapterVendorID: 1002, AdapterDeviceID: 68f9, AdapterSubsysID: 20081787, AdapterDriverVersion: 8.753.0.0 D3D10 Layers? D3D10 Layers- EMCheckCompatibility True Frame Module Signature [Expand] Source 0 xul.dll `anonymous namespace'::xOpen storage/src/TelemetryVFS.cpp:330 1 xul.dll quotaOpen db/sqlite3/src/test_quota.c:373 2 xul.dll sqlite3_quota_file db/sqlite3/src/test_quota.c:830 3 xul.dll mozilla::storage::Service::UpdateQutoaInformationForFile storage/src/mozStorageService.cpp:721 4 xul.dll mozilla::dom::indexedDB::IndexedDatabaseManager::EnsureQuotaManagementForDirectory dom/indexeddb/IndexedDatabaseManager.cpp:653 5 xul.dll mozilla::dom::indexedDB::OpenDatabaseHelper::DoDatabaseWork dom/indexeddb/OpenDatabaseHelper.cpp:749 6 xul.dll mozilla::dom::indexedDB::OpenDatabaseHelper::Run dom/indexeddb/OpenDatabaseHelper.cpp:988 7 xul.dll nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:631 8 xul.dll nsThreadStartupEvent::Run xpcom/threads/nsThread.cpp:201 9 nspr4.dll _PR_NativeRunThread nsprpub/pr/src/threads/combined/pruthr.c:426 10 nspr4.dll pr_root nsprpub/pr/src/md/windows/w95thred.c:122 11 msvcr80.dll _callthreadstartex f:\\sp\\vctools\\crt_bld\\self_x86\\crt\\src\\threadex.c:348 12 msvcr80.dll _threadstartex f:\\sp\\vctools\\crt_bld\\self_x86\\crt\\src\\threadex.c:326 13 kernel32.dll BaseThreadStart More reports at: https://crash-stats.mozilla.com/report/list?signature=%60anonymous%20namespace%27%27%3A%3AxOpen%28sqlite3_vfs*%2C%20char%20const*%2C%20sqlite3_file*%2C%20int%2C%20int*%29
Comment 1•12 years ago
|
||
ccing bent who better knows the quota management code
Assignee | ||
Comment 2•12 years ago
|
||
See comment 6 and the TelemetryVFS hunk in Bug 702292. Marco, can you land that hunk and get approval for relevant branches? It has r=taras.
Comment 3•12 years ago
|
||
sorry but I can't follow all that happened in that bug, may I get a proper patch attached here, will be easier for everyone, especially for approvals.
Comment 4•12 years ago
|
||
setting up according on existence of a patch somewhere in Bug 702292.
Assignee: nobody → khuey
Status: NEW → ASSIGNED
Assignee | ||
Updated•12 years ago
|
Whiteboard: startupcrash
Reporter | ||
Comment 5•12 years ago
|
||
85% of crashes happen within one minute. Does this qualify that crash for the startupcrash keyword?
Assignee | ||
Comment 6•12 years ago
|
||
Attachment #594436 -
Flags: review+
Attachment #594436 -
Flags: approval-mozilla-beta?
Attachment #594436 -
Flags: approval-mozilla-aurora?
Assignee | ||
Comment 7•12 years ago
|
||
I'm moving across the country this weekend, so I would appreciate someone else landing this.
Keywords: checkin-needed
Assignee | ||
Updated•12 years ago
|
Component: Storage → Telemetry
QA Contact: storage → telemetry
Assignee | ||
Comment 8•12 years ago
|
||
(In reply to Scoobidiver from comment #5) > 85% of crashes happen within one minute. Does this qualify that crash for > the startupcrash keyword? I removed the startupcrash annotation because this is triggered by webpages using IndexedDB (which hits the underlying bug in TelemetryVFS.cpp). This is only a crash at startup if users are unfortunate enough to have a page using IndexedDB load at startup.
Comment 9•12 years ago
|
||
https://hg.mozilla.org/integration/mozilla-inbound/rev/543de57b25f3
Keywords: checkin-needed
Target Milestone: --- → mozilla13
Comment 10•12 years ago
|
||
https://hg.mozilla.org/mozilla-central/rev/543de57b25f3
Status: ASSIGNED → RESOLVED
Closed: 12 years ago
Resolution: --- → FIXED
Comment 11•12 years ago
|
||
Comment on attachment 594436 [details] [diff] [review] Patch [Triage Comment] Top startup crasher - approved for Beta 11 and Aurora 12.
Attachment #594436 -
Flags: approval-mozilla-beta?
Attachment #594436 -
Flags: approval-mozilla-beta+
Attachment #594436 -
Flags: approval-mozilla-aurora?
Attachment #594436 -
Flags: approval-mozilla-aurora+
Assignee | ||
Comment 12•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-aurora/rev/c9fb84b0331e https://hg.mozilla.org/releases/mozilla-beta/rev/618a6670c5ed
status1.9.2:
--- → unaffected
status-firefox-esr10:
--- → affected
status-firefox10:
--- → affected
status-firefox11:
--- → fixed
status-firefox12:
--- → fixed
tracking-firefox-esr10:
--- → ?
Comment 13•12 years ago
|
||
How can i verify the fixing of this bug? Any info would be appreciated.
Comment 14•12 years ago
|
||
Kyle, are there any correlations with this crash that QA can try testing to help verify this fix?
Updated•12 years ago
|
Updated•12 years ago
|
Comment 15•12 years ago
|
||
Please land on mozilla-esr10 as soon as possible. For more information on the landing process, please see https://wiki.mozilla.org/Release_Management/ESR_Landing_Process
Comment 16•12 years ago
|
||
I have checked the crash reports and found 3 crashes with this signature ( on builds newer than the fixes: https://crash-stats.mozilla.com/report/index/e11b4ad3-0465-437c-b92b-564212120220 https://crash-stats.mozilla.com/report/index/506d40e6-7bae-4631-a401-d77082120219 https://crash-stats.mozilla.com/report/index/8b4d3c47-cb0b-4892-b1e7-d122e2120215 Should this be reopened? Thanks!
Comment 17•12 years ago
|
||
those don't come from UpdateQuotaInformationForFile and look a bit different. The crash is one line earlier too, so I'd file a separate bug.
Comment 18•12 years ago
|
||
(In reply to Marco Bonardo [:mak] from comment #17) > those don't come from UpdateQuotaInformationForFile and look a bit different. > The crash is one line earlier too, so I'd file a separate bug. Bug 731171 was logged for the crashes from comment #16. Is there anything else that can be done in order to verify this bug? Thanks!
Status: RESOLVED → VERIFIED
Updated•12 years ago
|
Status: VERIFIED → RESOLVED
Closed: 12 years ago → 12 years ago
Comment 19•12 years ago
|
||
Comment on attachment 594436 [details] [diff] [review] Patch [Triage comment] This patch is being tracked for the upcoming esr 10.0.3 release and needs to be landed today preferably, or tomorrow if need be. Let me know if there are any concerns about getting it landed in time. See https://wiki.mozilla.org/Release_Management/ESR_Landing_Process for details.
Attachment #594436 -
Flags: approval-mozilla-esr10+
Assignee | ||
Comment 20•12 years ago
|
||
https://hg.mozilla.org/releases/mozilla-esr10/rev/a80721e1b2d2
Comment 21•12 years ago
|
||
No reported crashes recently for any Firefox version. Marking VERIFIED.
Status: RESOLVED → VERIFIED
Whiteboard: [qa+] → [qa!]
Comment 22•12 years ago
|
||
Not seeing any ESR related crashes post-fix either.
You need to log in
before you can comment on or make changes to this bug.
Description
•