Activate some warnings we currently also use in TIP
authorSven-Ola Tuecke <sven-ola@gmx.de>
Thu, 20 Aug 2009 19:22:39 +0000 (21:22 +0200)
committerSven-Ola Tuecke <sven-ola@gmx.de>
Thu, 20 Aug 2009 19:22:39 +0000 (21:22 +0200)
25 files changed:
Makefile.inc
lib/arprefresh/src/olsrd_arprefresh.c
lib/bmf/src/Bmf.c
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/quagga/src/quagga.c
lib/secure/src/md5.c
lib/secure/src/olsrd_secure.c
lib/txtinfo/src/olsrd_txtinfo.c
src/defs.h
src/hna_set.c
src/linux/net.c
src/main.c
src/mid_set.c
src/mpr.c
src/olsr.c
src/olsr_switch/main.c
src/olsr_switch/ohs_cmd.c
src/parser.c
src/scheduler.c
src/unix/ifnet.c

index c57d79f..bdbd833 100644 (file)
@@ -76,10 +76,12 @@ 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 +=   -Wpointer-arith
-#WARNINGS +=   -Wcast-align
+WARNINGS +=    -Wshadow
+WARNINGS +=    -Wformat-2
+WARNINGS +=    -Wsequence-point
+WARNINGS +=    -Wpointer-arith
+WARNINGS +=    -Wcast-align
+# the following 2 do not work yet and need more work on it
 #WARNINGS +=   -Wconversion
 #WARNINGS +=   -Wredundant-decls
 ifeq ($(MUDFLAP),0)
@@ -90,8 +92,9 @@ endif
 #WARNINGS +=   -Wunreachable-code
 WARNINGS +=    -Winline
 WARNINGS +=    -Wdisabled-optimization
-#WARNINGS +=   -Werror
+WARNINGS +=    -Werror
 WARNINGS +=    -finline-functions-called-once
+WARNINGS +=    -funit-at-a-time
 WARNINGS +=    -fearly-inlining
 ifeq ($(DEBUG),0)
 WARNINGS +=    -fomit-frame-pointer
index 49bbacf..609ffe7 100644 (file)
@@ -48,6 +48,7 @@
 #include <netpacket/packet.h>
 #include <linux/types.h>
 #include <linux/filter.h>
+#include <unistd.h>
 
 #include "olsrd_arprefresh.h"
 #include "kernel_routes.h"
index 8e91092..7fcbb3f 100644 (file)
@@ -55,6 +55,7 @@
 #include <signal.h>             /* sigset_t, sigfillset(), sigdelset(), SIGINT */
 #include <netinet/ip.h>         /* struct ip */
 #include <netinet/udp.h>        /* struct udphdr */
+#include <unistd.h>             /* struct udphdr */
 
 /* OLSRD includes */
 #include "plugin_util.h"        /* set_plugin_int */
index 8afe3ba..6a8cd9f 100644 (file)
@@ -415,7 +415,6 @@ FindNeighbors(struct TBestNeighbors *neighbors, struct link_entry **bestNeighbor
       /* Consider only neighbors with an IP address that differs from the
        * 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", PLUGIN_NAME_SHORT,
                     olsr_ip_to_string(&buf, &walker->neighbor_iface_addr));
 
@@ -424,7 +423,6 @@ FindNeighbors(struct TBestNeighbors *neighbors, struct link_entry **bestNeighbor
 
       /* 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", PLUGIN_NAME_SHORT,
                     olsr_ip_to_string(&buf, &walker->neighbor_iface_addr));
 
@@ -433,7 +431,6 @@ FindNeighbors(struct TBestNeighbors *neighbors, struct link_entry **bestNeighbor
 
       /* 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", PLUGIN_NAME_SHORT,
                     olsr_ip_to_string(&buf, &walker->neighbor_iface_addr));
 
@@ -689,13 +686,11 @@ FindNeighbors(struct TBestNeighbors *neighbors, struct link_entry **bestNeighbor
 
       if (walker != bestLinkToNeighbor) {
         if (bestLinkToNeighbor == NULL) {
-          struct ipaddr_str buf;
           OLSR_PRINTF(9, "%s: ----> Not forwarding to %s: no link found\n", PLUGIN_NAME_SHORT,
                       olsr_ip_to_string(&buf, &walker->neighbor_iface_addr));
         } else {
 #ifndef NODEBUG
           struct interface *bestIntf = if_ifwithaddr(&bestLinkToNeighbor->local_iface_addr);
-          struct ipaddr_str buf;
           struct lqtextbuffer lqbuffer;
 #endif
           OLSR_PRINTF(9, "%s: ----> Not forwarding to %s: \"%s\" gives a better link to this neighbor, costing %s\n",
index b043e51..93a1bd6 100644 (file)
@@ -233,14 +233,14 @@ 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) {
       /* Found duplicate entry */
 
@@ -258,8 +258,8 @@ 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 184ac77..9212e81 100644 (file)
@@ -166,15 +166,15 @@ ipc_print_neigh_link(struct autobuf *abuf, 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 *the_link;
   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;
+    the_link = get_best_link_to_neighbor(&neighbor->neighbor_main_addr);
+    if (the_link) {
+      etx = the_link->linkcost;
     }
     style = SOLID;
   }
@@ -190,7 +190,7 @@ ipc_print_neigh_link(struct autobuf *abuf, const struct neighbor_entry *neighbor
 static int
 plugin_ipc_init(void)
 {
-  struct sockaddr_in sin;
+  struct sockaddr_in sock_in;
   uint32_t yes = 1;
 
   if (ipc_socket != -1) {
@@ -220,13 +220,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 = ipc_listen_ip.v4.s_addr;
-  sin.sin_port = htons(ipc_port);
+  memset(&sock_in, 0, sizeof(sock_in));
+  sock_in.sin_family = AF_INET;
+  sock_in.sin_addr.s_addr = ipc_listen_ip.v4.s_addr;
+  sock_in.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 *)&sock_in, sizeof(sock_in)) == -1) {
     olsr_printf(1, "(DOT DRAW)IPC bind %s\n", strerror(errno));
     CLOSE(ipc_socket);
     return 0;
@@ -319,7 +319,7 @@ dotdraw_write_data(void *foo __attribute__ ((unused))) {
  *Scheduled event
  */
 static int
-pcf_event(int changes_neighborhood, int changes_topology, int changes_hna)
+pcf_event(int my_changes_neighborhood, int my_changes_topology, int my_changes_hna)
 {
   struct neighbor_entry *neighbor_table_tmp;
   struct tc_entry *tc;
@@ -334,7 +334,7 @@ pcf_event(int changes_neighborhood, int changes_topology, int changes_hna)
     return 1;
   }
 
-  if (changes_neighborhood || changes_topology || changes_hna) {
+  if (my_changes_neighborhood || my_changes_topology || my_changes_hna) {
     /* Print tables to IPC socket */
     abuf_puts(&outbuffer, "digraph topology\n{\n");
 
index 95def71..33ecd93 100644 (file)
@@ -251,7 +251,7 @@ static const struct dynamic_file_entry dynamic_files[] = {
 static int
 get_http_socket(int port)
 {
-  struct sockaddr_in sin;
+  struct sockaddr_in sock_in;
   uint32_t yes = 1;
 
   /* Init ipc socket */
@@ -270,13 +270,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 = httpinfo_listen_ip.v4.s_addr;
-  sin.sin_port = htons(port);
+  memset(&sock_in, 0, sizeof(sock_in));
+  sock_in.sin_family = AF_INET;
+  sock_in.sin_addr.s_addr = httpinfo_listen_ip.v4.s_addr;
+  sock_in.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 *)&sock_in, sizeof(sock_in)) == -1) {
     olsr_printf(1, "(HTTPINFO) bind failed %s\n", strerror(errno));
     close(s);
     return -1;
@@ -687,7 +687,7 @@ section_title(struct autobuf *abuf, const char *title)
 }
 
 static void
-build_frame(struct autobuf *abuf, const char *title __attribute__ ((unused)), const char *link
+build_frame(struct autobuf *abuf, const char *title __attribute__ ((unused)), const char *the_link
             __attribute__ ((unused)), int width __attribute__ ((unused)), build_body_callback frame_body_cb)
 {
   abuf_puts(abuf, "<div id=\"maintable\">\n");
@@ -945,7 +945,7 @@ static void
 build_neigh_body(struct autobuf *abuf)
 {
   struct neighbor_entry *neigh;
-  struct link_entry *link = NULL;
+  struct link_entry *the_link = NULL;
   const char *colspan = resolve_ip_addresses ? " colspan=\"2\"" : "";
 
   section_title(abuf, "Links");
@@ -959,18 +959,18 @@ build_neigh_body(struct autobuf *abuf)
   abuf_puts(abuf, "</tr>\n");
 
   /* Link set */
-  OLSR_FOR_ALL_LINK_ENTRIES(link) {
+  OLSR_FOR_ALL_LINK_ENTRIES(the_link) {
     abuf_puts(abuf, "<tr>");
-    build_ipaddr_with_link(abuf, &link->local_iface_addr, -1);
-    build_ipaddr_with_link(abuf, &link->neighbor_iface_addr, -1);
-    abuf_appendf(abuf, "<td>%0.2f</td>", link->L_link_quality);
+    build_ipaddr_with_link(abuf, &the_link->local_iface_addr, -1);
+    build_ipaddr_with_link(abuf, &the_link->neighbor_iface_addr, -1);
+    abuf_appendf(abuf, "<td>%0.2f</td>", the_link->L_link_quality);
     if (olsr_cnf->lq_level > 0) {
       struct lqtextbuffer lqbuffer1, lqbuffer2;
-      abuf_appendf(abuf, "<td>(%s) %s</td>", get_link_entry_text(link, '/', &lqbuffer1),
-                 get_linkcost_text(link->linkcost, false, &lqbuffer2));
+      abuf_appendf(abuf, "<td>(%s) %s</td>", get_link_entry_text(the_link, '/', &lqbuffer1),
+                 get_linkcost_text(the_link->linkcost, false, &lqbuffer2));
     }
     abuf_puts(abuf, "</tr>\n");
-  } OLSR_FOR_ALL_LINK_ENTRIES_END(link);
+  } OLSR_FOR_ALL_LINK_ENTRIES_END(the_link);
 
   abuf_puts(abuf, "</table>\n");
 
@@ -1156,10 +1156,10 @@ build_cfgfile_body(struct autobuf *abuf)
 }
 
 static int
-check_allowed_ip(const struct allowed_net *const allowed_nets, const union olsr_ip_addr *const addr)
+check_allowed_ip(const struct allowed_net *const my_allowed_nets, const union olsr_ip_addr *const addr)
 {
   const struct allowed_net *alln;
-  for (alln = allowed_nets; alln != NULL; alln = alln->next) {
+  for (alln = my_allowed_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 a526c7c..a98ac53 100644 (file)
@@ -121,7 +121,7 @@ olsrd_get_plugin_parameters(const struct olsrd_plugin_parameters **params, int *
 }
 
 static int
-insert_plugin_ipnet(const char *sz_net, const char *sz_mask, struct allowed_net **allowed_nets)
+insert_plugin_ipnet(const char *sz_net, const char *sz_mask, struct allowed_net **my_allowed_nets)
 {
   struct allowed_net *an;
 
@@ -135,8 +135,8 @@ insert_plugin_ipnet(const char *sz_net, const char *sz_mask, struct allowed_net
     free(an);
     return 1;
   }
-  an->next = *allowed_nets;
-  *allowed_nets = an;
+  an->next = *my_allowed_nets;
+  *my_allowed_nets = an;
   return 0;
 }
 
index f3c6ea6..9175b4f 100644 (file)
@@ -301,7 +301,7 @@ ipc_action(int fd __attribute__ ((unused)))
  *Scheduled event
  */
 static int
-pcf_event(int changes_neighborhood, int changes_topology, int changes_hna __attribute__ ((unused)))
+pcf_event(int my_changes_neighborhood, int my_changes_topology, int my_changes_hna __attribute__ ((unused)))
 {
   int res;
   struct neighbor_entry *neighbor_table_tmp;
@@ -310,7 +310,7 @@ pcf_event(int changes_neighborhood, int changes_topology, int changes_hna __attr
 
   res = 0;
 
-  if (changes_neighborhood || changes_topology) {
+  if (my_changes_neighborhood || my_changes_topology) {
     /* Print tables to IPC socket */
 
     //ipc_send("start ", strlen("start "));
index 821f78e..d45a80b 100644 (file)
@@ -26,6 +26,7 @@
 #include <sys/socket.h>
 #include <netinet/in.h>
 #include <fcntl.h>
+#include <unistd.h>
 
 #include "quagga.h"
 #include "olsr.h"
index 1e7a3a5..24f96c5 100644 (file)
@@ -258,33 +258,33 @@ 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)) < ((UINT4) inputLen << 3))
     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);
+    MD5_memcpy((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], inputLen - i);
+  MD5_memcpy((POINTER) & context->buffer[idx], &input[i], inputLen - i);
 }
 
 /* MD5 finalization. Ends an MD5 message-digest operation, writing the
@@ -294,15 +294,15 @@ 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 a459763..e190730 100644 (file)
@@ -660,7 +660,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;
@@ -681,7 +680,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));
 
     return 0;
@@ -748,7 +746,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;
@@ -769,7 +766,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));
 
     return 0;
@@ -1044,15 +1040,15 @@ 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++) {
-    tmp_list = timestamps[index].next;
+  for (idx = 0; idx < HASHSIZE; idx++) {
+    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))) {
         struct ipaddr_str buf;
index 5ac8d19..149b2b7 100644 (file)
@@ -157,7 +157,7 @@ static int
 plugin_ipc_init(void)
 {
   struct sockaddr_storage sst;
-  struct sockaddr_in *sin;
+  struct sockaddr_in *sock_in;
   struct sockaddr_in6 *sin6;
   uint32_t yes = 1;
   socklen_t addrlen;
@@ -186,14 +186,14 @@ 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;
+      sock_in = (struct sockaddr_in *)&sst;
+      sock_in->sin_family = AF_INET;
       addrlen = sizeof(struct sockaddr_in);
 #ifdef SIN6_LEN
-      sin->sin_len = addrlen;
+      sock_in->sin_len = addrlen;
 #endif
-      sin->sin_addr.s_addr = txtinfo_listen_ip.v4.s_addr;
-      sin->sin_port = htons(ipc_port);
+      sock_in->sin_addr.s_addr = txtinfo_listen_ip.v4.s_addr;
+      sock_in->sin_port = htons(ipc_port);
     } else {
       sin6 = (struct sockaddr_in6 *)&sst;
       sin6->sin6_family = AF_INET6;
@@ -342,7 +342,7 @@ ipc_print_link(struct autobuf *abuf)
   struct ipaddr_str buf1, buf2;
   struct lqtextbuffer lqbuffer1, lqbuffer2;
 
-  struct link_entry *link = NULL;
+  struct link_entry *my_link = NULL;
 
 #ifdef ACTIVATE_VTIME_TXTINFO
   abuf_puts(abuf, "Table: Links\nLocal IP\tRemote IP\tVtime\tLQ\tNLQ\tCost\n");
@@ -351,22 +351,22 @@ ipc_print_link(struct autobuf *abuf)
 #endif
 
   /* Link set */
-  OLSR_FOR_ALL_LINK_ENTRIES(link) {
+  OLSR_FOR_ALL_LINK_ENTRIES(my_link) {
 #ifdef ACTIVATE_VTIME_TXTINFO
-    int diff = olsr_cnf->system_tick_divider * (unsigned int)(link->ASYM_time - now_times);
+    int diff = olsr_cnf->system_tick_divider * (unsigned int)(my_link->ASYM_time - now_times);
 
-    abuf_appendf(abuf, "%s\t%s\t%d.%03d\t%s\t%s\t\n", olsr_ip_to_string(&buf1, &link->local_iface_addr),
-              olsr_ip_to_string(&buf2, &link->neighbor_iface_addr),
+    abuf_appendf(abuf, "%s\t%s\t%d.%03d\t%s\t%s\t\n", olsr_ip_to_string(&buf1, &my_link->local_iface_addr),
+              olsr_ip_to_string(&buf2, &my_link->neighbor_iface_addr),
               diff/1000, diff%1000,
-              get_link_entry_text(link, '\t', &lqbuffer1),
-              get_linkcost_text(link->linkcost, false, &lqbuffer2));
+              get_link_entry_text(my_link, '\t', &lqbuffer1),
+              get_linkcost_text(my_link->linkcost, false, &lqbuffer2));
 #else
-    abuf_appendf(abuf, "%s\t%s\t\t%s\t%s\t\n", olsr_ip_to_string(&buf1, &link->local_iface_addr),
-              olsr_ip_to_string(&buf2, &link->neighbor_iface_addr),
-              get_link_entry_text(link, '\t', &lqbuffer1),
-              get_linkcost_text(link->linkcost, false, &lqbuffer2));
+    abuf_appendf(abuf, "%s\t%s\t\t%s\t%s\t\n", olsr_ip_to_string(&buf1, &my_link->local_iface_addr),
+              olsr_ip_to_string(&buf2, &my_link->neighbor_iface_addr),
+              get_link_entry_text(my_link, '\t', &lqbuffer1),
+              get_linkcost_text(my_link->linkcost, false, &lqbuffer2));
 #endif
-  } OLSR_FOR_ALL_LINK_ENTRIES_END(link);
+  } OLSR_FOR_ALL_LINK_ENTRIES_END(my_link);
 
   abuf_puts(abuf, "\n");
 }
@@ -436,7 +436,7 @@ ipc_print_hna(struct autobuf *abuf)
   struct ip_prefix_list *hna;
   struct hna_entry *tmp_hna;
   struct hna_net *tmp_net;
-  struct ipaddr_str addrbuf, mainaddrbuf;
+  struct ipaddr_str buf, mainaddrbuf;
 
   size = 0;
 
@@ -445,14 +445,12 @@ ipc_print_hna(struct autobuf *abuf)
   /* 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;
-      abuf_appendf(abuf, "%s/%d\t%s\n", olsr_ip_to_string(&addrbuf, &hna->net.prefix), hna->net.prefix_len,
+      abuf_appendf(abuf, "%s/%d\t%s\n", olsr_ip_to_string(&buf, &hna->net.prefix), hna->net.prefix_len,
                 olsr_ip_to_string(&mainaddrbuf, &olsr_cnf->main_addr));
     }
   } else {
     for (hna = olsr_cnf->hna_entries; hna != NULL; hna = hna->next) {
-      struct ipaddr_str addrbuf, mainaddrbuf;
-      abuf_appendf(abuf, "%s/%d\t%s\n", olsr_ip_to_string(&addrbuf, &hna->net.prefix), hna->net.prefix_len,
+      abuf_appendf(abuf, "%s/%d\t%s\n", olsr_ip_to_string(&buf, &hna->net.prefix), hna->net.prefix_len,
                 olsr_ip_to_string(&mainaddrbuf, &olsr_cnf->main_addr));
     }
   }
@@ -463,7 +461,7 @@ ipc_print_hna(struct autobuf *abuf)
     /* Check all networks */
     for (tmp_net = tmp_hna->networks.next; tmp_net != &tmp_hna->networks; tmp_net = tmp_net->next) {
 
-      abuf_appendf(abuf, "%s/%d\t%s\n", olsr_ip_to_string(&addrbuf, &tmp_net->A_network_addr), tmp_net->prefixlen,
+      abuf_appendf(abuf, "%s/%d\t%s\n", olsr_ip_to_string(&buf, &tmp_net->A_network_addr), tmp_net->prefixlen,
                 olsr_ip_to_string(&mainaddrbuf, &tmp_hna->A_gateway_addr));
     }
   }
@@ -475,7 +473,7 @@ ipc_print_hna(struct autobuf *abuf)
 static void
 ipc_print_mid(struct autobuf *abuf)
 {
-  int index;
+  int idx;
   unsigned short is_first;
   struct mid_entry *entry;
   struct mid_address *alias;
@@ -483,10 +481,10 @@ ipc_print_mid(struct autobuf *abuf)
   abuf_puts(abuf, "Table: MID\nIP address\tAliases\n");
 
   /* MID */
-  for (index = 0; index < HASHSIZE; index++) {
-    entry = mid_set[index].next;
+  for (idx = 0; idx < HASHSIZE; idx++) {
+    entry = mid_set[idx].next;
 
-    while (entry != &mid_set[index]) {
+    while (entry != &mid_set[idx]) {
       struct ipaddr_str buf;
       abuf_puts(abuf, olsr_ip_to_string(&buf, &entry->main_addr));
       alias = entry->aliases;
@@ -556,7 +554,7 @@ txtinfo_write_data(void *foo __attribute__ ((unused))) {
 }
 
 static void
-send_info(int send_what, int socket)
+send_info(int send_what, int the_socket)
 {
   struct autobuf abuf;
 
@@ -594,7 +592,7 @@ send_info(int send_what, int socket)
   outbuffer[outbuffer_count] = olsr_malloc(abuf.len, "txt output buffer");
   outbuffer_size[outbuffer_count] = abuf.len;
   outbuffer_written[outbuffer_count] = 0;
-  outbuffer_socket[outbuffer_count] = socket;
+  outbuffer_socket[outbuffer_count] = the_socket;
 
   memcpy(outbuffer[outbuffer_count], abuf.buf, abuf.len);
   outbuffer_count++;
index 4d68253..69e99a1 100644 (file)
@@ -50,7 +50,6 @@
 #include <stdio.h>
 #include <stdlib.h>
 #include <string.h>
-#include <unistd.h>
 #include <errno.h>
 #include <time.h>
 
index 2afe869..08b3d13 100644 (file)
@@ -345,8 +345,8 @@ olsr_input_hna(union olsr_message *m, struct interface *in_if __attribute__ ((un
   int hnasize;
   const uint8_t *curr, *curr_end;
 
-#ifdef DEBUG
   struct ipaddr_str buf;
+#ifdef DEBUG
   OLSR_PRINTF(5, "Processing HNA\n");
 #endif
 
@@ -402,7 +402,6 @@ olsr_input_hna(union olsr_message *m, struct interface *in_if __attribute__ ((un
    *      message MUST be discarded.
    */
   if (check_neighbor_link(from_addr) != SYM_LINK) {
-    struct ipaddr_str buf;
     OLSR_PRINTF(2, "Received HNA from NON SYM neighbor %s\n", olsr_ip_to_string(&buf, from_addr));
     return false;
   }
index 8bf9da1..dbe7329 100644 (file)
@@ -54,6 +54,7 @@
 #include <string.h>
 #include <stdio.h>
 #include <syslog.h>
+#include <unistd.h>
 
 /* Redirect proc entry */
 #define REDIRECT_PROC "/proc/sys/net/ipv4/conf/%s/send_redirects"
index 84549ad..9eda27a 100644 (file)
@@ -66,7 +66,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);
 bool olsr_win32_end_request = false;
@@ -487,7 +487,7 @@ int main(int argc, char *argv[]) {
  *@param signal the signal that triggered this callback
  */
 #ifndef WIN32
-void olsr_reconfigure(int signal __attribute__ ((unused))) {
+void olsr_reconfigure(int signo __attribute__ ((unused))) {
   /* if we are started with -nofork, we do not weant to go into the
    * background here. So we can simply stop on -HUP
    */
@@ -524,14 +524,14 @@ void 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)))
+static void olsr_shutdown(int signo __attribute__ ((unused)))
 #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 f34ab54..f663972 100644 (file)
@@ -528,9 +528,7 @@ olsr_print_mid_set(void)
 bool
 olsr_input_mid(union olsr_message *m, struct interface *in_if __attribute__ ((unused)), union olsr_ip_addr *from_addr)
 {
-#ifdef DEBUG
   struct ipaddr_str buf;
-#endif
   struct mid_alias *tmp_adr;
   struct mid_message message;
 
@@ -552,7 +550,6 @@ olsr_input_mid(union olsr_message *m, struct interface *in_if __attribute__ ((un
    */
 
   if (check_neighbor_link(from_addr) != SYM_LINK) {
-    struct ipaddr_str buf;
     OLSR_PRINTF(2, "Received MID from NON SYM neighbor %s\n", olsr_ip_to_string(&buf, from_addr));
     olsr_free_mid_packet(&message);
     return false;
@@ -563,7 +560,6 @@ olsr_input_mid(union olsr_message *m, struct interface *in_if __attribute__ ((un
 
   while (tmp_adr) {
     if (!mid_lookup_main_addr(&tmp_adr->alias_addr)) {
-      struct ipaddr_str buf;
       OLSR_PRINTF(1, "MID new: (%s, ", olsr_ip_to_string(&buf, &message.mid_origaddr));
       OLSR_PRINTF(1, "%s)\n", olsr_ip_to_string(&buf, &tmp_adr->alias_addr));
       insert_mid_alias(&message.mid_origaddr, &tmp_adr->alias_addr, message.vtime);
index f16be80..3301178 100644 (file)
--- a/src/mpr.c
+++ b/src/mpr.c
@@ -87,15 +87,15 @@ static struct neighbor_2_list_entry *
 olsr_find_2_hop_neighbors_with_1_link(int willingness)
 {
 
-  uint8_t index;
+  uint8_t 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) {
 
       //two_hop_neighbor->neighbor_2_state=0;
@@ -292,11 +292,11 @@ 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]; neighbor_2 = neighbor_2->next) {
+    for (neighbor_2 = two_hop_neighbortable[idx].next; neighbor_2 != &two_hop_neighbortable[idx]; neighbor_2 = neighbor_2->next) {
       /* Clear */
       neighbor_2->processed = 0;
     }
@@ -465,7 +465,7 @@ 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, removeit;
 
 #if 0
   printf("\n**MPR OPTIMIZING**\n\n");
@@ -480,7 +480,7 @@ olsr_optimize_mpr_set(void)
       }
 
       if (a_neighbor->is_mpr) {
-        remove = 1;
+        removeit = 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) {
@@ -493,11 +493,11 @@ 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;
+            removeit = 0;
           }
         }
 
-        if (remove) {
+        if (removeit) {
           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 = false;
index 0d4ca4f..b29b136 100644 (file)
@@ -66,6 +66,7 @@
 
 #include <stdarg.h>
 #include <signal.h>
+#include <unistd.h>
 
 bool changes_topology;
 bool changes_neighborhood;
index ef25459..80ae1dc 100644 (file)
@@ -99,10 +99,10 @@ static void 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 d01d304..e686d8c 100644 (file)
@@ -225,7 +225,7 @@ ohs_cmd_link(const char *args)
   struct ohs_connection *src, *dst;
   struct in_addr iaddr;
   int qual;
-  struct ohs_ip_link *link, *inv_link;
+  struct ohs_ip_link *my_link, *inv_link;
 
   args += get_next_token(args, tok_buf, TOK_BUF_SIZE);
 
@@ -303,23 +303,23 @@ ohs_cmd_link(const char *args)
       struct ipaddr_str srcaddrstr, dstaddrstr;
 
       if (src != dst) {
-        link = get_link(src, &dst->ip_addr);
+        my_link = get_link(src, &dst->ip_addr);
         inv_link = bi ? get_link(dst, &src->ip_addr) : NULL;
         if (qual == 100) {
           /* Remove link entry */
-          if (link) {
-            remove_link(src, link);
+          if (my_link) {
+            remove_link(src, my_link);
           }
           if (inv_link) {
             remove_link(dst, inv_link);
           }
         } else {
-          if (!link) {
+          if (!my_link) {
             /* Create new link */
-            link = add_link(src, dst);
+            my_link = add_link(src, dst);
           }
 
-          link->quality = qual;
+          my_link->quality = qual;
 
           if (bi) {
             if (!inv_link) {
index 59d1332..00adca9 100644 (file)
@@ -406,9 +406,7 @@ olsr_input(int fd)
   cpu_overload_exit = 0;
 
   for (;;) {
-#ifdef DEBUG
     struct ipaddr_str buf;
-#endif
     /* sockaddr_in6 is bigger than sockaddr !!!! */
     struct sockaddr_storage from;
     socklen_t fromlen;
@@ -452,7 +450,6 @@ 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);
       olsr_syslog(OLSR_LOG_ERR, "Could not find input interface for message from %s size %d\n", olsr_ip_to_string(&buf, &from_addr),
                   cc);
index b4bb706..a60e640 100644 (file)
@@ -188,7 +188,7 @@ olsr_scheduler(void)
  * @return the absolute timer in system clock tick units
  */
 static clock_t
-olsr_jitter(unsigned int rel_time, uint8_t jitter_pct, unsigned int random)
+olsr_jitter(unsigned int rel_time, uint8_t jitter_pct, unsigned int random_val)
 {
   unsigned int jitter_time;
 
@@ -204,7 +204,7 @@ olsr_jitter(unsigned int rel_time, uint8_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", jitter_pct, rel_time, rel_time - jitter_time);
@@ -283,14 +283,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++;
   }
@@ -492,7 +492,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 the_clock)
 {
   static char buf[4][sizeof("00:00:00.000")];
   static int idx = 0;
@@ -503,7 +503,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)(the_clock - now_times);
   sec = msec / MSEC_PER_SEC;
 
   snprintf(ret, sizeof(buf) / 4, "%02u:%02u:%02u.%03u", sec / 3600, (sec % 3600) / 60, (sec % 60), (msec % MSEC_PER_SEC));
index a2df8a1..677a987 100644 (file)
@@ -237,9 +237,7 @@ chk_if_changed(struct olsr_if *iface)
 
   /* IP version 6 */
   if (olsr_cnf->ip_version == AF_INET6) {
-#ifdef DEBUG
     struct ipaddr_str buf;
-#endif
     /* Get interface address */
 
     if (get_ipv6_address(ifr.ifr_name, &tmp_saddr6, iface->cnf->ipv6_addrtype) <= 0) {
@@ -255,7 +253,6 @@ chk_if_changed(struct olsr_if *iface)
 #endif
 
     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));
@@ -325,7 +322,6 @@ chk_if_changed(struct olsr_if *iface)
     if (memcmp
         (&((struct sockaddr_in *)&ifp->int_netmask)->sin_addr.s_addr, &((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));
@@ -347,7 +343,6 @@ chk_if_changed(struct olsr_if *iface)
 #endif
 
       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));