From: Sebastian Hahn <sebastian@torproject.org>
Date: Wed, 8 Jun 2011 19:16:11 +0000 (+0200)
Subject: Remove a few dead assignments during router parsing
X-Git-Tag: tor-0.2.2.29-beta~3^2~13
X-Git-Url: http://git.ipfire.org/gitweb.cgi?a=commitdiff_plain;h=e6fff7235e46b794cd77af516a1ecfab9c87de7f;p=thirdparty%2Ftor.git

Remove a few dead assignments during router parsing
---

diff --git a/changes/coverity_maint b/changes/coverity_maint
index 1bd1c417fb..fd8c440505 100644
--- a/changes/coverity_maint
+++ b/changes/coverity_maint
@@ -1,5 +1,6 @@
   o Code simplifications and refactoring:
     - Remove some dead code as indicated by coverity.
+    - Remove a few dead assignments during router parsing. Found by coverity.
   o Minor bugfixes:
     - Add some forgotten return value checks during unit tests. Found
       by coverity.
diff --git a/src/or/routerparse.c b/src/or/routerparse.c
index f855f9d027..42dbcacb51 100644
--- a/src/or/routerparse.c
+++ b/src/or/routerparse.c
@@ -1544,10 +1544,10 @@ router_parse_entry_from_string(const char *s, const char *end,
     }
   }
 
-  if ((tok = find_opt_by_keyword(tokens, K_CACHES_EXTRA_INFO)))
+  if (find_opt_by_keyword(tokens, K_CACHES_EXTRA_INFO))
     router->caches_extra_info = 1;
 
-  if ((tok = find_opt_by_keyword(tokens, K_ALLOW_SINGLE_HOP_EXITS)))
+  if (find_opt_by_keyword(tokens, K_ALLOW_SINGLE_HOP_EXITS))
     router->allow_single_hop_exits = 1;
 
   if ((tok = find_opt_by_keyword(tokens, K_EXTRA_INFO_DIGEST))) {
@@ -1560,7 +1560,7 @@ router_parse_entry_from_string(const char *s, const char *end,
     }
   }
 
-  if ((tok = find_opt_by_keyword(tokens, K_HIDDEN_SERVICE_DIR))) {
+  if (find_opt_by_keyword(tokens, K_HIDDEN_SERVICE_DIR)) {
     router->wants_to_be_hs_dir = 1;
   }