DNS: cache eviction count not always decremented

VERIFIED FIXED in mozilla0.9.7

Status

()

Core
Networking
P3
normal
VERIFIED FIXED
17 years ago
15 years ago

People

(Reporter: gordon, Assigned: gordon)

Tracking

Trunk
mozilla0.9.7
Points:
---

Firefox Tracking Flags

(Not tracked)

Details

(Whiteboard: r=gagan, sr=darin)

Attachments

(1 attachment)

(Assignee)

Description

17 years ago
There is a case in nsDNSService::FindOrCreateLookup() where a lookup is removed 
from the eviction queue, but the count for the queue is not decremented.  This 
results in the actually max queue length shortening to zero over time.
(Assignee)

Comment 1

17 years ago
Created attachment 37566 [details] [diff] [review]
Patch to fix eviction count.
(Assignee)

Updated

17 years ago
Priority: -- → P3
Target Milestone: --- → mozilla0.9.2

Updated

17 years ago
Whiteboard: r=gagan

Updated

17 years ago
Whiteboard: r=gagan → r=gagan, sr=darin
a=dbaron for trunk checkin (on behalf of drivers).
(http://bugzilla.mozilla.org/show_activity.cgi?id=84531 shows it has r= and sr=)

Updated

17 years ago
Blocks: 83989
(Assignee)

Comment 3

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

Updated

17 years ago
Summary: dns cache eviction count not always decremented → DNS: cache eviction count not always decremented

Comment 4

17 years ago
 not consistant...the count is very high compared to the number of entries

Status: RESOLVED → REOPENED
Resolution: FIXED → ---
(Assignee)

Comment 5

17 years ago
What platform did you see this on?  What build?
milestone->0.9.7. cc'ing stummala, who never replied.
Target Milestone: mozilla0.9.2 → mozilla0.9.7

Comment 7

17 years ago
I am not the QA contact for this bug, i reopened this bug not even on CC 
list. Thanks Christopher for CCing me. I might have used the 08-01 build. I will 
reproduce this bug and give the build ID
(Assignee)

Comment 8

17 years ago
After testing this a bit and examining the queue and queue count, I don't see any 
problem.  Sivakiran, if you can explain how you determined the queue count was 
incorrect, and can reproduce the problem we can reopen this.  Re-marking FIXED.
Status: REOPENED → RESOLVED
Last Resolved: 17 years ago17 years ago
Resolution: --- → FIXED
You need to log in before you can comment on or make changes to this bug.