[Varnish] #783: Critbit crash in Varnish 2.1.3

Varnish varnish-bugs at varnish-cache.org
Tue Sep 28 11:34:53 CEST 2010


#783: Critbit crash in Varnish 2.1.3
----------------------+-----------------------------------------------------
 Reporter:  anders    |        Owner:  phk   
     Type:  defect    |       Status:  closed
 Priority:  normal    |    Milestone:        
Component:  varnishd  |      Version:  2.1.3 
 Severity:  major     |   Resolution:  fixed 
 Keywords:            |  
----------------------+-----------------------------------------------------
Changes (by phk):

  * resolution:  duplicate => fixed


Comment:

 (In [5304]) 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/783#comment:2>
Varnish <http://varnish-cache.org/>
The Varnish HTTP Accelerator




More information about the varnish-bugs mailing list