[tor-bugs] #2954 [Tor Directory Authority]: ides corrupted its cached-microdescs.new file

[Tor Bug Tracker & Wiki]

#2954: ides corrupted its cached-microdescs.new file
----------------------------------------+-----------------------------------
    Reporter:  mikeperry                |       Owner:  nickm             
        Type:  defect                   |      Status:  assigned          
    Priority:  normal                   |   Milestone:  Tor: 0.2.3.x-final
   Component:  Tor Directory Authority  |     Version:                    
  Resolution:                           |    Keywords:                    
      Parent:                           |      Points:                    
Actualpoints:                           |  
----------------------------------------+-----------------------------------

Comment(by nickm):

 I think for this one, the problem might have been (as I think cpunks was
 saying above?) the lack of error handling code in
 microdescs_add_list_to_cache() interacting badly with some other bug.
 That would explain why this hasn't recurred, in spite of there not having
 been anything obvious to have fixed the corruption.  The fix in that case
 would be to address the XXXs in microdescs_add_list_to_cache().  This
 leads to the question: what's the right thing to do if we can't write
 microdesciptors to the cache?  I lean towards, "Abort, don't make the
 change, and log a warning."

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