[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]
Re: [tor-bugs] #5891 [Tor Directory Authority]: rep_hist_desc_stats_write() segfaults on empty stats
#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 ln5):
Replying to [comment:4 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().
Then you don't mind me adding this assert in rep_hist_desc_stats_write()?
str = rep_hist_format_desc_stats(now);
tor_assert(str != NULL);
> Please see branch task-5891 in my public repo. I stole your changes
file. Please tweak as needed.
Tested. Works fine. Thanks.
--
Ticket URL: <https://trac.torproject.org/projects/tor/ticket/5891#comment:5>
Tor Bug Tracker & Wiki <https://trac.torproject.org/>
The Tor Project: anonymity online
_______________________________________________
tor-bugs mailing list
tor-bugs@xxxxxxxxxxxxxxxxxxxx
https://lists.torproject.org/cgi-bin/mailman/listinfo/tor-bugs