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)?