[tor-bugs] #5095 [Tor Client]: Setting __ReloadTorrcOnSIGHUP=0 breaks logrotate

[Tor Bug Tracker & Wiki]

#5095: Setting __ReloadTorrcOnSIGHUP=0 breaks logrotate
------------------------+---------------------------------------------------
 Reporter:  arma        |          Owner:                    
     Type:  defect      |         Status:  needs_review      
 Priority:  normal      |      Milestone:  Tor: 0.2.3.x-final
Component:  Tor Client  |        Version:                    
 Keywords:              |         Parent:                    
   Points:              |   Actualpoints:                    
------------------------+---------------------------------------------------

Comment(by nickm):

 Replying to [comment:3 arma]:
 > 1) char *msg; wants to be char *msg = NULL; yes?

 Yes.

 > 2)
 > {{{
 > -  if (old_options) {
 > +  if (old_options && old_options != global_options) {
 > }}}
 > is just an optimization, since option_is_same() will be true at each
 step, yes? (It's fine as an optimization; I just want to make sure I'm not
 missing something deeper.)

 It's an optimization.

 > Looks fine other than that (I think -- it definitely needs some testing
 too).

 Added a fixup commit for issue 1; I hope folks will test. :)

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