Closed Bug 1589626 Opened Last month Closed Last month

LSNG: Add a check for null quota object

Categories

(Core :: Storage: localStorage & sessionStorage, defect, P1)

defect

Tracking

()

RESOLVED FIXED
mozilla71
Tracking Status
firefox-esr60 --- unaffected
firefox-esr68 --- disabled
firefox67 --- disabled
firefox68 --- disabled
firefox69 --- disabled
firefox70 + disabled
firefox71 --- fixed

People

(Reporter: janv, Assigned: janv)

References

(Blocks 1 open bug)

Details

(Keywords: crash)

Crash Data

Attachments

(1 file)

mozilla::dom::quota::QuotaObject::LockedMaybeUpdateSize started showing up in the crash stats. I believe it's related to a mismatch between base domain computed by MozURL and nsIPrincipal. I think we should rather fail datastore preparation instead of crashing if this happens, at least for now.

Type: defect → task
Summary: LSNG: Check for null quota object → LSNG: Add a check for null quota object
Pushed by jvarga@mozilla.com:
https://hg.mozilla.org/integration/autoland/rev/c36aa74864ff
LSNG: Add a check for null quota object; r=ttung
Status: ASSIGNED → RESOLVED
Closed: Last month
Resolution: --- → FIXED
Target Milestone: --- → mozilla71
Keywords: crash

In early crash results from 70 release, this is in the top 10 crash signatures.

Tracking to see what the volume looks like after a bit more time on release.

I guess this bug is moot for Fx70 now given bug 1592136.

You need to log in before you can comment on or make changes to this bug.