Cleanup in MID and Topology output
authorHenning Rogge <hrogge@googlemail.com>
Mon, 13 Apr 2009 20:13:10 +0000 (22:13 +0200)
committerHenning Rogge <hrogge@googlemail.com>
Mon, 13 Apr 2009 20:13:10 +0000 (22:13 +0200)
src/mid_set.c
src/neighbor_table.c
src/tc_set.c
src/two_hop_neighbor_table.c

index 2a9bb9f..04f5b3f 100644 (file)
@@ -297,7 +297,9 @@ olsr_update_mid_entry(const union olsr_ip_addr *main_addr,
                       olsr_reltime vtime, uint16_t mid_seqno)
 {
   struct mid_entry *alias;
-
+#if !defined REMOVE_LOG_DEBUG
+  struct ipaddr_str buf1, buf2;
+#endif
   if (!olsr_validate_address(alias_addr)) {
     return;
   }
@@ -307,7 +309,6 @@ olsr_update_mid_entry(const union olsr_ip_addr *main_addr,
    */
   alias = olsr_lookup_mid_entry(alias_addr);
   if (alias) {
-
     /* Update sequence number for alias purging */
     alias->mid_seqno = mid_seqno;
 
@@ -317,6 +318,7 @@ olsr_update_mid_entry(const union olsr_ip_addr *main_addr,
     olsr_set_mid_timer(alias->mid_tc, vtime);
     return;
   }
+
   /*
    * This is a fresh alias.
    */
@@ -444,7 +446,6 @@ olsr_prune_mid_entries(const union olsr_ip_addr *main_addr, uint16_t mid_seqno)
 {
   struct tc_entry *tc = olsr_locate_tc_entry(main_addr);
   struct mid_entry *alias;
-
   OLSR_FOR_ALL_TC_MID_ENTRIES(tc, alias) {
     if (alias->mid_seqno != mid_seqno) {
       olsr_delete_mid_entry(alias);
@@ -462,7 +463,7 @@ olsr_print_mid_set(void)
 #if !defined REMOVE_LOG_INFO
   struct tc_entry *tc;
   struct mid_entry *alias;
-  struct ipaddr_str buf;
+  struct ipaddr_str buf1, buf2;
 
   OLSR_INFO(LOG_MID,
              "\n--- %s ------------------------------------------------- MID\n\n",
@@ -472,8 +473,9 @@ olsr_print_mid_set(void)
     bool first = true;
     OLSR_FOR_ALL_TC_MID_ENTRIES(tc, alias) {
       OLSR_INFO_NH(LOG_MID, "%-15s: %s\n",
-          first ? olsr_ip_to_string(&buf, &tc->addr) : "",
-          olsr_ip_to_string(&buf, &alias->mid_alias_addr));
+          first ? olsr_ip_to_string(&buf1, &tc->addr) : "",
+          olsr_ip_to_string(&buf2, &alias->mid_alias_addr));
+      first = false;
     } OLSR_FOR_ALL_TC_MID_ENTRIES_END(tc, alias);
   } OLSR_FOR_ALL_TC_ENTRIES_END(tc);
 #endif
@@ -533,14 +535,14 @@ olsr_input_mid(union olsr_message *msg,
     return false;
   }
 
-  OLSR_DEBUG(LOG_MID, "Processing MID from %s, seq 0x%04x\n",
-             olsr_ip_to_string(&buf, &originator), msg_seq);
 
   /*
    * How many aliases ?
    */
   alias_count = (msg_size - 12) / olsr_cnf->ipsize;
 
+  OLSR_DEBUG(LOG_MID, "Processing MID from %s with %d aliases, seq 0x%04x\n",
+        olsr_ip_to_string(&buf, &originator), alias_count, msg_seq);
   /*
    * Now walk the list of alias advertisements one by one.
    */
index ffc1c8a..66983a9 100644 (file)
@@ -424,7 +424,7 @@ olsr_print_neighbor_table(void)
       struct link_entry *lnk = get_best_link_to_neighbor(&neigh->neighbor_main_addr);
       if(lnk) {
         struct ipaddr_str buf;
-        OLSR_INFO(LOG_NEIGHTABLE, "%-*s  %s  %s  %s  %d\n",
+        OLSR_INFO_NH(LOG_NEIGHTABLE, "%-*s  %s  %s  %s  %d\n",
                     ipwidth, olsr_ip_to_string(&buf, &neigh->neighbor_main_addr),
                     neigh->status == SYM ? "YES " : "NO  ",
                     neigh->is_mpr ? "YES " : "NO  ",
index 4aa6d4b..20f570b 100644 (file)
@@ -665,7 +665,7 @@ olsr_print_tc_table(void)
   OLSR_INFO(LOG_TC,
              "\n--- %s ------------------------------------------------- TOPOLOGY\n\n",
              olsr_wallclock_string());
-       OLSR_INFO_NH(LOG_TC, "%-*s %-*s %-14s  %s\n", ipwidth,
+       OLSR_INFO_NH(LOG_TC, "%-*s %-*s           %-14s  %s\n", ipwidth,
              "Source IP addr", ipwidth, "Dest IP addr", "      LQ      ",
              "ETX");
 
@@ -675,11 +675,11 @@ olsr_print_tc_table(void)
       struct ipaddr_str addrbuf, dstaddrbuf;
       struct lqtextbuffer lqbuffer1, lqbuffer2;
 
-      OLSR_INFO_NH(LOG_TC, "%-*s %-*s %d %-14s %s\n",
+      OLSR_INFO_NH(LOG_TC, "%-*s %-*s %5s      %-14s %s\n",
                    ipwidth, olsr_ip_to_string(&addrbuf, &tc->addr),
                    ipwidth, olsr_ip_to_string(&dstaddrbuf,
                                             &tc_edge->T_dest_addr),
-                         tc_edge->flags,
+                         (tc_edge->flags & TC_EDGE_FLAG_LOCAL) ? "local" : "",
                    get_tc_edge_entry_text(tc_edge, '/', &lqbuffer1),
                    get_linkcost_text(tc_edge->cost, false, &lqbuffer2));
 
index c61b444..745e1a6 100644 (file)
@@ -240,7 +240,7 @@ olsr_print_two_hop_neighbor_table(void)
         struct lqtextbuffer lqbuffer;
 
         OLSR_INFO_NH(LOG_2NEIGH, "%-15s  %-15s  %s\n",
-                      first ? "": olsr_ip_to_string(&buf, &neigh2->neighbor_2_addr),
+                      first ? olsr_ip_to_string(&buf, &neigh2->neighbor_2_addr) : "",
                       olsr_ip_to_string(&buf, &entry->neighbor->neighbor_main_addr),
                       get_linkcost_text(entry->path_linkcost, false, &lqbuffer));