[tor-bugs] #5891 [Tor Directory Authority]: rep_hist_desc_stats_write() segfaults on empty stats

[Tor Bug Tracker & Wiki]

#5891: rep_hist_desc_stats_write() segfaults on empty stats
-------------------------------------+--------------------------------------
 Reporter:  ln5                      |          Owner:                    
     Type:  defect                   |         Status:  needs_review      
 Priority:  normal                   |      Milestone:  Tor: 0.2.3.x-final
Component:  Tor Directory Authority  |        Version:                    
 Keywords:                           |         Parent:                    
   Points:                           |   Actualpoints:                    
-------------------------------------+--------------------------------------

Comment(by karsten):

 Replying to [comment:3 ln5]:
 > That's a good fix but I think we need this change too since
 > rep_hist_format_desc_stats() still can return NULL.

 Only if we're not collecting stats, and we're checking that case in
 rep_hist_desc_stats_write() before calling rep_hist_format_desc_stats().

 > I'm happy to test a patch.

 Please see branch task-5891 in my public repo.  I stole your changes file.
 Please tweak as needed.

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