[Author Prev][Author Next][Thread Prev][Thread Next][Author Index][Thread Index]

[or-cvs] [tor/maint-0.2.1] Clean up a bit of C logic, and fix an erroneous warning.



Author: Nick Mathewson <nickm@xxxxxxxxxxxxxx>
Date: Fri, 29 May 2009 23:28:01 -0400
Subject: Clean up a bit of C logic, and fix an erroneous warning.
Commit: d66c3797650698778bd098bbaf9d3bbeebfa9fcf

(Simplify "if (cond) return 1; return 0;" to "return cond;", and don't
give a warning when we start with accounting on but dirport off.)
---
 src/or/config.c |   11 +++++------
 1 files changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/or/config.c b/src/or/config.c
index 5bf3fd4..8199951 100644
--- a/src/or/config.c
+++ b/src/or/config.c
@@ -2634,12 +2634,11 @@ is_listening_on_low_port(uint16_t port_option,
 #else
   const config_line_t *l;
   uint16_t p;
-  if (listen_options == NULL) {
-    if (port_option < 1024) {
-      return 1;
-    }
-    return 0;
-  }
+  if (port_option == 0)
+    return 0; /* We're not listening */
+  if (listen_options == NULL)
+    return (port_option < 1024);
+
   for (l = listen_options; l; l = l->next) {
     parse_addr_port(LOG_WARN, l->value, NULL, NULL, &p);
     if (p<1024) {
-- 
1.5.6.5