* killed lots of #ifdef ... #endif which make the source quite unreadable
authorBernd Petrovitsch <bernd@firmix.at>
Fri, 1 Aug 2008 13:35:46 +0000 (15:35 +0200)
committerBernd Petrovitsch <bernd@firmix.at>
Fri, 1 Aug 2008 13:35:46 +0000 (15:35 +0200)
27 files changed:
Makefile.inc
lib/bmf/src/Bmf.c
lib/bmf/src/NetworkInterfaces.c
lib/nameservice/src/nameservice.c
src/build_msg.c
src/duplicate_set.c
src/hna_set.c
src/hysteresis.c
src/ipcalc.c
src/ipcalc.h
src/link_set.c
src/linux/net.c
src/main.c
src/mid_set.c
src/mpr.c
src/mpr_selector_set.c
src/neighbor_table.c
src/net_olsr.c
src/olsr.c
src/olsr_spf.c
src/packet.c
src/parser.c
src/process_package.c
src/rebuild_packet.c
src/routing_table.c
src/tc_set.c
src/unix/ifnet.c

index a76b9dc..21dc098 100644 (file)
@@ -8,7 +8,7 @@
 DEBUG ?= 0
 
 # compile OLSR_PRINTF out
-NO_DEBUG_MESSAGES ?= 0
+NO_DEBUG_MESSAGES ?= 1
 
 # the optimize option to be set for gcc
 OPTIMIZE ?= 
@@ -185,6 +185,7 @@ CPPFLAGS += -DNDEBUG
 endif
 ifeq ($(NO_DEBUG_MESSAGES),1)
 CPPFLAGS +=    -DNODEBUG
+CPPFLAGS +=    -DUSED_FOR_DEBUG_ONLY='__attribute__((unused))'
 endif
 
 # a make function to quote "/" and "."
index 3b116fe..ab0ec52 100644 (file)
@@ -290,9 +290,7 @@ static void BmfPacketCaptured(
   struct ip* ipHeader; /* The IP header inside the captured IP packet */
   u_int32_t crc32;
   struct TEncapHeader* encapHdr;
-#ifndef NODEBUG
   struct ipaddr_str srcBuf, dstBuf;
-#endif
   ipHeader = GetIpHeader(encapsulationUdpData);
 
   dst.v4 = ipHeader->ip_dst;
@@ -439,9 +437,7 @@ static void BmfPacketCaptured(
       {
         /* Case 1.1 */
         {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
           OLSR_PRINTF(
             8,
             "%s: --> not encap-forwarding on \"%s\": I am not selected as MPR by neighbor %s\n",
@@ -616,9 +612,7 @@ static void BmfEncapsulationPacketReceived(
   struct TEncapHeader* encapsulationHdr;
   u_int16_t encapsulationUdpDataLen;
   struct TBmfInterface* walker;
-#ifndef NODEBUG
   struct ipaddr_str mcSrcBuf, mcDstBuf, forwardedByBuf, forwardedToBuf;
-#endif
   /* Are we talking to ourselves? */
   if (if_ifwithaddr(forwardedBy) != NULL)
   {
@@ -897,9 +891,7 @@ static void BmfEncapsulationPacketReceived(
 
     else /* walker->olsrIntf != NULL && !iAmMpr */
     {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       /* 'walker' is an OLSR interface, but I am not selected as MPR. In that
        * case, don't forward. */
       OLSR_PRINTF(
@@ -934,9 +926,7 @@ static void BmfTunPacketCaptured(unsigned char* encapsulationUdpData)
   struct ip* ipHeader;
   u_int32_t crc32;
   struct TEncapHeader* encapHdr;
-#ifndef NODEBUG
   struct ipaddr_str srcIpBuf, dstIpBuf;
-#endif
   ipPacket = GetIpPacket(encapsulationUdpData);
   ipPacketLen = GetIpTotalLength(ipPacket);
   ipHeader = GetIpHeader(encapsulationUdpData);
index 38f3c31..fc503e9 100644 (file)
@@ -431,9 +431,7 @@ void FindNeighbors(
     struct link_entry* walker;
 
     OLSR_FOR_ALL_LINK_ENTRIES(walker) {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       union olsr_ip_addr* neighborMainIp;
 
       /* Consider only links from the specified interface */
@@ -455,9 +453,7 @@ void FindNeighbors(
        * passed IP addresses (if passed). Rely on short-circuit boolean evaluation. */
       if (source != NULL && ipequal(neighborMainIp, MainAddressOf(source)))
       {
-#ifndef NODEBUG
         struct ipaddr_str buf;
-#endif
         OLSR_PRINTF(
           9,
           "%s: ----> Not forwarding to %s: is source of pkt\n",
@@ -470,9 +466,7 @@ void FindNeighbors(
       /* Rely on short-circuit boolean evaluation */
       if (forwardedBy != NULL && ipequal(neighborMainIp, MainAddressOf(forwardedBy)))
       {
-#ifndef NODEBUG
         struct ipaddr_str buf;
-#endif
         OLSR_PRINTF(
           9,
           "%s: ----> Not forwarding to %s: is the node that forwarded the pkt\n",
@@ -485,9 +479,7 @@ void FindNeighbors(
       /* Rely on short-circuit boolean evaluation */
       if (forwardedTo != NULL && ipequal(neighborMainIp, MainAddressOf(forwardedTo)))
       {
-#ifndef NODEBUG
         struct ipaddr_str buf;
-#endif
         OLSR_PRINTF(
           9,
           "%s: ----> Not forwarding to %s: is the node to which the pkt was forwarded\n",
@@ -542,9 +534,7 @@ void FindNeighbors(
     /* TODO: get_link_set() is not thread-safe! */
     for (walker = get_link_set(); walker != NULL; walker = walker->next) 
     {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       union olsr_ip_addr* neighborMainIp;
       struct link_entry* bestLinkToNeighbor;
       struct tc_entry* tcLastHop;
@@ -656,9 +646,7 @@ void FindNeighbors(
 
       if (forwardedBy != NULL)
       {
-#ifndef NODEBUG
         struct ipaddr_str forwardedByBuf, niaBuf;
-#endif
         OLSR_PRINTF(
           9,
           "%s: ----> 2-hop path from %s via me to %s will cost %5.2f\n",
@@ -743,9 +731,7 @@ void FindNeighbors(
     }
 
     OLSR_FOR_ALL_LINK_ENTRIES(walker) {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       union olsr_ip_addr* neighborMainIp;
       struct link_entry* bestLinkToNeighbor;
       struct tc_entry* tcLastHop;
@@ -837,9 +823,7 @@ void FindNeighbors(
       {
         if (bestLinkToNeighbor == NULL)
         {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
           OLSR_PRINTF(
             9,
             "%s: ----> Not forwarding to %s: no link found\n",
@@ -954,9 +938,7 @@ void FindNeighbors(
   }
   else
   {
-#ifndef NODEBUG
     struct ipaddr_str buf;
-#endif
     OLSR_PRINTF(
       9,
       "%s: ----> %d neighbors found on \"%s\"; best neighbor to forward to: %s\n",
index 148e5d7..bca01cd 100644 (file)
@@ -427,9 +427,7 @@ remove_nonvalid_names_from_list(struct name_entry *my_list, int type)
        }
 
        if ( !valid  ) {
-#ifndef NODEBUG
                struct ipaddr_str strbuf;
-#endif
                OLSR_PRINTF(1, "NAME PLUGIN: invalid or malformed parameter %s (%s), fix your config!\n", my_list->name, olsr_ip_to_string(&strbuf, &my_list->ip));
                next = my_list->next;
                free(my_list->name);
@@ -438,9 +436,7 @@ remove_nonvalid_names_from_list(struct name_entry *my_list, int type)
                my_list = NULL;
                return remove_nonvalid_names_from_list(next, type);
        } else {
-#ifndef NODEBUG
                struct ipaddr_str strbuf;
-#endif
                OLSR_PRINTF(2, "NAME PLUGIN: validate parameter %s (%s) -> OK\n", my_list->name, olsr_ip_to_string(&strbuf, &my_list->ip));
                my_list->next = remove_nonvalid_names_from_list(my_list->next, type);
                return my_list;
@@ -541,9 +537,7 @@ olsr_start_write_file_timer(void)
 void
 olsr_namesvc_delete_db_entry(struct db_entry *db)
 {
-#ifndef NODEBUG
        struct ipaddr_str strbuf;
-#endif
        OLSR_PRINTF(2, "NAME PLUGIN: %s timed out... deleting\n", 
                                olsr_ip_to_string(&strbuf, &db->originator));
 
@@ -673,9 +667,7 @@ olsr_parser(union olsr_message *m,
        /* Check that the neighbor this message was received from is symmetric. 
        If not - back off*/
        if(check_neighbor_link(ipaddr) != SYM_LINK) {
-#ifndef NODEBUG
                struct ipaddr_str strbuf;
-#endif
                OLSR_PRINTF(3, "NAME PLUGIN: Received msg from NON SYM neighbor %s\n", olsr_ip_to_string(&strbuf, ipaddr));
                return;
        }
@@ -768,9 +760,7 @@ create_packet(struct name* to, struct name_entry *from)
 {
        char *pos = (char*) to;
        int k;
-#ifndef NODEBUG
        struct ipaddr_str strbuf;
-#endif
        OLSR_PRINTF(3, "NAME PLUGIN: Announcing name %s (%s) %d\n", 
                from->name, olsr_ip_to_string(&strbuf, &from->ip), from->len);
        to->type = htons(from->type);
@@ -790,9 +780,7 @@ create_packet(struct name* to, struct name_entry *from)
 void
 decap_namemsg(struct name *from_packet, struct name_entry **to, olsr_bool *this_table_changed )
 {
-#ifndef NODEBUG
        struct ipaddr_str strbuf;
-#endif
        struct name_entry *tmp;
        struct name_entry *already_saved_name_entries;
        char *name = (char*)from_packet + sizeof(struct name);
@@ -845,9 +833,7 @@ decap_namemsg(struct name *from_packet, struct name_entry **to, olsr_bool *this_
                        }
                        if (!ipequal(&already_saved_name_entries->ip, &from_packet->ip))
                        {
-#ifndef NODEBUG
                                struct ipaddr_str strbuf2, strbuf3;
-#endif
                                OLSR_PRINTF(4, "NAME PLUGIN: updating ip %s -> %s (%s)\n",
                                        olsr_ip_to_string(&strbuf, &already_saved_name_entries->ip),
                                        olsr_ip_to_string(&strbuf2, &from_packet->ip),
@@ -893,9 +879,7 @@ decap_namemsg(struct name *from_packet, struct name_entry **to, olsr_bool *this_
 void
 update_name_entry(union olsr_ip_addr *originator, struct namemsg *msg, int msg_size, olsr_reltime vtime)
 {
-#ifndef NODEBUG
        struct ipaddr_str strbuf;
-#endif
        char *pos, *end_pos;
        struct name *from_packet; 
        int i;
@@ -973,9 +957,7 @@ insert_new_name_in_list(union olsr_ip_addr *originator,
                entry = list2db(list_node);
 
                if (ipequal(originator, &entry->originator)) {
-#ifndef NODEBUG
                        struct ipaddr_str strbuf;
-#endif
                        // found
                        OLSR_PRINTF(4, "NAME PLUGIN: found entry for (%s) in its hash table\n",
                                                olsr_ip_to_string(&strbuf, originator));
@@ -993,9 +975,7 @@ insert_new_name_in_list(union olsr_ip_addr *originator,
 
        if (! entry_found)
        {
-#ifndef NODEBUG
                struct ipaddr_str strbuf;
-#endif
                OLSR_PRINTF(3, "NAME PLUGIN: create new db entry for ip (%s) in hash table\n",
                                        olsr_ip_to_string(&strbuf, originator));
 
@@ -1469,9 +1449,7 @@ allowed_ip(const union olsr_ip_addr *addr)
        struct ip_prefix_list *hna;
        struct interface *iface;
        union olsr_ip_addr tmp_ip, tmp_msk;
-#ifndef NODEBUG
        struct ipaddr_str strbuf;
-#endif
        
        OLSR_PRINTF(6, "checking %s\n", olsr_ip_to_string(&strbuf, addr));
        
@@ -1574,9 +1552,7 @@ allowed_hostname_or_ip_in_service(const char *service_line, const regmatch_t *ho
        //ip in service-line is allowed 
        if (inet_pton(olsr_cnf->ip_version, hostname_or_ip, &olsr_ip) > 0) {
                if (allowed_ip(&olsr_ip)) {
-#ifndef NODEBUG
                        struct ipaddr_str strbuf;
-#endif
                        OLSR_PRINTF(2, "NAME PLUGIN: ip %s in service %s is OK\n", olsr_ip_to_string(&strbuf, &olsr_ip), service_line);
                        free(hostname_or_ip);
                        hostname_or_ip = NULL;
index 4278adc..f6ee24e 100644 (file)
@@ -320,7 +320,7 @@ serialize_hello4(struct hello_message *message, struct interface *ifp)
       /* Link statuses */
       for(j = 0; j <= MAX_LINK; j++)
        {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
           struct ipaddr_str buf;
 #endif
 
@@ -491,7 +491,7 @@ serialize_hello6(struct hello_message *message, struct interface *ifp)
     {
       for(j = 0; j <= MAX_LINK; j++)
        {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
           struct ipaddr_str buf;
 #endif
          first_entry = OLSR_TRUE;
@@ -606,7 +606,7 @@ serialize_hello6(struct hello_message *message, struct interface *ifp)
 static olsr_bool
 serialize_tc4(struct tc_message *message, struct interface *ifp)           
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str buf;
 #endif
   olsr_u16_t remainsize, curr_size;
@@ -733,7 +733,7 @@ serialize_tc4(struct tc_message *message, struct interface *ifp)
 static olsr_bool
 serialize_tc6(struct tc_message *message, struct interface *ifp)           
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str buf;
 #endif
   olsr_u16_t remainsize, curr_size;
@@ -892,7 +892,7 @@ serialize_mid4(struct interface *ifp)
     {
       if(!ipequal(&olsr_cnf->main_addr, &ifs->ip_addr))
        {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
           struct ipaddr_str buf;
 #endif
 
@@ -994,7 +994,7 @@ serialize_mid6(struct interface *ifp)
     {
       if(!ipequal(&olsr_cnf->main_addr, &ifs->ip_addr))
        {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
           struct ipaddr_str buf;
 #endif
          if((curr_size + olsr_cnf->ipsize) > remainsize)
index d608b58..67c15b1 100644 (file)
@@ -89,9 +89,7 @@ int olsr_shall_process_message(void *ip, olsr_u16_t seqnr) {
   void *mainIp;
   clock_t valid_until;
 
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   // get main address
   mainIp = mid_lookup_main_addr(ip);
   if (mainIp == NULL) {
index aab2c6b..a29f9d3 100644 (file)
@@ -208,7 +208,7 @@ olsr_add_hna_net(struct hna_entry *hna_gw, const union olsr_ip_addr *net,
 static void
 olsr_expire_hna_net_entry(void *context)
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str buf1, buf2;
 #endif
   struct hna_net *net_to_delete;
@@ -332,18 +332,14 @@ olsr_print_hna_set(void)
            {
              if(olsr_cnf->ip_version == AF_INET)
                {
-#ifndef NODEBUG
                   struct ipaddr_str buf;
-#endif
                  OLSR_PRINTF(1, "%-15s ", olsr_ip_to_string(&buf, &tmp_net->A_network_addr));
                  OLSR_PRINTF(1, "%-15d ", tmp_net->prefix_len);
                  OLSR_PRINTF(1, "%-15s\n", olsr_ip_to_string(&buf, &tmp_hna->A_gateway_addr));
                }
              else
                {
-#ifndef NODEBUG
                   struct ipaddr_str buf;
-#endif
                  OLSR_PRINTF(1, "%-27s/%d", olsr_ip_to_string(&buf, &tmp_net->A_network_addr), tmp_net->A_netmask.v6);
                  OLSR_PRINTF(1, "%s\n", olsr_ip_to_string(&buf, &tmp_hna->A_gateway_addr));
                }
index fe735a2..28a0ebc 100644 (file)
@@ -77,9 +77,7 @@ olsr_process_hysteresis(struct link_entry *entry)
     {
       if(entry->L_link_pending == 1)
        {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
          OLSR_PRINTF(1, "HYST[%s] link set to NOT pending!\n", 
                      olsr_ip_to_string(&buf, &entry->neighbor_iface_addr));
          changes_neighborhood = OLSR_TRUE;
@@ -101,9 +99,7 @@ olsr_process_hysteresis(struct link_entry *entry)
     {
       if(entry->L_link_pending == 0)
        {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
          OLSR_PRINTF(1, "HYST[%s] link set to pending!\n", 
                      olsr_ip_to_string(&buf, &entry->neighbor_iface_addr));
          changes_neighborhood = OLSR_TRUE;
@@ -149,9 +145,7 @@ olsr_process_hysteresis(struct link_entry *entry)
 void
 olsr_update_hysteresis_hello(struct link_entry *entry, olsr_reltime htime)
 {
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   OLSR_PRINTF(3, "HYST[%s]: HELLO update vtime %u ms\n",
               olsr_ip_to_string(&buf, &entry->neighbor_iface_addr), htime+htime/2);
 
@@ -171,7 +165,7 @@ update_hysteresis_incoming(union olsr_ip_addr *remote, struct interface *local,
   /* Calculate new quality */
   if(lnk != NULL)
     {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
       struct ipaddr_str buf;
 #endif
       lnk->L_link_quality = olsr_hyst_calc_stability(lnk->L_link_quality);
index e13e590..26d34cb 100644 (file)
@@ -71,9 +71,7 @@ prefix_to_netmask(olsr_u8_t *a, int len, olsr_u8_t prefixlen)
 olsr_u8_t
 netmask_to_prefix(const olsr_u8_t *adr, int len)
 {
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   const olsr_u8_t * const a_end = adr+len;
   olsr_u16_t prefix = 0;
   const olsr_u8_t *a;
index 52378e3..2178125 100644 (file)
@@ -49,7 +49,7 @@
 
 struct ipaddr_str {
     char buf[MAX(INET6_ADDRSTRLEN, INET_ADDRSTRLEN)];
-};
+} USED_FOR_DEBUG_ONLY;
 
 /*
  * Macros for comparing and copying IP addresses
index 5a3fdce..21d1de8 100644 (file)
@@ -428,9 +428,7 @@ olsr_expire_link_sym_timer(void *context)
 void
 olsr_expire_link_hello_timer(void *context)
 {
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   struct link_entry *link;
 
   link = (struct link_entry *)context;
@@ -514,9 +512,7 @@ add_link_entry(const union olsr_ip_addr *local,
 
 #ifdef DEBUG
   {
-#ifndef NODEBUG
     struct ipaddr_str localbuf, rembuf;
-#endif
     OLSR_PRINTF(1, "Adding %s=>%s to link set\n",
                olsr_ip_to_string(&localbuf, local), olsr_ip_to_string(&rembuf,
                                                                       remote));
@@ -587,9 +583,7 @@ add_link_entry(const union olsr_ip_addr *local,
   neighbor = olsr_lookup_neighbor_table(remote_main);
   if (!neighbor) {
 #ifdef DEBUG
-#ifndef NODEBUG
     struct ipaddr_str buf;
-#endif
     OLSR_PRINTF(3, "ADDING NEW NEIGHBOR ENTRY %s FROM LINK SET\n",
                olsr_ip_to_string(&buf, remote_main));
 #endif
index c17f339..7942de6 100644 (file)
@@ -583,9 +583,7 @@ int
 join_mcast(struct interface *ifs, int sock)
 {
   /* See linux/in6.h */
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   struct ipv6_mreq mcastreq;
 
   mcastreq.ipv6mr_multiaddr = ifs->int6_multaddr.sin6_addr;
index 7399c08..7162862 100644 (file)
@@ -109,9 +109,7 @@ main(int argc, char *argv[])
   struct if_config_options *default_ifcnf;
   char conf_file_name[FILENAME_MAX];
   struct tms tms_buf;
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
 #ifdef WIN32
   WSADATA WsaData;
   size_t len;
index 753f310..979e325 100644 (file)
@@ -85,7 +85,7 @@ olsr_init_mid_set(void)
 static void
 olsr_expire_mid_entry(void *context)
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str buf;
 #endif
   struct mid_entry *mid;
@@ -201,9 +201,7 @@ insert_mid_tuple(union olsr_ip_addr *m_addr, struct mid_address *alias,
     /* Delete possible 2 hop neighbor */
     if ((tmp_2_neighbor =
         olsr_lookup_two_hop_neighbor_table_mid(&tmp_adr->alias)) != NULL) {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(1, "Deleting 2 hop node from MID: %s to ",
                  olsr_ip_to_string(&buf, &tmp_adr->alias));
       OLSR_PRINTF(1, "%s\n", olsr_ip_to_string(&buf, m_addr));
@@ -218,9 +216,7 @@ insert_mid_tuple(union olsr_ip_addr *m_addr, struct mid_address *alias,
          olsr_lookup_neighbor_table_alias(&tmp_adr->alias)) != NULL)
        && ((real_neigh = olsr_lookup_neighbor_table_alias(m_addr)) != NULL))
     {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(1, "[MID]Deleting bogus neighbor entry %s real ",
                  olsr_ip_to_string(&buf, &tmp_adr->alias));
       OLSR_PRINTF(1, "%s\n", olsr_ip_to_string(&buf, m_addr));
@@ -256,9 +252,7 @@ insert_mid_alias(union olsr_ip_addr *main_add, const union olsr_ip_addr *alias,
   struct neighbor_entry *ne_old, *ne_new;
   struct mid_entry *me_old;
   int ne_ref_rp_count;
-#ifndef NODEBUG
   struct ipaddr_str buf1, buf2;
-#endif
   struct mid_address *adr;
   if (!olsr_validate_address(alias))
     return;
@@ -385,9 +379,7 @@ int
 olsr_update_mid_table(const union olsr_ip_addr *adr, olsr_reltime vtime)
 {
   olsr_u32_t hash;
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   struct mid_entry *tmp_list = mid_set;
 
   OLSR_PRINTF(3, "MID: update %s\n", olsr_ip_to_string(&buf, adr));
@@ -452,9 +444,7 @@ olsr_prune_aliases(const union olsr_ip_addr *m_addr,
     }
 
     if (declared_aliases == NULL) {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       /* Current alias not found in list of declared aliases: free current alias */
       OLSR_PRINTF(1, "MID remove: (%s, ",
                  olsr_ip_to_string(&buf, &entry->main_addr));
@@ -550,9 +540,7 @@ olsr_print_mid_set(void)
     for (tmp_list = mid_set[idx].next; tmp_list != &mid_set[idx];
         tmp_list = tmp_list->next) {
       struct mid_address *tmp_addr;
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(1, "%s: ", olsr_ip_to_string(&buf, &tmp_list->main_addr));
       for (tmp_addr = tmp_list->aliases; tmp_addr;
           tmp_addr = tmp_addr->next_alias) {
index 87a5440..4e2daf1 100644 (file)
--- a/src/mpr.c
+++ b/src/mpr.c
@@ -168,9 +168,7 @@ olsr_chosen_mpr(struct neighbor_entry *one_hop_neighbor, olsr_u16_t *two_hop_cov
   struct neighbor_2_list_entry *second_hop_entries; 
   struct neighbor_entry        *dup_neighbor;
   olsr_u16_t                   count;
-#ifndef NODEBUG
   struct ipaddr_str            buf;
-#endif  
   count = *two_hop_covered_count;
 
   OLSR_PRINTF(1, "Setting %s as MPR\n", olsr_ip_to_string(&buf, &one_hop_neighbor->neighbor_main_addr));
@@ -414,9 +412,7 @@ add_will_always_nodes(void)
 #endif
 
   OLSR_FOR_ALL_NBR_ENTRIES(a_neighbor) {
-#ifndef NODEBUG
     struct ipaddr_str buf;
-#endif
     if ((a_neighbor->status == NOT_SYM) ||
         (a_neighbor->willingness != WILL_ALWAYS)) {
       continue;
@@ -559,9 +555,7 @@ olsr_optimize_mpr_set(void)
         }
 
         if (remove) {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
           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 028608d..aa01c0a 100644 (file)
@@ -99,7 +99,7 @@ olsr_is_mpr(void)
 static void
 olsr_expire_mpr_sel_entry(void *context)
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str buf;
 #endif
   struct mpr_selector *mpr_sel;
@@ -146,9 +146,7 @@ olsr_set_mpr_sel_timer(struct mpr_selector *mpr_sel, olsr_reltime rel_timer)
 struct mpr_selector *
 olsr_add_mpr_selector(const union olsr_ip_addr *addr, olsr_reltime vtime)
 {
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   struct mpr_selector *new_entry;
 
   OLSR_PRINTF(1, "MPRS: adding %s\n", olsr_ip_to_string(&buf, addr));
@@ -210,9 +208,7 @@ olsr_lookup_mprs_set(const union olsr_ip_addr *addr)
 int
 olsr_update_mprs_set(const union olsr_ip_addr *addr, olsr_reltime vtime)
 {
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   struct mpr_selector *mprs = olsr_lookup_mprs_set(addr);
 
   OLSR_PRINTF(5, "MPRS: Update %s\n", olsr_ip_to_string(&buf, addr));
@@ -237,9 +233,7 @@ olsr_print_mprs_set(void)
   struct mpr_selector *mprs;
   OLSR_PRINTF(1, "MPR SELECTORS: ");
   for(mprs = mprs_list.next; mprs != &mprs_list; mprs = mprs->next) {
-#ifndef NODEBUG
     struct ipaddr_str buf;
-#endif
     OLSR_PRINTF(1, "%s ", olsr_ip_to_string(&buf, &mprs->MS_main_addr));
   }
   OLSR_PRINTF(1, "\n");
index bf71e4f..31b1fa7 100644 (file)
@@ -418,9 +418,7 @@ olsr_print_neighbor_table(void)
     for(neigh = neighbortable[idx].next; neigh != &neighbortable[idx]; neigh = neigh->next) {
       struct link_entry *lnk = get_best_link_to_neighbor(&neigh->neighbor_main_addr);
       if(lnk) {
-#ifndef NODEBUG
         struct ipaddr_str buf;
-#endif
         OLSR_PRINTF(1, "%-*s  %5.3f  %5.3f  %s  %s  %s  %d\n",
                     iplen,
                     olsr_ip_to_string(&buf, &neigh->neighbor_main_addr),
index 67399d5..52cf854 100644 (file)
@@ -461,9 +461,7 @@ net_output(struct interface *ifp)
 void
 olsr_add_invalid_address(const union olsr_ip_addr *adr)
 {
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   struct deny_address_entry *new_entry = olsr_malloc(sizeof(struct deny_address_entry), "Add deny address");
 
   new_entry->addr = *adr;
@@ -480,9 +478,7 @@ olsr_validate_address(const union olsr_ip_addr *adr)
 
   for (deny_entry = deny_entries; deny_entry != NULL; deny_entry = deny_entry->next) {
     if(ipequal(adr, &deny_entry->addr))        {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(1, "Validation of address %s failed!\n", olsr_ip_to_string(&buf, adr));
       return OLSR_FALSE;
     }
index b5dec7b..c79091e 100644 (file)
@@ -339,9 +339,7 @@ olsr_forward_message(union olsr_message *m,
   if(olsr_lookup_mprs_set(src) == NULL)
     {
 #ifdef DEBUG
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(5, "Forward - sender %s not MPR selector\n", olsr_ip_to_string(&buf, src));
 #endif
       return 0;
index 715bc17..ca204fd 100644 (file)
@@ -461,9 +461,7 @@ olsr_calculate_routing_table (void)
        * does not contain a next-hop.
        */
       if (tc != tc_myself) {
-#ifndef NODEBUG
         struct ipaddr_str buf;
-#endif
         OLSR_PRINTF(2, "SPF: %s no next-hop\n", olsr_ip_to_string(&buf, &tc->addr));
       }
 #endif
index 24bb1cb..0780632 100644 (file)
@@ -118,7 +118,7 @@ olsr_build_hello_packet(struct hello_message *message, struct interface *outif)
 
   /* Walk all links of this interface */
   OLSR_FOR_ALL_LINK_ENTRIES(links) {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
     struct ipaddr_str buf;
 #endif
     int lnk = lookup_link_status(links);
@@ -202,7 +202,7 @@ olsr_build_hello_packet(struct hello_message *message, struct interface *outif)
   if (ifnet != NULL && ifnet->int_next != NULL)
     OLSR_FOR_ALL_NBR_ENTRIES(neighbor) {
 
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
     struct ipaddr_str buf;
 #endif
     /* Check that the neighbor is not added yet */
index d916a26..49aabc3 100644 (file)
@@ -327,9 +327,7 @@ void parse_packet(struct olsr *olsr, int size, struct interface *in_if, union ol
       /* IPv4 */
       if (m->v4.ttl <= 0 && olsr_cnf->lq_fish == 0)
       {
-#ifndef NODEBUG
         struct ipaddr_str buf;
-#endif
         OLSR_PRINTF(2, "Dropping packet type %d from neigh %s with TTL 0\n",
             m->v4.olsr_msgtype,
             olsr_ip_to_string(&buf, from_addr));
@@ -341,9 +339,7 @@ void parse_packet(struct olsr *olsr, int size, struct interface *in_if, union ol
       /* IPv6 */
       if (m->v6.ttl <= 0 && olsr_cnf->lq_fish == 0)
       {
-#ifndef NODEBUG
         struct ipaddr_str buf;
-#endif
         OLSR_PRINTF(2, "Dropping packet type %d from %s with TTL 0\n",
             m->v4.olsr_msgtype,
             olsr_ip_to_string(&buf, from_addr));
@@ -363,7 +359,7 @@ void parse_packet(struct olsr *olsr, int size, struct interface *in_if, union ol
     /* Should be the same for IPv4 and IPv6 */
     if (ipequal((union olsr_ip_addr *)&m->v4.originator, &olsr_cnf->main_addr)
         || !olsr_validate_address((union olsr_ip_addr *)&m->v4.originator)) {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
       struct ipaddr_str buf;
 #endif
 #ifdef DEBUG
@@ -404,9 +400,7 @@ void parse_packet(struct olsr *olsr, int size, struct interface *in_if, union ol
     
     /* UNKNOWN PACKETTYPE */
     if (processed == 0) {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       unk_chgestruct(&unkpacket, m);
       
       OLSR_PRINTF(3, "Unknown type: %d, size %d, from %s\n",
@@ -444,7 +438,7 @@ void olsr_input(int fd) {
   cpu_overload_exit = 0;
   
   for (;;) {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
     struct ipaddr_str buf;
 #endif
     /* sockaddr_in6 is bigger than sockaddr !!!! */
index 0c41728..2367ea0 100644 (file)
@@ -82,7 +82,7 @@ process_message_neighbors(struct neighbor_entry *neighbor, const struct hello_me
       message_neighbors != NULL;
       message_neighbors = message_neighbors->next)
     {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
       struct ipaddr_str buf;
 #endif
       union olsr_ip_addr      *neigh_addr;
@@ -500,9 +500,7 @@ olsr_hello_tap(struct hello_message *message,
   /* Check willingness */
   if(neighbor->willingness != message->willingness)
     {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(1, "Willingness for %s changed from %d to %d - UPDATING\n", 
                  olsr_ip_to_string(&buf, &neighbor->neighbor_main_addr),
                  neighbor->willingness,
@@ -542,7 +540,7 @@ olsr_process_received_mid(union olsr_message *m,
                           struct interface *in_if __attribute__((unused)),
                           union olsr_ip_addr *from_addr)
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str buf;
 #endif
   struct mid_alias *tmp_adr;
@@ -567,9 +565,7 @@ olsr_process_received_mid(union olsr_message *m,
      */
 
     if(check_neighbor_link(from_addr) != SYM_LINK) {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(2, "Received MID from NON SYM neighbor %s\n", olsr_ip_to_string(&buf, from_addr));
       olsr_free_mid_packet(&message);
       return;
@@ -580,9 +576,7 @@ olsr_process_received_mid(union olsr_message *m,
 
     while (tmp_adr) {
       if (!mid_lookup_main_addr(&tmp_adr->alias_addr)){
-#ifndef NODEBUG
         struct ipaddr_str buf;
-#endif
         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);
@@ -673,9 +667,7 @@ olsr_process_received_hna(union olsr_message *m,
      *      message MUST be discarded.
      */
     if (check_neighbor_link(from_addr) != SYM_LINK) {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(2, "Received HNA from NON SYM neighbor %s\n", olsr_ip_to_string(&buf, from_addr));
       return;
     }
index 8aa09b0..5b6a327 100644 (file)
@@ -105,9 +105,7 @@ mid_chgestruct(struct mid_message *mmsg, const union olsr_message *m)
       
       if(olsr_cnf->debug_level > 1)
        {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
          OLSR_PRINTF(3, "Alias list for %s: ", olsr_ip_to_string(&buf, &mmsg->mid_origaddr));
          OLSR_PRINTF(3, "%s", olsr_ip_to_string(&buf, &mmsg->addr));
          alias_tmp = mmsg->mid_addr;
@@ -157,9 +155,7 @@ mid_chgestruct(struct mid_message *mmsg, const union olsr_message *m)
 
       if(olsr_cnf->debug_level > 1)
        {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
          OLSR_PRINTF(3, "Alias list for %s", ip6_to_string(&buf, &mmsg->mid_origaddr.v6));
          OLSR_PRINTF(3, "%s", ip6_to_string(&buf, &mmsg->addr.v6));
 
index c70bc51..7aaad04 100644 (file)
@@ -524,7 +524,7 @@ struct rt_path *
 olsr_insert_routing_table(union olsr_ip_addr *dst, int plen,
                           union olsr_ip_addr *originator, int origin)
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str dstbuf, origbuf;
 #endif
   struct tc_entry *tc;
@@ -586,7 +586,7 @@ void
 olsr_delete_routing_table(union olsr_ip_addr *dst, int plen,
                           union olsr_ip_addr *originator)
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str dstbuf, origbuf;
 #endif
 
index 81c1ae9..1929993 100644 (file)
@@ -132,7 +132,7 @@ olsr_seq_inrange_high(int beg, int end, olsr_u16_t seq)
 static struct tc_entry *
 olsr_add_tc_entry(union olsr_ip_addr *adr)
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str buf;
 #endif
   struct tc_entry *tc;
@@ -430,7 +430,7 @@ struct tc_edge_entry *
 olsr_add_tc_edge_entry(struct tc_entry *tc, union olsr_ip_addr *addr,
                       olsr_u16_t ansn)
 {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
   struct ipaddr_str buf;
 #endif
   struct tc_entry *tc_neighbor;
@@ -797,9 +797,7 @@ olsr_input_tc(union olsr_message *msg,
              struct interface *input_if __attribute__ ((unused)),
              union olsr_ip_addr *from_addr)
 {
-#ifndef NODEBUG
   struct ipaddr_str buf;
-#endif
   olsr_u16_t size, msg_seq, ansn;
   olsr_u8_t type, ttl, msg_hops, lower_border, upper_border;
   olsr_reltime vtime;
index bfcc05b..9d5561f 100644 (file)
@@ -259,7 +259,7 @@ chk_if_changed(struct olsr_if *iface)
   /* IP version 6 */
   if(olsr_cnf->ip_version == AF_INET6)
     {
-#if !defined(NODEBUG) && defined(DEBUG)
+#ifdef DEBUG
       struct ipaddr_str buf;
 #endif
       /* Get interface address */
@@ -281,9 +281,7 @@ chk_if_changed(struct olsr_if *iface)
 
       if(memcmp(&tmp_saddr6.sin6_addr, &ifp->int6_addr.sin6_addr, olsr_cnf->ipsize) != 0)
        {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
          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));
@@ -365,9 +363,7 @@ chk_if_changed(struct olsr_if *iface)
                &((struct sockaddr_in *)&ifr.ifr_netmask)->sin_addr.s_addr, 
                olsr_cnf->ipsize) != 0)
        {
-#ifndef NODEBUG
           struct ipaddr_str buf;
-#endif
          /* 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));
@@ -393,9 +389,7 @@ 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)
            {
-#ifndef NODEBUG
               struct ipaddr_str buf;
-#endif
              /* 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));
@@ -744,9 +738,7 @@ chk_if_up(struct olsr_if *iface, int debuglvl __attribute__((unused)))
   if(olsr_cnf->ip_version == AF_INET6)
     {
       /* Get interface address */
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       if(get_ipv6_address(ifr.ifr_name, &ifs.int6_addr, iface->cnf->ipv6_addrtype) <= 0)
        {
          if(iface->cnf->ipv6_addrtype == IPV6_ADDR_SITELOCAL)
@@ -857,18 +849,14 @@ chk_if_up(struct olsr_if *iface, int debuglvl __attribute__((unused)))
 
   if(olsr_cnf->ip_version == AF_INET)
     {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(1, "\tAddress:%s\n", ip4_to_string(&buf, ifs.int_addr.sin_addr));
       OLSR_PRINTF(1, "\tNetmask:%s\n", ip4_to_string(&buf, ifs.int_netmask.sin_addr));
       OLSR_PRINTF(1, "\tBroadcast address:%s\n", ip4_to_string(&buf, ifs.int_broadaddr.sin_addr));
     }
   else
     {
-#ifndef NODEBUG
       struct ipaddr_str buf;
-#endif
       OLSR_PRINTF(1, "\tAddress: %s\n", ip6_to_string(&buf, &ifs.int6_addr.sin6_addr));
       OLSR_PRINTF(1, "\tMulticast: %s\n", ip6_to_string(&buf, &ifs.int6_multaddr.sin6_addr));
     }