run_scheduled_events() has funny-looking operator
In git show df9b7646 we have
@@ -1161,11 +1161,11 @@ run_scheduled_events(time_t now)
if (router_rebuild_descriptor(1)<0) {
log_info(LD_CONFIG, "Couldn't rebuild router descriptor");
}
- if (advertised_server_mode())
+ if (advertised_server_mode() & !options->DisableNetwork)
router_upload_dir_desc_to_dirservers(0);
}
which is probably not the operator we want. Reported by 'ultramage' on irc.
It looks like there's no actual impact from this bug, since both of those parameters are either-0-or-1. Still, probably good to fix.
Do we fix in 0.2.3 (because that's where we introduced the bug) or in 0.2.4 (because it's not really a bug)?