Having a reject-all policy summarized with a lack of summary is stupid. Explicitly mention them as reject 1-65535.

svn:r16543

1 files changed, 6 insertions, 8 deletions

diff --git a/src/or/dirserv.c b/src/or/dirserv.c
index 75e9d0324e..5454b6d45c 100644
--- a/src/or/dirserv.c
+++ b/src/or/dirserv.c
@@ -1974,15 +1974,13 @@ routerstatus_format_entry(char *buf, size_t buf_len,
     cp += strlen(cp);
 
     summary = policy_summarize(desc->exit_policy);
-    if (summary) {
-      r = tor_snprintf(cp, buf_len - (cp-buf), "p %s\n", summary);
-      if (r<0) {
-        log_warn(LD_BUG, "Not enough space in buffer.");
-        return -1;
-      }
-      cp += strlen(cp);
-      tor_free(summary);
+    r = tor_snprintf(cp, buf_len - (cp-buf), "p %s\n", summary);
+    if (r<0) {
+      log_warn(LD_BUG, "Not enough space in buffer.");
+      return -1;
     }
+    cp += strlen(cp);
+    tor_free(summary);
   }
 
   return 0;