[or-cvs] r9140: and then revert half of r9138 (tor/trunk/src/or)

[arma at seul.org]

Author: arma
Date: 2006-12-16 02:04:26 -0500 (Sat, 16 Dec 2006)
New Revision: 9140

Modified:
   tor/trunk/src/or/router.c
Log:
and then revert half of r9138


Modified: tor/trunk/src/or/router.c
===================================================================
--- tor/trunk/src/or/router.c	2006-12-16 07:00:59 UTC (rev 9139)
+++ tor/trunk/src/or/router.c	2006-12-16 07:04:26 UTC (rev 9140)
@@ -863,11 +863,9 @@
     SMARTLIST_FOREACH(family, char *, name,
      {
        routerinfo_t *member;
-       if (!strcasecmp(name, options->Nickname)) {
-         /* Don't list ourself in our own family; that's redundant */
-         tor_free(name);
-         continue;
-       } else
+       if (!strcasecmp(name, options->Nickname))
+         member = ri;
+       else
          member = router_get_by_nickname(name, 1);
        if (!member) {
          if (!smartlist_string_isin(warned_nonexistent_family, name) &&
@@ -880,6 +878,8 @@
          }
          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 */
        } else {
          char *fp = tor_malloc(HEX_DIGEST_LEN+2);
          fp[0] = '$';