DeactivateEntry: bad cache entry state when quitting

VERIFIED FIXED in mozilla0.9.9

Status

()

Core
Networking: Cache
P1
normal
VERIFIED FIXED
17 years ago
17 years ago

People

(Reporter: jesup, Assigned: gordon)

Tracking

Trunk
mozilla0.9.9
x86
All
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: patch)

Attachments

(2 attachments)

(Reporter)

Description

17 years ago
Mozilla 20011030xx

Got "DeactivateEntry: bad cache entry state" warning while quitting.  Looking at
the code, I'm not suprised: DeactivateAndClearEntry() marks it inactive, then
DeactivateEntry() says if (doomed) {...} else if (active) {...} else {Warn}. 
However, it can't be doomed because nsCacheService::Shutdown() cleared the
doomed list before calling ClearActiveEntries().

GDB dumps/traces to be attached.
(Reporter)

Comment 1

17 years ago
Created attachment 55922 [details]
GDB trace/dump
(Assignee)

Comment 2

17 years ago
Targeting for necko stability milestone 0.9.8.
Target Milestone: --- → mozilla0.9.8
(Assignee)

Updated

17 years ago
Target Milestone: mozilla0.9.8 → mozilla0.9.9
(Assignee)

Updated

17 years ago
Priority: -- → P1

Updated

17 years ago
Keywords: nsbeta1+
(Assignee)

Comment 3

17 years ago
Created attachment 69575 [details] [diff] [review]
Change NS_WARNING to NS_ASSERTION that doesn't fire if we're shutting down.

Looking for r/sr.
(Assignee)

Updated

17 years ago
Whiteboard: patch
(Assignee)

Comment 4

17 years ago
Comment on attachment 69575 [details] [diff] [review]
Change NS_WARNING to NS_ASSERTION that doesn't fire if we're shutting down.

sr=darin
Attachment #69575 - Flags: superreview+

Comment 5

17 years ago
Comment on attachment 69575 [details] [diff] [review]
Change NS_WARNING to NS_ASSERTION that doesn't fire if we're shutting down.

r=gagan
Attachment #69575 - Flags: review+
(Assignee)

Comment 6

17 years ago
Patch checked in. Marking FIXED.
Status: NEW → RESOLVED
Last Resolved: 17 years ago
Resolution: --- → FIXED

Comment 7

17 years ago
verified
Status: RESOLVED → VERIFIED
You need to log in before you can comment on or make changes to this bug.