From 9cd8feb64913cc0c5d4c0be35dc77eaf93dc8921 Mon Sep 17 00:00:00 2001
From: Roger Dingledine <arma@torproject.org>
Date: Sat, 16 Dec 2006 06:56:58 +0000
Subject: [PATCH] fix an infinite loop and a memory leak in r9131

svn:r9138
---
 src/or/router.c | 11 +++++------
 1 file changed, 5 insertions(+), 6 deletions(-)

diff --git a/src/or/router.c b/src/or/router.c
index 940f462c5a..fbf2469fcc 100644
--- a/src/or/router.c
+++ b/src/or/router.c
@@ -864,9 +864,11 @@ router_rebuild_descriptor(int force)
     SMARTLIST_FOREACH(family, char *, name,
      {
        routerinfo_t *member;
-       if (!strcasecmp(name, options->Nickname))
-         member = ri;
-       else
+       if (!strcasecmp(name, options->Nickname)) {
+         /* Don't list ourself in our own family; that's redundant */
+         tor_free(name);
+         continue;
+       } else
          member = router_get_by_nickname(name, 1);
        if (!member) {
          if (!smartlist_string_isin(warned_nonexistent_family, name) &&
@@ -879,9 +881,6 @@ router_rebuild_descriptor(int force)
          }
          smartlist_add(ri->declared_family, name);
          name = NULL;
-       } else if (router_is_me(member)) {
-         /* Don't list ourself in our own family; that's redundant */
-         continue;
        } else {
          char *fp = tor_malloc(HEX_DIGEST_LEN+2);
          fp[0] = '$';