[tor-bugs] #7191 [Tor]: smartlist_bsearch_idx() is broken for short lists

[Tor Bug Tracker & Wiki]

#7191: smartlist_bsearch_idx() is broken for short lists
--------------------+-------------------------------------------------------
 Reporter:  andrea  |          Owner:  andrea            
     Type:  defect  |         Status:  needs_review      
 Priority:  major   |      Milestone:  Tor: 0.2.3.x-final
Component:  Tor     |        Version:  Tor: 0.2.4.3-alpha
 Keywords:          |         Parent:                    
   Points:          |   Actualpoints:                    
--------------------+-------------------------------------------------------

Comment(by nickm):

 Replying to [comment:7 andrea]:
 > > For 0.2.3 purposes, I wonder if some of the assertions couldn't turn
 into LD_BUG entries.  What do you think?
 >
 > Hmm - if we adopt a policy like that, won't it make it a pain in the ass
 to change all asserts to LD_BUG logs when dev branches become stable in
 the future?  Maybe we should have a macro that turns into an assert or an
 LD_BUG as appropriate?

 Maybe we should forget that I said "for 0.2.3 purposes" then and just look
 whether any of these asserts could/should be LD_BUG.

 "No they shouldn't'" is one possible answer.

-- 
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/7191#comment:10>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online