Closed Bug 1239687 Opened 4 years ago Closed 4 years ago

After shutdown leak invalid/doomed file handles immediately

Categories

(Core :: Networking: Cache, defect)

defect
Not set

Tracking

()

RESOLVED FIXED
mozilla46
Tracking Status
firefox46 --- fixed

People

(Reporter: mayhemer, Assigned: mayhemer)

Details

Attachments

(1 file)

No description provided.
Hmm.. I'm not sure about the metadata write part.  It probably may happen we first write metadata but not the tailing data.  That will lead to a semi-broken entry anyway.  It seems to be an overkill to do any prioritization here..?
Summary: After shutdown: leak invalid/doomed file handles, write metadata with priority → After shutdown leak invalid/doomed file handles immediately
Attached patch v1Splinter Review
Not sure how big impact this is going to have..

https://treeherder.mozilla.org/#/jobs?repo=try&revision=2b3a6f97700e
Attachment #8707906 - Flags: review?(michal.novotny)
Attachment #8707906 - Flags: review?(michal.novotny) → review+
Keywords: checkin-needed
https://hg.mozilla.org/mozilla-central/rev/edc2c8151935
Status: ASSIGNED → RESOLVED
Closed: 4 years ago
Resolution: --- → FIXED
Target Milestone: --- → mozilla46
You need to log in before you can comment on or make changes to this bug.