[Varnish] #761: critbit segfault

[Varnish]

#761: critbit segfault
----------------------+-----------------------------------------------------
 Reporter:  sky       |        Owner:  phk   
     Type:  defect    |       Status:  closed
 Priority:  normal    |    Milestone:        
Component:  varnishd  |      Version:  2.1.3 
 Severity:  normal    |   Resolution:  fixed 
 Keywords:            |  
----------------------+-----------------------------------------------------

Comment(by tfheen):

 (In [5368]) Merge r5304: Remove bogus assert

 After considerable thinking, I have come to the conclusion that this
 assert is bogus.

 There is a window from our lookup until we lock the refcount where
 a different thread conceiveably could loose the last reference to
 the objhdr, before we get the lock to increase it.

 The situation the assert was meant to protect, where we insert
 a new objhdr, is handled in the "oh == noh" clause earlier, were
 the weaker "> 0" check is used (rather than "== 1") to cater for
 the exact same race.

 Fixes: #761
 Fixes: #783

-- 
Ticket URL: </ticket/761#comment:4>
Varnish <http://varnish-cache.org/>
The Varnish HTTP Accelerator