Closed Bug 1689680 Opened 4 years ago Closed 4 years ago

Corruption errors when accessing the local storage archive during initialization should be ignored and the archive be wiped

Categories

(Core :: Storage: Quota Manager, defect, P1)

defect

Tracking

()

RESOLVED FIXED
87 Branch
Tracking Status
firefox87 --- fixed

People

(Reporter: sg, Assigned: sg)

References

(Blocks 1 open bug)

Details

Attachments

(2 files)

Errors such as NS_ERROR_FILE_CORRUPTED happen during InitializeLocalStorageArchive and LoadLocalStorageArchiveVersion, e.g.

Severity: -- → S2
Priority: -- → P1
Depends on: 1685677
Assignee: nobody → sgiesecke

Relevant stack traces from 86b:

      21    52789 dom/quota/ActorsParent.cpp:546                     NS_ERROR_FILE_CORRUPTED # Bug 1689680
                  dom/quota/ActorsParent.cpp:5985                    NS_ERROR_FILE_CORRUPTED
                  dom/quota/ActorsParent.cpp:6208                    NS_ERROR_FILE_CORRUPTED
      11       12 dom/quota/ActorsParent.cpp:576                                      # Bug 1689680
       9        9 dom/quota/ActorsParent.cpp:546                     NS_ERROR_FILE_CORRUPTED # Bug 1689680
       7        9 dom/quota/ActorsParent.cpp:546                     NS_ERROR_FILE_CORRUPTED # Bug 1689680
                  dom/quota/ActorsParent.cpp:6200                    NS_ERROR_FILE_CORRUPTED
       6        9 dom/quota/ActorsParent.cpp:546                     NS_ERROR_FILE_CORRUPTED # Bug 1689680
                  dom/quota/ActorsParent.cpp:5985                    NS_ERROR_FILE_CORRUPTED
Status: NEW → ASSIGNED
Keywords: leave-open
Pushed by sgiesecke@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/4f08e8c00b86 Add StorageDBUpdater::CreateCurrentSchema and call that if we know we have a fresh database file. r=dom-storage-reviewers,janv
Summary: Errors when accessing the local storage archive during initialization should be ignored and the archive be wiped → Corruption errors when accessing the local storage archive during initialization should be ignored and the archive be wiped
Pushed by sgiesecke@mozilla.com: https://hg.mozilla.org/integration/autoland/rev/d5f2a5538e66 Do fallback from existing archive file to webapps store file to empty file more consistently. r=dom-storage-reviewers,ttung
Keywords: leave-open
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → 87 Branch
You need to log in before you can comment on or make changes to this bug.

Attachment

General

Created:
Updated:
Size: