Closed Bug 1131092 Opened 5 years ago Closed 5 years ago

Fix mFileStatus ordering in CacheEntry::Load

Categories

(Core :: Networking: Cache, defect, major)

defect
Not set
major

Tracking

()

RESOLVED FIXED
mozilla38
Tracking Status
firefox38 --- fixed

People

(Reporter: mayhemer, Assigned: mayhemer)

References

Details

Attachments

(1 file, 1 obsolete file)

Attached patch v1 (obsolete) — Splinter Review
Based on log from bug 1120631.

Apparently there is a code path that allows access to CacheEntry.mFile (as mFileStatus == OK) sooner than CacheFile::Init() is called on it.

This is there since ever.
Attachment #8561418 - Flags: review?(michal.novotny)
Attachment #8561418 - Flags: review?(michal.novotny) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/4a61c3921843
Status: ASSIGNED → RESOLVED
Closed: 5 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla38
You need to log in before you can comment on or make changes to this bug.