* activate -Wshadow
authorBernd Petrovitsch <bernd@firmix.at>
Sun, 7 Sep 2008 21:26:43 +0000 (23:26 +0200)
committerBernd Petrovitsch <bernd@firmix.at>
Sun, 7 Sep 2008 21:26:43 +0000 (23:26 +0200)
22 files changed:
Makefile.inc
lib/bmf/src/NetworkInterfaces.c
lib/bmf/src/PacketHistory.c
lib/dot_draw/src/olsrd_dot_draw.c
lib/httpinfo/src/olsrd_httpinfo.c
lib/httpinfo/src/olsrd_plugin.c
lib/pgraph/src/olsrd_pgraph.c
lib/secure/src/md5.c
lib/secure/src/olsrd_secure.c
lib/txtinfo/src/olsrd_txtinfo.c
make/Makefile.fbsd
make/Makefile.linux
make/Makefile.nbsd
make/Makefile.osx
make/Makefile.wince
src/main.c
src/mpr.c
src/olsr_switch/main.c
src/olsr_switch/ohs_cmd.c
src/parser.c
src/scheduler.c
src/unix/ifnet.c

index ba45f3b..7a2683d 100644 (file)
@@ -76,8 +76,8 @@ WARNINGS +=   -Wwrite-strings
 WARNINGS +=    -Wbad-function-cast
 WARNINGS +=    -Wpointer-arith
 WARNINGS +=    -Wcast-qual
-# the following 5 do not work yet and need more work on it
-#WARNINGS +=   -Wshadow
+WARNINGS +=    -Wshadow
+# the following 4 do not work yet and need more work on it
 #WARNINGS +=   -Wpointer-arith
 #WARNINGS +=   -Wcast-align
 #WARNINGS +=   -Wconversion
index beb1c4f..3e5b020 100644 (file)
@@ -454,7 +454,6 @@ void FindNeighbors(
        * passed IP addresses (if passed). Rely on short-circuit boolean evaluation. */
       if (source != NULL && ipequal(neighborMainIp, MainAddressOf(source)))
       {
-        struct ipaddr_str buf;
         OLSR_PRINTF(
           9,
           "%s: ----> Not forwarding to %s: is source of pkt\n",
@@ -467,7 +466,6 @@ void FindNeighbors(
       /* Rely on short-circuit boolean evaluation */
       if (forwardedBy != NULL && ipequal(neighborMainIp, MainAddressOf(forwardedBy)))
       {
-        struct ipaddr_str buf;
         OLSR_PRINTF(
           9,
           "%s: ----> Not forwarding to %s: is the node that forwarded the pkt\n",
@@ -480,7 +478,6 @@ void FindNeighbors(
       /* Rely on short-circuit boolean evaluation */
       if (forwardedTo != NULL && ipequal(neighborMainIp, MainAddressOf(forwardedTo)))
       {
-        struct ipaddr_str buf;
         OLSR_PRINTF(
           9,
           "%s: ----> Not forwarding to %s: is the node to which the pkt was forwarded\n",
@@ -824,7 +821,6 @@ void FindNeighbors(
       {
         if (bestLinkToNeighbor == NULL)
         {
-          struct ipaddr_str buf;
           OLSR_PRINTF(
             9,
             "%s: ----> Not forwarding to %s: no link found\n",
@@ -835,7 +831,6 @@ void FindNeighbors(
         {
 #ifndef NODEBUG
           struct interface* bestIntf = if_ifwithaddr(&bestLinkToNeighbor->local_iface_addr);
-          struct ipaddr_str buf;
           struct lqtextbuffer lqbuffer;
 #endif
           OLSR_PRINTF(
index 7bc6781..a079147 100644 (file)
@@ -235,14 +235,14 @@ void InitPacketHistory(void)
  * ------------------------------------------------------------------------- */
 int CheckAndMarkRecentPacket(u_int32_t crc32)
 {
-  u_int32_t index;
+  u_int32_t idx;
   struct TDupEntry* walker;
   struct TDupEntry* newEntry;
 
-  index = Hash(crc32);
-  assert(index < HISTORY_HASH_SIZE);
+  idx = Hash(crc32);
+  assert(idx < HISTORY_HASH_SIZE);
 
-  for (walker = PacketHistory[index]; walker != NULL; walker = walker->next)
+  for (walker = PacketHistory[idx]; walker != NULL; walker = walker->next)
   {
     if (walker->crc32 == crc32)
     {
@@ -263,8 +263,8 @@ int CheckAndMarkRecentPacket(u_int32_t crc32)
     newEntry->timeOut = GET_TIMESTAMP(HISTORY_HOLD_TIME);
 
     /* Add new entry at the front of the list */
-    newEntry->next = PacketHistory[index];
-    PacketHistory[index] = newEntry;
+    newEntry->next = PacketHistory[idx];
+    PacketHistory[idx] = newEntry;
   }
 
   return 0;
index 71a21e5..7b97ebc 100644 (file)
@@ -172,15 +172,15 @@ ipc_print_neigh_link(const struct neighbor_entry *neighbor)
   olsr_linkcost etx = 0.0;
   const char *style;
   const char *adr = olsr_ip_to_string(&mainaddrstrbuf, &olsr_cnf->main_addr);
-  struct link_entry* link;
+  struct link_entry* lnk;
   struct lqtextbuffer lqbuffer;
   
   if (neighbor->status == 0) { /* non SYM */
     style = "dashed";
   } else {   
-    link = get_best_link_to_neighbor(&neighbor->neighbor_main_addr);
-    if (link) {
-      etx = link->linkcost;
+    lnk = get_best_link_to_neighbor(&neighbor->neighbor_main_addr);
+    if (lnk) {
+      etx = lnk->linkcost;
     }
     style = "solid";
   }
@@ -200,7 +200,7 @@ ipc_print_neigh_link(const struct neighbor_entry *neighbor)
 static int
 plugin_ipc_init(void)
 {
-  struct sockaddr_in sin;
+  struct sockaddr_in addr;
   olsr_u32_t yes = 1;
 
   if (ipc_socket != -1) {
@@ -231,13 +231,13 @@ plugin_ipc_init(void)
   /* Bind the socket */
       
   /* complete the socket structure */
-  memset(&sin, 0, sizeof(sin));
-  sin.sin_family = AF_INET;
-  sin.sin_addr.s_addr = INADDR_ANY;
-  sin.sin_port = htons(ipc_port);
+  memset(&addr, 0, sizeof(addr));
+  addr.sin_family = AF_INET;
+  addr.sin_addr.s_addr = INADDR_ANY;
+  addr.sin_port = htons(ipc_port);
       
   /* bind the socket to the port number */
-  if (bind(ipc_socket, (struct sockaddr *) &sin, sizeof(sin)) == -1) {
+  if (bind(ipc_socket, (struct sockaddr *) &addr, sizeof(addr)) == -1) {
     olsr_printf(1, "(DOT DRAW)IPC bind %s\n", strerror(errno));
     CLOSE(ipc_socket);
     return 0;
@@ -292,9 +292,9 @@ ipc_action(int fd __attribute__((unused)))
  *Scheduled event
  */
 static int
-pcf_event(int changes_neighborhood,
-         int changes_topology,
-         int changes_hna)
+pcf_event(int chgs_neighborhood,
+         int chgs_topology,
+         int chgs_hna)
 {
   struct neighbor_entry *neighbor_table_tmp;
   struct tc_entry *tc;
@@ -304,7 +304,7 @@ pcf_event(int changes_neighborhood,
   struct ip_prefix_list *hna;
   int res = 0;
 
-  if (changes_neighborhood || changes_topology || changes_hna) {
+  if (chgs_neighborhood || chgs_topology || chgs_hna) {
     
     /* Print tables to IPC socket */
     ipc_send_str("digraph topology\n{\n");
index 5aadad6..aa80824 100644 (file)
@@ -258,7 +258,7 @@ static const struct dynamic_file_entry dynamic_files[] =
 static int
 get_http_socket(int port)
 {
-  struct sockaddr_in sin;
+  struct sockaddr_in addr;
   olsr_u32_t yes = 1;
 
   /* Init ipc socket */
@@ -277,13 +277,13 @@ get_http_socket(int port)
   /* Bind the socket */
 
   /* complete the socket structure */
-  memset(&sin, 0, sizeof(sin));
-  sin.sin_family = AF_INET;
-  sin.sin_addr.s_addr = INADDR_ANY;
-  sin.sin_port = htons(port);
+  memset(&addr, 0, sizeof(addr));
+  addr.sin_family = AF_INET;
+  addr.sin_addr.s_addr = INADDR_ANY;
+  addr.sin_port = htons(port);
 
   /* bind the socket to the port number */
-  if (bind(s, (struct sockaddr *) &sin, sizeof(sin)) == -1) {
+  if (bind(s, (struct sockaddr *) &addr, sizeof(addr)) == -1) {
     olsr_printf(1, "(HTTPINFO) bind failed %s\n", strerror(errno));
     close(s);
     return -1;
@@ -664,7 +664,7 @@ static int section_title(char *buf, olsr_u32_t bufsize, const char *title)
 static int build_frame(char *buf,
                        olsr_u32_t bufsize,
                        const char *title __attribute__((unused)),
-                       const char *link __attribute__((unused)),
+                       const char *lnk __attribute__((unused)),
                        int width __attribute__((unused)),
                        build_body_callback frame_body_cb)
 {
@@ -968,7 +968,7 @@ static int build_config_body(char *buf, olsr_u32_t bufsize)
 static int build_neigh_body(char *buf, olsr_u32_t bufsize)
 {
   struct neighbor_entry *neigh;
-  struct link_entry *link = NULL;
+  struct link_entry *lnk;
   int size = 0;
   const char *colspan = resolve_ip_addresses ? " colspan=\"2\"" : "";
 
@@ -983,20 +983,20 @@ static int build_neigh_body(char *buf, olsr_u32_t bufsize)
   size += snprintf(&buf[size], bufsize-size, "</tr>\n");
 
   /* Link set */
-  OLSR_FOR_ALL_LINK_ENTRIES(link) {
+  OLSR_FOR_ALL_LINK_ENTRIES(lnk) {
     size += snprintf(&buf[size], bufsize-size, "<tr>");
-    size += build_ipaddr_with_link(&buf[size], bufsize, &link->local_iface_addr, -1);
-    size += build_ipaddr_with_link(&buf[size], bufsize, &link->neighbor_iface_addr, -1);
-    size += snprintf(&buf[size], bufsize-size, "<td align=\"right\">%0.2f</td>", link->L_link_quality);
+    size += build_ipaddr_with_link(&buf[size], bufsize, &lnk->local_iface_addr, -1);
+    size += build_ipaddr_with_link(&buf[size], bufsize, &lnk->neighbor_iface_addr, -1);
+    size += snprintf(&buf[size], bufsize-size, "<td align=\"right\">%0.2f</td>", lnk->L_link_quality);
     if (olsr_cnf->lq_level > 0) {
       struct lqtextbuffer lqbuffer1, lqbuffer2;
       size += snprintf(&buf[size], bufsize-size,
                        "<td align=\"right\">(%s) %s</td>",
-                       get_link_entry_text(link, '/', &lqbuffer1),
-                       get_linkcost_text(link->linkcost, OLSR_FALSE, &lqbuffer2));
+                       get_link_entry_text(lnk, '/', &lqbuffer1),
+                       get_linkcost_text(lnk->linkcost, OLSR_FALSE, &lqbuffer2));
     }
     size += snprintf(&buf[size], bufsize-size, "</tr>\n");
-  } OLSR_FOR_ALL_LINK_ENTRIES_END(link);
+  } OLSR_FOR_ALL_LINK_ENTRIES_END(lnk);
 
   size += snprintf(&buf[size], bufsize-size, "</table>\n");
 
@@ -1005,7 +1005,6 @@ static int build_neigh_body(char *buf, olsr_u32_t bufsize)
                    "<tr><th align=\"center\"%s>IP Address</th><th align=\"center\">SYM</th><th align=\"center\">MPR</th><th align=\"center\">MPRS</th><th align=\"center\">Willingness</th><th>2 Hop Neighbors</th></tr>\n", colspan);
   /* Neighbors */
   OLSR_FOR_ALL_NBR_ENTRIES(neigh) {
-
     struct neighbor_2_list_entry *list_2;
     int thop_cnt;
     size += snprintf(&buf[size], bufsize-size, "<tr>");
@@ -1215,10 +1214,10 @@ static int build_cfgfile_body(char *buf, olsr_u32_t bufsize)
   return size;
 }
 
-static int check_allowed_ip(const struct allowed_net * const allowed_nets, const union olsr_ip_addr * const addr)
+static int check_allowed_ip(const struct allowed_net * const all_nets, const union olsr_ip_addr * const addr)
 {
     const struct allowed_net *alln;
-    for (alln = allowed_nets; alln != NULL; alln = alln->next) {
+    for (alln = all_nets; alln != NULL; alln = alln->next) {
         if ((addr->v4.s_addr & alln->mask.v4.s_addr) == (alln->net.v4.s_addr & alln->mask.v4.s_addr)) {
             return 1;
         }
index ea4f309..f6d55de 100644 (file)
@@ -114,7 +114,7 @@ void olsrd_get_plugin_parameters(const struct olsrd_plugin_parameters **params,
     *size = sizeof(plugin_parameters)/sizeof(*plugin_parameters);
 }
 
-static int insert_plugin_ipnet(const char *sz_net, const char *sz_mask, struct allowed_net **allowed_nets)
+static int insert_plugin_ipnet(const char *sz_net, const char *sz_mask, struct allowed_net **all_nets)
 {
     struct allowed_net *an;
 
@@ -129,8 +129,8 @@ static int insert_plugin_ipnet(const char *sz_net, const char *sz_mask, struct a
         free(an);
        return 1;
     }
-    an->next = *allowed_nets;
-    *allowed_nets = an;
+    an->next = *all_nets;
+    *all_nets = an;
     return 0;
 }
 
index 64e9f05..b0a7a73 100644 (file)
@@ -308,9 +308,9 @@ static void ipc_action(int fd __attribute__((unused)))
 /**
  *Scheduled event
  */
-static int pcf_event(int changes_neighborhood,
-                     int changes_topology,
-                     int changes_hna __attribute__((unused)))
+static int pcf_event(int chgs_neighborhood,
+                     int chgs_topology,
+                     int chgs_hna __attribute__((unused)))
 {
   int res;
   struct neighbor_entry *neighbor_table_tmp;
@@ -319,7 +319,7 @@ static int pcf_event(int changes_neighborhood,
 
   res = 0;
 
-  if (changes_neighborhood || changes_topology) {
+  if (chgs_neighborhood || chgs_topology) {
     /* Print tables to IPC socket */
 
     //ipc_send("start ", strlen("start "));
index 51abd9d..98700da 100644 (file)
@@ -251,10 +251,10 @@ void MD5Init (MD5_CTX *context)
  */
 void MD5Update (MD5_CTX *context, const unsigned char *input, const unsigned int inputLen)
 {
-  unsigned int i, index, partLen;
+  unsigned int i, idx, partLen;
 
   /* Compute number of bytes mod 64 */
-  index = (unsigned int)((context->count[0] >> 3) & 0x3F);
+  idx = (unsigned int)((context->count[0] >> 3) & 0x3F);
 
   /* Update number of bits */
   if ((context->count[0] += ((UINT4)inputLen << 3))
@@ -262,26 +262,26 @@ void MD5Update (MD5_CTX *context, const unsigned char *input, const unsigned int
     context->count[1]++;
   context->count[1] += ((UINT4)inputLen >> 29);
   
-  partLen = 64 - index;
+  partLen = 64 - idx;
   
   /* Transform as many times as possible.
    */
   if (inputLen >= partLen) {
     MD5_memcpy
-      ((POINTER)&context->buffer[index], input, partLen);
+      ((POINTER)&context->buffer[idx], input, partLen);
     MD5Transform (context->state, context->buffer);
   
     for (i = partLen; i + 63 < inputLen; i += 64)
       MD5Transform (context->state, &input[i]);
     
-    index = 0;
+    idx = 0;
   }
   else
     i = 0;
   
   /* Buffer remaining input */
   MD5_memcpy 
-    ((POINTER)&context->buffer[index], &input[i],
+    ((POINTER)&context->buffer[idx], &input[i],
      inputLen-i);
 }
 
@@ -291,15 +291,15 @@ void MD5Update (MD5_CTX *context, const unsigned char *input, const unsigned int
 void MD5Final (unsigned char digest[16], MD5_CTX *context)
 {
   unsigned char bits[8];
-  unsigned int index, padLen;
+  unsigned int idx, padLen;
 
   /* Save number of bits */
   Encode (bits, context->count, 8);
 
   /* Pad out to 56 mod 64.
    */
-  index = (unsigned int)((context->count[0] >> 3) & 0x3f);
-  padLen = (index < 56) ? (56 - index) : (120 - index);
+  idx = (unsigned int)((context->count[0] >> 3) & 0x3f);
+  padLen = (idx < 56) ? (56 - idx) : (120 - idx);
   MD5Update (context, PADDING, padLen);
   
   /* Append length (before padding) */
index f5a9a8e..8134427 100644 (file)
@@ -707,7 +707,6 @@ parse_cres(struct interface *olsr_if, char *in_msg)
   /* Now to check the digest from the emitted challenge */
   if((entry = lookup_timestamp_entry((const union olsr_ip_addr *)&msg->originator)) == NULL)
     {
-      struct ipaddr_str buf;
       olsr_printf(1, "[ENC]Received challenge-response from non-registered node %s!\n",
                  olsr_ip_to_string(&buf, (union olsr_ip_addr *)&msg->originator));
       return 0;
@@ -731,7 +730,6 @@ parse_cres(struct interface *olsr_if, char *in_msg)
 
   if(memcmp(msg->res_sig, sha1_hash, SIGNATURE_SIZE) != 0)
     {
-      struct ipaddr_str buf;
       olsr_printf(1, "[ENC]Error in challenge signature from %s!\n",
                  olsr_ip_to_string(&buf, (union olsr_ip_addr *)&msg->originator));
       
@@ -810,7 +808,6 @@ parse_rres(char *in_msg)
   /* Now to check the digest from the emitted challenge */
   if((entry = lookup_timestamp_entry((const union olsr_ip_addr *)&msg->originator)) == NULL)
     {
-      struct ipaddr_str buf;
       olsr_printf(1, "[ENC]Received response-response from non-registered node %s!\n",
                  olsr_ip_to_string(&buf, (union olsr_ip_addr *)&msg->originator));
       return 0;
@@ -834,7 +831,6 @@ parse_rres(char *in_msg)
 
   if(memcmp(msg->res_sig, sha1_hash, SIGNATURE_SIZE) != 0)
     {
-      struct ipaddr_str buf;
       olsr_printf(1, "[ENC]Error in response signature from %s!\n",
                  olsr_ip_to_string(&buf, (union olsr_ip_addr *)&msg->originator));
       
@@ -1156,16 +1152,16 @@ timeout_timestamps(void* foo __attribute__((unused)))
 {
   struct stamp *tmp_list;
   struct stamp *entry_to_delete;
-  int index;
+  int idx;
 
   /* Update our local timestamp */
   gettimeofday(&now, NULL);
 
-  for(index=0;index<HASHSIZE;index++)
+  for(idx=0;idx<HASHSIZE;idx++)
     {
-      tmp_list = timestamps[index].next;
+      tmp_list = timestamps[idx].next;
       /*Traverse MID list*/
-      while(tmp_list != &timestamps[index])
+      while(tmp_list != &timestamps[idx])
        {
          /*Check if the entry is timed out*/
          if((TIMED_OUT(tmp_list->valtime)) && (TIMED_OUT(tmp_list->conftime)))
index 5ca7b13..d556c5c 100644 (file)
@@ -158,8 +158,8 @@ static int
 plugin_ipc_init(void)
 {
     struct sockaddr_storage sst;
-    struct sockaddr_in *sin;
-    struct sockaddr_in6 *sin6;
+    struct sockaddr_in *addr;
+    struct sockaddr_in6 *addr6;
     olsr_u32_t yes = 1;
     socklen_t addrlen;
 
@@ -188,23 +188,23 @@ plugin_ipc_init(void)
         /* complete the socket structure */
         memset(&sst, 0, sizeof(sst));
         if (olsr_cnf->ip_version == AF_INET) {
-           sin = (struct sockaddr_in *)&sst;
-           sin->sin_family = AF_INET;
+           addr = (struct sockaddr_in *)&sst;
+           addr->sin_family = AF_INET;
            addrlen = sizeof(struct sockaddr_in);
 #ifdef SIN6_LEN
-           sin->sin_len = addrlen;
+           addr->sin_len = addrlen;
 #endif
-           sin->sin_addr.s_addr = INADDR_ANY;
-           sin->sin_port = htons(ipc_port);
+           addr->sin_addr.s_addr = INADDR_ANY;
+           addr->sin_port = htons(ipc_port);
         } else {
-           sin6 = (struct sockaddr_in6 *)&sst;
-           sin6->sin6_family = AF_INET6;
+           addr6 = (struct sockaddr_in6 *)&sst;
+           addr6->sin6_family = AF_INET6;
            addrlen = sizeof(struct sockaddr_in6);
 #ifdef SIN6_LEN
-           sin6->sin6_len = addrlen;
+           addr6->sin6_len = addrlen;
 #endif
-           sin6->sin6_addr = in6addr_any;
-           sin6->sin6_port = htons(ipc_port);
+           addr6->sin6_addr = in6addr_any;
+           addr6->sin6_port = htons(ipc_port);
         }
       
         /* bind the socket to the port number */
@@ -238,8 +238,8 @@ plugin_ipc_init(void)
 static void ipc_action(int fd)
 {
     struct sockaddr_storage pin;
-    struct sockaddr_in *sin4;
-    struct sockaddr_in6 *sin6;
+    struct sockaddr_in *addr4;
+    struct sockaddr_in6 *addr6;
     char addr[INET6_ADDRSTRLEN];
     fd_set rfds;
     struct timeval tv;
@@ -259,23 +259,23 @@ static void ipc_action(int fd)
 
     tv.tv_sec = tv.tv_usec = 0;
     if (olsr_cnf->ip_version == AF_INET) {
-        sin4 = (struct sockaddr_in *)&pin;
-        if (inet_ntop(olsr_cnf->ip_version, &sin4->sin_addr, addr,
+        addr4 = (struct sockaddr_in *)&pin;
+        if (inet_ntop(olsr_cnf->ip_version, &addr4->sin_addr, addr,
            INET6_ADDRSTRLEN) == NULL)
              addr[0] = '\0';
-        if (!ip4equal(&sin4->sin_addr, &ipc_accept_ip.v4)) {
+        if (!ip4equal(&addr4->sin_addr, &ipc_accept_ip.v4)) {
             olsr_printf(1, "(TXTINFO) From host(%s) not allowed!\n", addr);
             close(ipc_connection);
             return;
         }
     } else {
-        sin6 = (struct sockaddr_in6 *)&pin;
-        if (inet_ntop(olsr_cnf->ip_version, &sin6->sin6_addr, addr,
+        addr6 = (struct sockaddr_in6 *)&pin;
+        if (inet_ntop(olsr_cnf->ip_version, &addr6->sin6_addr, addr,
            INET6_ADDRSTRLEN) == NULL)
              addr[0] = '\0';
        /* Use in6addr_any (::) in olsr.conf to allow anybody. */
         if (!ip6equal(&in6addr_any, &ipc_accept_ip.v6) &&
-           !ip6equal(&sin6->sin6_addr, &ipc_accept_ip.v6)) {
+           !ip6equal(&addr6->sin6_addr, &ipc_accept_ip.v6)) {
             olsr_printf(1, "(TXTINFO) From host(%s) not allowed!\n", addr);
             close(ipc_connection);
             return;
@@ -350,19 +350,19 @@ static void ipc_print_link(void)
     struct ipaddr_str buf1, buf2;
     struct lqtextbuffer lqbuffer1, lqbuffer2;
 
-    struct link_entry *link = NULL;
+    struct link_entry *lnk;
 
     ipc_sendf("Table: Links\nLocal IP\tRemote IP\tHyst.\tLQ\tNLQ\tCost\n");
 
     /* Link set */
-    OLSR_FOR_ALL_LINK_ENTRIES(link) {
+    OLSR_FOR_ALL_LINK_ENTRIES(lnk) {
        ipc_sendf( "%s\t%s\t%0.2f\t%s\t%s\t\n",
-                   olsr_ip_to_string(&buf1, &link->local_iface_addr),
-                   olsr_ip_to_string(&buf2, &link->neighbor_iface_addr),
-                   link->L_link_quality, 
-                   get_link_entry_text(link, '\t', &lqbuffer1),
-                   get_linkcost_text(link->linkcost, OLSR_FALSE, &lqbuffer2));
-    } OLSR_FOR_ALL_LINK_ENTRIES_END(link);
+                   olsr_ip_to_string(&buf1, &lnk->local_iface_addr),
+                   olsr_ip_to_string(&buf2, &lnk->neighbor_iface_addr),
+                   lnk->L_link_quality, 
+                   get_link_entry_text(lnk, '\t', &lqbuffer1),
+                   get_linkcost_text(lnk->linkcost, OLSR_FALSE, &lqbuffer2));
+    } OLSR_FOR_ALL_LINK_ENTRIES_END(lnk);
 
     ipc_sendf("\n");
 }
@@ -430,7 +430,6 @@ static void ipc_print_hna(void)
     /* Announced HNA entries */
     if (olsr_cnf->ip_version == AF_INET) {
         for(hna = olsr_cnf->hna_entries; hna != NULL; hna = hna->next) {
-            struct ipaddr_str addrbuf, mainaddrbuf;
             ipc_sendf("%s/%d\t%s\n",
                       olsr_ip_to_string(&addrbuf, &hna->net.prefix),
                       hna->net.prefix_len,
@@ -438,7 +437,6 @@ static void ipc_print_hna(void)
         }
     } else {
         for(hna = olsr_cnf->hna_entries; hna != NULL; hna = hna->next) {
-            struct ipaddr_str addrbuf, mainaddrbuf;
             ipc_sendf("%s/%d\t%s\n",
                       olsr_ip_to_string(&addrbuf, &hna->net.prefix),
                       hna->net.prefix_len,
index d3eb242..4046aeb 100644 (file)
@@ -2,13 +2,6 @@
 # FREEBSD SPECIFIC CONFIGURATION
 #
 
-PREFIX ?= /usr/local
-SBINDIR = $(PREFIX)/sbin
-ETCDIR = $(PREFIX)/etc
-LIBDIR = $(PREFIX)/lib
-DOCDIR = $(PREFIX)/share/doc
-MANDIR = $(PREFIX)/man
-
 SRCS +=                $(wildcard src/bsd/*.c) $(wildcard src/unix/*.c)
 HDRS +=                $(wildcard src/bsd/*.h) $(wildcard src/unix/*.h)
 
index 4c7f1de..9753d2e 100644 (file)
@@ -4,7 +4,7 @@
 
 ARCH := $(shell uname -m)
 DESTDIR ?=
-LIBDIR =       $(DESTDIR)$(shell if [ "$(ARCH)" = "x86_64" -a -d "/usr/lib64" ]; then echo "/usr/lib64"; else echo "/usr/lib"; fi)
+LIBDIR =       $(DESTDIR)$(shell if [ "$(ARCH)" = "x86_64" -a -d "$(PREFIX)/lib64" ]; then echo "$(PREFIX)/lib64"; else echo "$(PREFIX)/lib"; fi)
 
 SRCS +=        $(wildcard src/linux/*.c src/unix/*.c)
 HDRS +=                $(wildcard src/linux/*.h src/unix/*.h)
index 474fc2a..fbdd49e 100644 (file)
@@ -2,9 +2,6 @@
 # NETBSD SPECIFIC CONFIGURATION
 #
 
-DESTDIR ?=
-LIBDIR =       $(DESTDIR)/usr/lib
-
 SRCS +=                $(wildcard src/bsd/*.c src/unix/*.c)
 HDRS +=                $(wildcard src/bsd/*.h src/unix/*.h)
 
index 43547ea..1f51af6 100644 (file)
@@ -6,9 +6,6 @@
 # to do it (flags).
 STRIP = \#
 
-DESTDIR ?=
-LIBDIR =       $(DESTDIR)/usr/lib
-
 SRCS +=                $(wildcard src/bsd/*.c src/unix/*.c)
 HDRS +=                $(wildcard src/bsd/*.h src/unix/*.h)
 
index e0c1821..49479b4 100644 (file)
@@ -2,9 +2,6 @@
 # WINDOWS CE SPECIFIC CONFIGURATION
 #
 
-DESTDIR ?=
-LIBDIR =       $(DESTDIR)/usr/lib
-
 SRCS +=                $(wildcard src/win32/*.c)
 HDRS +=                $(wildcard src/win32/*.h)
 
index 61036d9..106c225 100644 (file)
@@ -69,7 +69,7 @@
 
 #ifdef WIN32
 #define close(x) closesocket(x)
-int __stdcall SignalHandler(unsigned long signal) __attribute__((noreturn));
+int __stdcall SignalHandler(unsigned long signo) __attribute__((noreturn));
 void ListInterfaces(void);
 void DisableIcmpRedirects(void);
 olsr_bool olsr_win32_end_request = OLSR_FALSE;
@@ -434,7 +434,7 @@ main(int argc, char *argv[])
  */
 #ifndef WIN32
 void
-olsr_reconfigure(int signal __attribute__((unused)))
+olsr_reconfigure(int signo __attribute__((unused)))
 {
   if(!fork())
     {
@@ -457,15 +457,15 @@ olsr_reconfigure(int signal __attribute__((unused)))
  */
 #ifdef WIN32
 int __stdcall
-SignalHandler(unsigned long signal)
+SignalHandler(unsigned long signo)
 #else
 static void
-olsr_shutdown(int signal __attribute__((unused)))
+olsr_shutdown(int signo USED_ONLY_FOR_DEBUG)
 #endif
 {
   struct interface *ifn;
 
-  OLSR_PRINTF(1, "Received signal %d - shutting down\n", (int)signal);
+  OLSR_PRINTF(1, "Received signal %d - shutting down\n", (int)signo);
 
 #ifdef WIN32
   OLSR_PRINTF(1, "Waiting for the scheduler to stop.\n");
index 3ed6606..b03c6c8 100644 (file)
--- a/src/mpr.c
+++ b/src/mpr.c
@@ -101,18 +101,18 @@ olsr_find_2_hop_neighbors_with_1_link(int willingness)
 {
   
  
-  olsr_u8_t                    index;
+  int                           idx;
   struct neighbor_2_list_entry *two_hop_list_tmp = NULL;
   struct neighbor_2_list_entry *two_hop_list = NULL;
   struct neighbor_entry        *dup_neighbor;
   struct neighbor_2_entry      *two_hop_neighbor = NULL;
 
 
-  for(index=0;index<HASHSIZE;index++)
+  for(idx=0;idx<HASHSIZE;idx++)
     {
 
-      for(two_hop_neighbor = two_hop_neighbortable[index].next;
-         two_hop_neighbor != &two_hop_neighbortable[index];
+      for(two_hop_neighbor = two_hop_neighbortable[idx].next;
+         two_hop_neighbor != &two_hop_neighbortable[idx];
          two_hop_neighbor = two_hop_neighbor->next)
        {
          
@@ -333,13 +333,13 @@ olsr_check_mpr_changes(void)
 static void
 olsr_clear_two_hop_processed(void)
 {
-  int index;
+  int idx;
   
-  for(index=0;index<HASHSIZE;index++)
+  for(idx=0;idx<HASHSIZE;idx++)
     {
       struct neighbor_2_entry  *neighbor_2;
-      for(neighbor_2 = two_hop_neighbortable[index].next;
-         neighbor_2 != &two_hop_neighbortable[index];
+      for(neighbor_2 = two_hop_neighbortable[idx].next;
+         neighbor_2 != &two_hop_neighbortable[idx];
          neighbor_2 = neighbor_2->next)
        {
          /* Clear */
@@ -520,15 +520,14 @@ olsr_calculate_mpr(void)
 static void
 olsr_optimize_mpr_set(void)
 {
-  struct neighbor_entry *a_neighbor, *dup_neighbor;
-  struct neighbor_2_list_entry *two_hop_list;
-  int i, remove;
+  int i;
 
 #if 0
   printf("\n**MPR OPTIMIZING**\n\n");
 #endif
 
   for(i = WILL_NEVER + 1; i < WILL_ALWAYS; i++) {
+    struct neighbor_entry *a_neighbor;
 
     OLSR_FOR_ALL_NBR_ENTRIES(a_neighbor) {
              
@@ -537,13 +536,14 @@ olsr_optimize_mpr_set(void)
       }
              
       if (a_neighbor->is_mpr) {
-        remove = 1;
+        struct neighbor_2_list_entry *two_hop_list;
+        int remove_it = 1;
 
         for (two_hop_list = a_neighbor->neighbor_2_list.next;
              two_hop_list != &a_neighbor->neighbor_2_list;
              two_hop_list = two_hop_list->next) {
                      
-          dup_neighbor = olsr_lookup_neighbor_table(&two_hop_list->neighbor_2->neighbor_2_addr);
+          const struct neighbor_entry *dup_neighbor = olsr_lookup_neighbor_table(&two_hop_list->neighbor_2->neighbor_2_addr);
                      
           if ((dup_neighbor != NULL) && (dup_neighbor->status != NOT_SYM)) {
             continue;
@@ -552,11 +552,12 @@ olsr_optimize_mpr_set(void)
           //printf("\t[%s] coverage %d\n", olsr_ip_to_string(&buf, &two_hop_list->neighbor_2->neighbor_2_addr), two_hop_list->neighbor_2->mpr_covered_count);
           /* Do not remove if we find a entry which need this MPR */
           if (two_hop_list->neighbor_2->mpr_covered_count <= olsr_cnf->mpr_coverage) {
-            remove = 0;
+            remove_it = 0;
+           break;
           }      
         }
 
-        if (remove) {
+        if (remove_it) {
           struct ipaddr_str buf;
           OLSR_PRINTF(3, "MPR OPTIMIZE: removiong mpr %s\n\n", olsr_ip_to_string(&buf, &a_neighbor->neighbor_main_addr));
           a_neighbor->is_mpr = OLSR_FALSE;
index 5a4e783..1fa77a5 100644 (file)
@@ -105,10 +105,10 @@ ohs_listen_loop(void);
 
 #ifdef WIN32
 int __stdcall
-ohs_close(unsigned long signal __attribute__((unused)))
+ohs_close(unsigned long signo __attribute__((unused)))
 #else
 void
-ohs_close(int signal __attribute__((unused)))
+ohs_close(int signo __attribute__((unused)))
 #endif
 {
   printf("OHS: exit\n");
index afb4bce..fe12ca6 100644 (file)
@@ -244,7 +244,6 @@ ohs_cmd_link(const char *args)
   struct ohs_connection *src, *dst;
   struct in_addr iaddr;
   int qual;
-  struct ohs_ip_link *link, *inv_link;
 
   args += get_next_token(args, tok_buf, TOK_BUF_SIZE);
 
@@ -322,23 +321,23 @@ ohs_cmd_link(const char *args)
       struct ipaddr_str srcaddrstr, dstaddrstr;
 
       if(src != dst) {
-        link = get_link(src, &dst->ip_addr);
-        inv_link = bi ? get_link(dst, &src->ip_addr) : NULL;
+        struct ohs_ip_link *fwd_link = get_link(src, &dst->ip_addr);
+        struct ohs_ip_link *inv_link = bi ? get_link(dst, &src->ip_addr) : NULL;
         if(qual == 100)  {
           /* Remove link entry */
-          if (link) {
-           remove_link(src, link);
+          if (fwd_link) {
+           remove_link(src, fwd_link);
           }
           if (inv_link) {
             remove_link(dst, inv_link);
           }
         } else  {
-          if (!link) {
+          if (!fwd_link) {
             /* Create new link */
-            link = add_link(src, dst);
+            fwd_link = add_link(src, dst);
           }
              
-          link->quality = qual;
+          fwd_link->quality = qual;
              
           if (bi) {
             if(!inv_link) {
index d9cf827..1da9b54 100644 (file)
@@ -487,7 +487,6 @@ void olsr_input(int fd) {
       return;
     
     if ((olsr_in_if = if_ifwithsock(fd)) == NULL) {
-      struct ipaddr_str buf;
       OLSR_PRINTF(1, "Could not find input interface for message from %s size %d\n",
           olsr_ip_to_string(&buf, &from_addr),
           cc);
index c14edfb..93113d8 100644 (file)
@@ -197,7 +197,7 @@ olsr_scheduler(void)
  * @return the absolute timer in system clock tick units
  */
 static clock_t
-olsr_jitter(unsigned int rel_time, olsr_u8_t jitter_pct, unsigned int random)
+olsr_jitter(unsigned int rel_time, olsr_u8_t jitter_pct, unsigned int random_val)
 {
   unsigned int jitter_time;
 
@@ -213,7 +213,7 @@ olsr_jitter(unsigned int rel_time, olsr_u8_t jitter_pct, unsigned int random)
    * Play some tricks to avoid overflows with integer arithmetic.
    */
   jitter_time = (jitter_pct * rel_time) / 100;
-  jitter_time = random / (1 + RAND_MAX / jitter_time);
+  jitter_time = random_val / (1 + RAND_MAX / jitter_time);
 
 #if 0
   OLSR_PRINTF(3, "TIMER: jitter %u%% rel_time %ums to %ums\n",
@@ -299,14 +299,14 @@ void
 olsr_init_timers(void)
 {
   struct list_node *timer_head_node;
-  int index;
+  int idx;
 
   OLSR_PRINTF(5, "TIMER: init timers\n");
 
   memset(timer_wheel, 0, sizeof(timer_wheel));
 
   timer_head_node = timer_wheel;
-  for (index = 0; index < TIMER_WHEEL_SLOTS; index++) {
+  for (idx = 0; idx < TIMER_WHEEL_SLOTS; idx++) {
     list_head_init(timer_head_node);
     timer_head_node++;
   }
@@ -539,7 +539,7 @@ olsr_wallclock_string(void)
  * @return buffer to a formatted system time string.
  */
 const char *
-olsr_clock_string(clock_t clock)
+olsr_clock_string(clock_t clk)
 {
   static char buf[4][sizeof("00:00:00.000")];
   static int idx = 0;
@@ -550,7 +550,7 @@ olsr_clock_string(clock_t clock)
   idx = (idx + 1) & 3;
 
   /* On most systems a clocktick is a 10ms quantity. */
-  msec = olsr_cnf->system_tick_divider * (unsigned int)(clock - now_times);
+  msec = olsr_cnf->system_tick_divider * (unsigned int)(clk - now_times);
   sec = msec / MSEC_PER_SEC;
 
   snprintf(ret, sizeof(buf) / 4, "%02u:%02u:%02u.%03u",
index 5420ba1..96efbbf 100644 (file)
@@ -284,7 +284,6 @@ chk_if_changed(struct olsr_if *iface)
 
       if(memcmp(&tmp_saddr6.sin6_addr, &ifp->int6_addr.sin6_addr, olsr_cnf->ipsize) != 0)
        {
-          struct ipaddr_str buf;
          OLSR_PRINTF(1, "New IP address for %s:\n", ifr.ifr_name);
          OLSR_PRINTF(1, "\tOld: %s\n", ip6_to_string(&buf, &ifp->int6_addr.sin6_addr));
          OLSR_PRINTF(1, "\tNew: %s\n", ip6_to_string(&buf, &tmp_saddr6.sin6_addr));
@@ -366,7 +365,6 @@ chk_if_changed(struct olsr_if *iface)
                &((struct sockaddr_in *)&ifr.ifr_netmask)->sin_addr.s_addr, 
                olsr_cnf->ipsize) != 0)
        {
-          struct ipaddr_str buf;
          /* New address */
          OLSR_PRINTF(1, "IPv4 netmask changed for %s\n", ifr.ifr_name);
          OLSR_PRINTF(1, "\tOld:%s\n", ip4_to_string(&buf, ifp->int_netmask.sin_addr));
@@ -392,7 +390,6 @@ chk_if_changed(struct olsr_if *iface)
          
          if(ifp->int_broadaddr.sin_addr.s_addr != ((struct sockaddr_in *)&ifr.ifr_broadaddr)->sin_addr.s_addr)
            {
-              struct ipaddr_str buf;
              /* New address */
              OLSR_PRINTF(1, "IPv4 broadcast changed for %s\n", ifr.ifr_name);
              OLSR_PRINTF(1, "\tOld:%s\n", ip4_to_string(&buf, ifp->int_broadaddr.sin_addr));