Crash in sqlite3_extended_result_codes

NEW
Unassigned

Status

()

defect
P3
critical
Last year
Last month

People

(Reporter: philipp, Unassigned)

Tracking

({crash, regression})

62 Branch
Points:
---
Dependency tree / graph

Firefox Tracking Flags

(firefox-esr52 unaffected, firefox-esr60 unaffected, firefox61 unaffected, firefox62 wontfix, firefox63 wontfix, firefox64 fix-optional)

Details

(Whiteboard: [fxsearch], crash signature)

This bug was filed from the Socorro interface and is
report bp-272404fd-cf75-47e1-a220-ce8250180627.
=============================================================

Top 10 frames of crashing thread:

0 nss3.dll sqlite3_extended_result_codes db/sqlite3/src/sqlite3.c:150419
1 xul.dll mozilla::storage::Connection::prepareStatement storage/mozStorageConnection.cpp:1231
2 xul.dll mozilla::storage::Statement::initialize storage/mozStorageStatement.cpp:132
3 xul.dll mozilla::storage::Connection::CreateStatement storage/mozStorageConnection.cpp:1832
4 xul.dll static nsresult mozilla::places::`anonymous namespace'::MigrateV48FrecenciesRunnable::Run toolkit/components/places/Database.cpp:2440
5 xul.dll nsThread::ProcessNextEvent xpcom/threads/nsThread.cpp:1051
6 xul.dll NS_ProcessNextEvent xpcom/threads/nsThreadUtils.cpp:519
7 xul.dll mozilla::ipc::MessagePumpForNonMainThreads::Run ipc/glue/MessagePump.cpp:334
8 xul.dll MessageLoop::RunHandler ipc/chromium/src/base/message_loop.cc:318
9 xul.dll MessageLoop::Run ipc/chromium/src/base/message_loop.cc:298

=============================================================

crash reports with this signature are showing up in greater numbers than in the past at the start of the 62.0b cycle - probably due to the places migration in bug 1463017.
(In reply to [:philipp] from comment #0)
> crash reports with this signature are showing up in greater numbers than in
> the past at the start of the 62.0b cycle - probably due to the places
> migration in bug 1463017.

That migration problem (large wal) is fixed, so I don't think there's a relation with that specific bug.
Though, it starts from MigrateV48FrecenciesRunnable.
It's crashing on a call to sqlite3_mutex_enter(db->mutex);

It's a crash on shutdown, just check the Shutdown Progress aggregation, we're likely not handling properly the database closing under us.
Priority: -- → P2
Whiteboard: [fxsearch]
Pretty low volume, let's fix-optional for 62 (tho we'd likely want a patch for uplift depending on complexity).
Only 3 crashes on the 63 branch (nightly and beta) since July, marking as fix-optional for 63.
Priority: P2 → P3
You need to log in before you can comment on or make changes to this bug.