If you think a bug might affect users in the 57 release, please set the correct tracking and status flags for Release Management.

Crash in mozilla::storage::Service::UpdateQutoaInformationForFile @ `anonymous namespace''::xOpen

VERIFIED FIXED in Firefox 11

Status

()

Toolkit
Telemetry
--
critical
VERIFIED FIXED
6 years ago
5 years ago

People

(Reporter: Scoobidiver (away), Assigned: khuey)

Tracking

({crash, topcrash})

10 Branch
mozilla13
x86
Windows 7
crash, topcrash
Points:
---

Firefox Tracking Flags

(firefox10 affected, firefox11 verified, firefox12 verified, firefox-esr1011+ verified, status1.9.2 unaffected)

Details

(Whiteboard: [qa!], crash signature)

Attachments

(1 attachment)

(Reporter)

Description

6 years ago
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
ccing bent who better knows the quota management code
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.
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.
setting up according on existence of a patch somewhere in Bug 702292.
Assignee: nobody → khuey
Status: NEW → ASSIGNED

Updated

6 years ago
Keywords: topcrash
Whiteboard: startupcrash
(Reporter)

Comment 5

6 years ago
85% of crashes happen within one minute. Does this qualify that crash for the startupcrash keyword?
Created attachment 594436 [details] [diff] [review]
Patch
Attachment #594436 - Flags: review+
Attachment #594436 - Flags: approval-mozilla-beta?
Attachment #594436 - Flags: approval-mozilla-aurora?
I'm moving across the country this weekend, so I would appreciate someone else landing this.
Keywords: checkin-needed
Component: Storage → Telemetry
QA Contact: storage → telemetry
(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.
https://hg.mozilla.org/integration/mozilla-inbound/rev/543de57b25f3
Keywords: checkin-needed
Target Milestone: --- → mozilla13
https://hg.mozilla.org/mozilla-central/rev/543de57b25f3
Status: ASSIGNED → RESOLVED
Last Resolved: 6 years ago
Resolution: --- → FIXED
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+
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: --- → ?
Whiteboard: [qa+]
How can i verify the fixing of this bug?
Any info would be appreciated.
Kyle, are there any correlations with this crash that QA can try testing to help verify this fix?

Updated

6 years ago
tracking-firefox-esr10: ? → .1+

Updated

6 years ago
tracking-firefox-esr10: .1+ → 11+
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
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!
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.
(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
Status: VERIFIED → RESOLVED
Last Resolved: 6 years ago6 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+
https://hg.mozilla.org/releases/mozilla-esr10/rev/a80721e1b2d2
status-firefox-esr10: affected → fixed
No reported crashes recently for any Firefox version. Marking VERIFIED.
Status: RESOLVED → VERIFIED
status-firefox11: fixed → verified
status-firefox12: fixed → verified
Whiteboard: [qa+] → [qa!]
Not seeing any ESR related crashes post-fix either.
status-firefox-esr10: fixed → verified
You need to log in before you can comment on or make changes to this bug.