diff options
| author | Nick Mathewson <nickm@torproject.org> | 2011-07-07 11:52:13 -0400 |
|---|---|---|
| committer | Nick Mathewson <nickm@torproject.org> | 2011-07-07 11:52:13 -0400 |
| commit | 64c8e8edda5a7f3525a1ab365c4bd72b19212322 (patch) | |
| tree | f332a7c68897a39b81976bc9b0392e5e81848013 /src/or/rendclient.c | |
| parent | 174cbff8cf855dc1bbf690694f539ba3f85a508c (diff) | |
| download | tor-64c8e8edda5a7f3525a1ab365c4bd72b19212322.tar.gz tor-64c8e8edda5a7f3525a1ab365c4bd72b19212322.zip | |
Kill redundant checks around routerset_contains_*()
All of the routerset_contains*() functions return 0 if their
routerset_t argument is NULL. Therefore, there's no point in
doing "if (ExcludeNodes && routerset_contains*(ExcludeNodes...))",
for example.
This patch fixes every instance of
if (X && routerstatus_contains*(X,...))
Note that there are other patterns that _aren't_ redundant. For
example, we *don't* want to change:
if (EntryNodes && !routerstatus_contains(EntryNodes,...))
Fixes #2797. No bug here; just needless code.
Diffstat (limited to 'src/or/rendclient.c')
| -rw-r--r-- | src/or/rendclient.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/src/or/rendclient.c b/src/or/rendclient.c index b618d0fe4f..516455983c 100644 --- a/src/or/rendclient.c +++ b/src/or/rendclient.c @@ -948,7 +948,7 @@ rend_client_get_random_intro_impl(const rend_cache_entry_t *entry, intro->extend_info = extend_info_from_node(node); } /* Check if we should refuse to talk to this router. */ - if (options->ExcludeNodes && strict && + if (strict && routerset_contains_extendinfo(options->ExcludeNodes, intro->extend_info)) { n_excluded++; |