[tor-bugs] #9200 [Tor]: named_flag[v_sl_idx] check when computing chosen_name in dirvote.c is wrong

[Tor Bug Tracker & Wiki]

#9200: named_flag[v_sl_idx] check when computing chosen_name in dirvote.c is wrong
----------------------+-----------------------------------------------------
 Reporter:  nickm     |          Owner:                    
     Type:  defect    |         Status:  needs_review      
 Priority:  major     |      Milestone:  Tor: 0.2.4.x-final
Component:  Tor       |        Version:                    
 Keywords:  tor-auth  |         Parent:                    
   Points:            |   Actualpoints:                    
----------------------+-----------------------------------------------------

Comment(by andrea):

 This patch looks okay to me, but ISO C99 6.5.7.3 also states that the
 bitwise shift operators have undefined behavior if the shift is greater
 than the width of the left operand.  Shouldn't we perhaps be checking a
 maximum as well if we're being careful about this?

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