Browse Source

Remove two extrneous semicolons in dirserv.c

In 6fbdf635 we added a couple of statements like:
    if (test) {
       ...
    };

The extraneous semicolons there get flagged as worrisome empty
statements by the cparser library, so let's fix them.

Patch by Christian Grothoff; fixes bug 7115.
Nick Mathewson 11 years ago
parent
commit
e4821fa14d
2 changed files with 6 additions and 2 deletions
  1. 4 0
      changes/bug7115
  2. 2 2
      src/or/dirserv.c

+ 4 - 0
changes/bug7115

@@ -0,0 +1,4 @@
+  o Minor bugfixes (portability)
+    - Remove a couple of extraneous semicolos that were upsetting the
+      cparser library. Patch by Christian Grothoff. Fixes bug 7115;
+      bugfix on 0.2.2.1-alpha.

+ 2 - 2
src/or/dirserv.c

@@ -2181,7 +2181,7 @@ routerstatus_format_entry(char *buf, size_t buf_len,
             "(wanted descriptor %s).",
             id, dd);
         return -1;
-      };
+      }
 
       /* This assert can fire for the control port, because
        * it can request NS documents before all descriptors
@@ -2205,7 +2205,7 @@ routerstatus_format_entry(char *buf, size_t buf_len,
         tor_assert(tor_memeq(desc->cache_info.signed_descriptor_digest,
               rs->descriptor_digest,
               DIGEST_LEN));
-      };
+      }
     }
 
     if (format == NS_CONTROL_PORT && rs->has_bandwidth) {