Closed Bug 656675 Opened 14 years ago Closed 7 years ago

SQLite storage statements should assert that they're only ever used on one thread

Categories

(Core :: SQLite and Embedded Database Bindings, defect)

x86
macOS
defect
Not set
normal

Tracking

()

RESOLVED INACTIVE

People

(Reporter: justin.lebar+bug, Unassigned)

Details

blocking-b2g: --- → koi?
Mass-modify - removal of no longer relevant blocking flags.
blocking-b2g: koi? → ---
It's not super-clear from the original bug what the dream was here. It seems like a reasonable behaviour would be if this was the equivalent of the mozStorageStatementScoper noticing that a statement was concurrently being used on multiple threads. Since buggy code might not use mozStorageStatementScoper and the statement object would need to provide help anyways, the easiest debug algorithm would be to: - Have all statement methods check if mUsedOnThread is null. If null, assign the current thread. If non-null, assert it's the same thread as our thread. - Have Reset() clear mUsedOnThread. Any complaint from this would be valid/useful feedback I think, even if it's just failure to Reset() the statement in a timely fashion.
Per policy at https://wiki.mozilla.org/Bug_Triage/Projects/Bug_Handling/Bug_Husbandry#Inactive_Bugs. If this bug is not an enhancement request or a bug not present in a supported release of Firefox, then it may be reopened.
Status: NEW → RESOLVED
Closed: 7 years ago
Resolution: --- → INACTIVE
Product: Toolkit → Core
You need to log in before you can comment on or make changes to this bug.