Skip to content
  • Nick Mathewson's avatar
    When enabling periodic events, schedule but don't run them immediately. · 176999fd
    Nick Mathewson authored
    When we fixed 25939 in f7633c1f, we
    introduced a call to rescan_periodic_events() from inside the onion
    service logic. But this meant that we could rescan the event list --
    thereby running event callbacks! -- from inside the hidden service code.
    This could cause us to run some of our event callbacks from an
    inconsistent state, if we were in the middle of changing options.
    
    A related bug (#25761) prevented us from rescanning our periodic
    events as appropriate, but when we fixed THAT one, this bug reared
    its ugly head.
    
    The fix here is that "enabling" an event should cause us to run it
    from the event loop, but not immediately from the point where we
    enable it.
    
    Fixes bug 27003; bugfix on 0.3.4.1-alpha.
    176999fd