Fix a crash if you enable FascistFirewall but not FirewallPorts.

Reported by Frediano Ziglio.


svn:r6746
This commit is contained in:
Roger Dingledine 2006-07-08 17:38:46 +00:00
parent 0bbc4aca9b
commit 51454157a0

View File

@ -2216,7 +2216,7 @@ options_validate(or_options_t *old_options, or_options_t *options,
return -1;
if (options->FascistFirewall && !options->ReachableAddresses) {
if (smartlist_len(options->FirewallPorts)) {
if (options->FirewallPorts && smartlist_len(options->FirewallPorts)) {
/* We already have firewall ports set, so migrate them to
* ReachableAddresses, which will set ReachableORAddresses and
* ReachableDirAddresses if they aren't set explicitly. */