BIND 10 #643: Memory leaks in the cache

[BIND 10 Development]

#643: Memory leaks in the cache
-------------------------------------+-------------------------------------
                 Reporter:  vorner   |                Owner:  zhanglikun
                     Type:  defect   |               Status:  reviewing
                 Priority:  major    |            Milestone:  R-Team-
                Component:           |  Sprint-20110316
  Unclassified                       |           Resolution:
                 Keywords:           |            Sensitive:  0
Estimated Number of Hours:  0.0      |  Add Hours to Ticket:  0
                Billable?:  1        |          Total Hours:  0
                Internal?:  0        |
-------------------------------------+-------------------------------------
Changes (by jelte):

 * owner:  jelte => zhanglikun


Comment:

 Only one slight comment, the 'left the size of the list 0' seems a bit
 off; should this be something like "When done, the size of the list will
 be 0"? (it occurs at least twice in lru_list.h).

 Oh and just a question (i think the answer is yes), is it safe to call the
 drop handler from the lru_list (are we sure we cannot do this while the
 entry is somehow still in the hash table?)

 If the answer is yes, and you clarify the comment, I think this is good
 and can be merged (i don't see any valgrind problems anymore in
 cache/tests)

-- 
Ticket URL: <http://bind10.isc.org/ticket/643#comment:8>
BIND 10 Development <http://bind10.isc.org>
BIND 10 Development