remove ipequal functions, they are often mixed up with ipcmp()
authorHenning Rogge <hrogge@googlemail.com>
Mon, 16 Mar 2009 19:46:42 +0000 (20:46 +0100)
committerHenning Rogge <hrogge@googlemail.com>
Mon, 16 Mar 2009 19:46:42 +0000 (20:46 +0100)
24 files changed:
lib/bmf/src/Bmf.c
lib/bmf/src/NetworkInterfaces.c
lib/dot_draw/src/olsrd_dot_draw.c
lib/nameservice/src/mapwrite.c
lib/nameservice/src/nameservice.c
src/bsd/kernel_routes.c
src/build_msg.c
src/interfaces.c
src/ipcalc.h
src/link_set.c
src/linux/kernel_routes.c
src/lq_packet.c
src/mpr_selector_set.c
src/neighbor_table.c
src/olsr_ip_prefix_list.c
src/parser.c
src/process_package.c
src/process_routes.c
src/routing_table.c
src/routing_table.h
src/tc_set.c
src/two_hop_neighbor_table.c
src/unix/ifnet.c
src/win32/ifnet.c

index b93bde8..0d8fae9 100644 (file)
@@ -299,7 +299,7 @@ static void BmfPacketCaptured(
 
   /* Only forward multicast packets. If configured, also forward local broadcast packets */
   if (IsMulticast(&dst) ||
-      (EnableLocalBroadcast != 0 && olsr_ipequal(&dst, &intf->broadAddr)))
+      (EnableLocalBroadcast != 0 && olsr_ipcmp(&dst, &intf->broadAddr) == 0))
   {
     /* continue */
   }
@@ -938,7 +938,7 @@ static void BmfTunPacketCaptured(unsigned char* encapsulationUdpData)
 
   /* Only forward multicast packets. If configured, also forward local broadcast packets */
   if (IsMulticast(&dstIp) ||
-      (EnableLocalBroadcast != 0 && olsr_ipequal(&dstIp, &broadAddr)))
+      (EnableLocalBroadcast != 0 && olsr_ipcmp(&dstIp, &broadAddr) == 0))
   {
     /* continue */
   }
index 912e4bd..6fb16b0 100644 (file)
@@ -448,7 +448,7 @@ void FindNeighbors(
     float currEtx;
 
     /* Consider only links from the specified interface */
-    if (! olsr_ipequal(&intf->intAddr, &walker->local_iface_addr))
+    if (olsr_ipcmp(&intf->intAddr, &walker->local_iface_addr) != 0)
     {
       continue; /* for */
     }
@@ -464,7 +464,7 @@ void FindNeighbors(
 
     /* 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 && olsr_ipequal(neighborMainIp, MainAddressOf(source)))
+    if (source != NULL && olsr_ipcmp(neighborMainIp, MainAddressOf(source)) == 0)
     {
       OLSR_PRINTF(
         9,
@@ -476,7 +476,7 @@ void FindNeighbors(
     }
 
     /* Rely on short-circuit boolean evaluation */
-    if (forwardedBy != NULL && olsr_ipequal(neighborMainIp, MainAddressOf(forwardedBy)))
+    if (forwardedBy != NULL && olsr_ipcmp(neighborMainIp, MainAddressOf(forwardedBy)) == 0)
     {
       OLSR_PRINTF(
         9,
@@ -488,7 +488,7 @@ void FindNeighbors(
     }
 
     /* Rely on short-circuit boolean evaluation */
-    if (forwardedTo != NULL && olsr_ipequal(neighborMainIp, MainAddressOf(forwardedTo)))
+    if (forwardedTo != NULL && olsr_ipcmp(neighborMainIp, MainAddressOf(forwardedTo)) == 0)
     {
       OLSR_PRINTF(
         9,
index 0ac312e..bf2e75d 100644 (file)
@@ -240,7 +240,7 @@ ipc_action(int fd __attribute__((unused)), void *data __attribute__((unused)), u
     return;
   }
 #ifndef _WRS_KERNEL
-  if (!ip4equal(&pin.sin_addr, &ipc_accept_ip.v4)) {
+  if (ip4cmp(&pin.sin_addr, &ipc_accept_ip.v4) != 0) {
     OLSR_PRINTF(0, "Front end-connection from foreign host (%s) not allowed!\n", inet_ntoa(pin.sin_addr));
     CLOSESOCKET(ipc_connection);
     return;
index 5bf2195..8872ab8 100644 (file)
@@ -68,7 +68,7 @@ static char* lookup_position_latlon(union olsr_ip_addr *ip)
   struct db_entry *entry;
   struct list_node *list_head, *list_node;
 
-  if (olsr_ipequal(ip, &olsr_cnf->router_id)) {
+  if (olsr_ipcmp(ip, &olsr_cnf->router_id) == 0) {
     return my_latlon_str;
   }
 
@@ -79,7 +79,7 @@ static char* lookup_position_latlon(union olsr_ip_addr *ip)
 
           entry = list2db(list_node);
 
-      if (entry->names && olsr_ipequal(&entry->originator, ip)) {
+      if (entry->names && olsr_ipcmp(&entry->originator, ip) == 0) {
         return entry->names->name;
       }
     }
@@ -107,7 +107,7 @@ void mapwrite_work(FILE* fmap)
     {
       if (olsr_cnf->ip_version == AF_INET)
       {
-        if (!(ip4equal((struct in_addr *)&olsr_cnf->router_id, &ifs->interf->int_addr.sin_addr)))
+        if (ip4cmp((struct in_addr *)&olsr_cnf->router_id, &ifs->interf->int_addr.sin_addr) != 0)
         {
           if (0 > fprintf(fmap, "Mid('%s','%s');\n",
             olsr_ip_to_string(&strbuf1, &olsr_cnf->router_id),
@@ -117,7 +117,7 @@ void mapwrite_work(FILE* fmap)
           }
         }
       }
-      else if (!(ip6equal((struct in6_addr *)&olsr_cnf->router_id, &ifs->interf->int6_addr.sin6_addr)))
+      else if (ip6cmp((struct in6_addr *)&olsr_cnf->router_id, &ifs->interf->int6_addr.sin6_addr) != 0)
       {
         if (0 > fprintf(fmap, "Mid('%s','%s');\n",
           olsr_ip_to_string(&strbuf1, &olsr_cnf->router_id),
index dde0d56..298d708 100644 (file)
@@ -409,7 +409,7 @@ name_init(void)
        //for service
 
        for (name = my_names; name != NULL; name = name->next) {
-               if (olsr_ipequal(&name->ip, &ipz)) {
+               if (olsr_ipcmp(&name->ip, &ipz) == 0) {
                        OLSR_PRINTF(2, "NAME PLUGIN: insert main addr for name %s \n", name->name);
                        name->ip = olsr_cnf->router_id;
                }
@@ -706,7 +706,7 @@ olsr_parser(union olsr_message *m,
 
        /* Check if message originated from this node.
        If so - back off */
-       if(olsr_ipequal(&originator, &olsr_cnf->router_id))
+       if(olsr_ipcmp(&originator, &olsr_cnf->router_id) == 0)
                return false;
 
        /* Check that the neighbor this message was received from is symmetric.
@@ -863,7 +863,7 @@ decap_namemsg(struct name *from_packet, struct name_entry **to, bool *this_table
                        OLSR_PRINTF(4, "NAME PLUGIN: received name or service entry %s (%s) already in hash table\n",
                                name, olsr_ip_to_string(&strbuf, &already_saved_name_entries->ip));
                        return;
-               } else if (type_of_from_packet==NAME_FORWARDER && olsr_ipequal(&already_saved_name_entries->ip, &from_packet->ip) ) {
+               } else if (type_of_from_packet==NAME_FORWARDER && olsr_ipcmp(&already_saved_name_entries->ip, &from_packet->ip) == 0) {
                        OLSR_PRINTF(4, "NAME PLUGIN: received forwarder entry %s (%s) already in hash table\n",
                                name, olsr_ip_to_string(&strbuf, &already_saved_name_entries->ip));
                        return;
@@ -880,7 +880,7 @@ decap_namemsg(struct name *from_packet, struct name_entry **to, bool *this_table
                                *this_table_changed = true;
                                olsr_start_write_file_timer();
                        }
-                       if (!olsr_ipequal(&already_saved_name_entries->ip, &from_packet->ip))
+                       if (olsr_ipcmp(&already_saved_name_entries->ip, &from_packet->ip) != 0)
                        {
                                struct ipaddr_str strbuf2, strbuf3;
                                OLSR_PRINTF(4, "NAME PLUGIN: updating ip %s -> %s (%s)\n",
@@ -1008,7 +1008,7 @@ insert_new_name_in_list(union olsr_ip_addr *originator,
 
                entry = list2db(list_node);
 
-               if (olsr_ipequal(originator, &entry->originator)) {
+               if (olsr_ipcmp(originator, &entry->originator) == 0) {
                        struct ipaddr_str strbuf;
                        // found
                        OLSR_PRINTF(4, "NAME PLUGIN: found entry for (%s) in its hash table\n",
@@ -1524,7 +1524,7 @@ allowed_ip(const union olsr_ip_addr *addr)
 
        OLSR_FOR_ALL_INTERFACES(iface) {
                OLSR_PRINTF(6, "interface %s\n", olsr_ip_to_string(&strbuf, &iface->ip_addr));
-               if (olsr_ipequal(&iface->ip_addr, addr)) {
+               if (olsr_ipcmp(&iface->ip_addr, addr) == 0) {
                        OLSR_PRINTF(6, "MATCHED\n");
                        return true;
                }
@@ -1555,7 +1555,7 @@ allowed_ip(const union olsr_ip_addr *addr)
                        for (i = 0; i < sizeof(tmp_ip.v6.s6_addr); i++) {
                                tmp_ip.v6.s6_addr[i] = addr->v6.s6_addr[i] & tmp_msk.v6.s6_addr[i];
                        }
-                       if (olsr_ipequal(&tmp_ip, &hna->net.prefix)) {
+                       if (olsr_ipcmp(&tmp_ip, &hna->net.prefix) == 0) {
                                OLSR_PRINTF(6, "MATCHED\n");
                                return true;
                        }
@@ -1722,7 +1722,7 @@ lookup_name_latlon(union olsr_ip_addr *ip)
                        entry = list2db(list_node);
 
                        for (name = entry->names; name != NULL; name = name->next) {
-                               if (olsr_ipequal(&name->ip, ip)) return name->name;
+                               if (olsr_ipcmp(&name->ip, ip) == 0) return name->name;
                        }
                }
        }
index d71b5ee..ae3612c 100644 (file)
@@ -79,7 +79,7 @@ olsr_rt_flags(const struct rt_entry *rt)
 
   nh = olsr_get_nh(rt);
 
-  if (!olsr_ipequal(&rt->rt_dst.prefix, &nh->gateway)) {
+  if (olsr_ipcmp(&rt->rt_dst.prefix, &nh->gateway) != 0) {
     flags |= RTF_GATEWAY;
   }
 
index 3e1084d..2760b6e 100644 (file)
@@ -196,7 +196,7 @@ serialize_mid4(struct interface *ifp)
 
   /* Don't add the main address... it's already there */
   OLSR_FOR_ALL_INTERFACES(ifs) {
-      if(!olsr_ipequal(&olsr_cnf->router_id, &ifs->ip_addr))
+      if(olsr_ipcmp(&olsr_cnf->router_id, &ifs->ip_addr) != 0)
        {
          if((curr_size + olsr_cnf->ipsize) > remainsize)
            {
@@ -289,7 +289,7 @@ serialize_mid6(struct interface *ifp)
 
   /* Don't add the main address... it's already there */
   OLSR_FOR_ALL_INTERFACES(ifs) {
-      if(!olsr_ipequal(&olsr_cnf->router_id, &ifs->ip_addr))
+      if(olsr_ipcmp(&olsr_cnf->router_id, &ifs->ip_addr) != 0)
        {
          if((curr_size + olsr_cnf->ipsize) > remainsize)
            {
index 57c739d..d8dbf50 100644 (file)
@@ -183,7 +183,7 @@ remove_interface(struct interface **pinterf)
   net_remove_buffer(ifp);
 
   /* Check main addr */
-  if (!olsr_cnf->fixed_origaddr && olsr_ipequal(&olsr_cnf->router_id, &ifp->ip_addr)) {
+  if (!olsr_cnf->fixed_origaddr && olsr_ipcmp(&olsr_cnf->router_id, &ifp->ip_addr) == 0) {
     if (list_is_empty(&interface_head)) {
       /* No more interfaces */
       memset(&olsr_cnf->router_id, 0, olsr_cnf->ipsize);
@@ -257,8 +257,8 @@ if_ifwithaddr(const union olsr_ip_addr *addr)
 
     /* IPv4 */
     OLSR_FOR_ALL_INTERFACES(ifp) {
-      if (ip4equal(&ifp->int_addr.sin_addr, &addr->v4)) {
-       return ifp;
+      if (ip4cmp(&ifp->int_addr.sin_addr, &addr->v4) == 0) {
+        return ifp;
       }
     } OLSR_FOR_ALL_INTERFACES_END(ifp);
 
@@ -266,8 +266,8 @@ if_ifwithaddr(const union olsr_ip_addr *addr)
 
     /* IPv6 */
     OLSR_FOR_ALL_INTERFACES(ifp) {
-      if (ip6equal(&ifp->int6_addr.sin6_addr, &addr->v6)) {
-       return ifp;
+      if (ip6cmp(&ifp->int6_addr.sin6_addr, &addr->v6) == 0) {
+        return ifp;
       }
     } OLSR_FOR_ALL_INTERFACES_END(ifp);
   }
index 1eb491c..acda1da 100644 (file)
@@ -71,16 +71,12 @@ struct ipprefix_str {
  * Macros for comparing and copying IP addresses
  */
 static INLINE int ip4cmp(const struct in_addr *a, const struct in_addr *b) { return a->s_addr > b->s_addr ? +1 : a->s_addr < b->s_addr ? -1 : 0; }
-static INLINE int ip4equal(const struct in_addr *a, const struct in_addr *b) { return a->s_addr == b->s_addr; }
 
 static INLINE int ip6cmp(const struct in6_addr *a, const struct in6_addr *b) { return memcmp(a, b, sizeof(*a)); }
-static INLINE int ip6equal(const struct in6_addr *a, const struct in6_addr *b) { return ip6cmp(a, b) == 0; }
 
 static INLINE int ipcmp(int ip_version, const union olsr_ip_addr *a, const union olsr_ip_addr *b) { return ip_version == AF_INET ? ip4cmp(&a->v4, &b->v4) : ip6cmp(&a->v6, &b->v6); }
-static INLINE int ipequal(int ip_version, const union olsr_ip_addr *a, const union olsr_ip_addr *b) { return ip_version == AF_INET ? ip4equal(&a->v4, &b->v4) : ip6equal(&a->v6, &b->v6); }
 
 static INLINE int olsr_ipcmp(const union olsr_ip_addr *a, const union olsr_ip_addr *b) { return ipcmp(olsr_cnf->ip_version, a, b); }
-static INLINE int olsr_ipequal(const union olsr_ip_addr *a, const union olsr_ip_addr *b) { return ipequal(olsr_cnf->ip_version, a, b); }
 
 /* Do not use this - this is as evil as the COPY_IP() macro was and only used in
  * source which also needs cleanups.
index a798093..e4a43f0 100644 (file)
@@ -201,7 +201,7 @@ get_best_link_to_neighbor(const union olsr_ip_addr *remote)
   OLSR_FOR_ALL_LINK_ENTRIES(walker) {
 
     /* if this is not a link to the neighour in question, skip */
-    if (!olsr_ipequal(&walker->neighbor->neighbor_main_addr, main_addr))
+    if (olsr_ipcmp(&walker->neighbor->neighbor_main_addr, main_addr) != 0)
       continue;
 
     /* get the link cost */
@@ -213,7 +213,7 @@ get_best_link_to_neighbor(const union olsr_ip_addr *remote)
      */
     if ((tmp_lc < curr_lcost) ||
         ((tmp_lc == curr_lcost) &&
-         olsr_ipequal(&walker->local_iface_addr, remote))) {
+         olsr_ipcmp(&walker->local_iface_addr, remote) == 0)) {
 
       /* memorize the link quality */
       curr_lcost = tmp_lc;
@@ -258,8 +258,8 @@ set_loss_link_multiplier(struct link_entry *entry)
      * use the default multiplier only if there isn't any entry that
      * has a matching IP address.
      */
-    if ((val == 0 && olsr_ipequal(&mult->addr, &all_zero)) ||
-       olsr_ipequal(&mult->addr, &entry->neighbor_iface_addr)) {
+    if ((val == 0 && olsr_ipcmp(&mult->addr, &all_zero) == 0) ||
+        olsr_ipcmp(&mult->addr, &entry->neighbor_iface_addr) == 0) {
       val = mult->value;
     }
   }
@@ -561,7 +561,7 @@ check_neighbor_link(const union olsr_ip_addr *int_addr)
   struct link_entry *link;
 
   OLSR_FOR_ALL_LINK_ENTRIES(link) {
-    if (olsr_ipequal(int_addr, &link->neighbor_iface_addr)) {
+    if (olsr_ipcmp(int_addr, &link->neighbor_iface_addr) == 0) {
       return lookup_link_status(link);
     }
   } OLSR_FOR_ALL_LINK_ENTRIES_END(link);
@@ -585,12 +585,12 @@ lookup_link_entry(const union olsr_ip_addr *remote,
   struct link_entry *link;
 
   OLSR_FOR_ALL_LINK_ENTRIES(link) {
-    if (olsr_ipequal(remote, &link->neighbor_iface_addr) &&
+    if (olsr_ipcmp(remote, &link->neighbor_iface_addr) == 0 &&
        (link->if_name ? !strcmp(link->if_name, local->int_name)
-        : olsr_ipequal(&local->ip_addr, &link->local_iface_addr)))
+        : olsr_ipcmp(&local->ip_addr, &link->local_iface_addr) == 0))
     {
       /* check the remote-main address only if there is one given */
-      if (NULL != remote_main && !olsr_ipequal(remote_main, &link->neighbor->neighbor_main_addr))
+      if (NULL != remote_main && olsr_ipcmp(remote_main, &link->neighbor->neighbor_main_addr) != 0)
       {
         /* Neighbor has changed it's main_addr, update */
 #if !defined REMOVE_DEBUG
@@ -724,7 +724,7 @@ check_link_status(const struct lq_hello_message *message,
      * Note: If a neigh has 2 cards we can reach, the neigh
      * will send a Hello with the same IP mentined twice
      */
-    if (olsr_ipequal(&neighbors->addr, &in_if->ip_addr)) {
+    if (olsr_ipcmp(&neighbors->addr, &in_if->ip_addr) == 0) {
       ret = neighbors->link_type;
       if (SYM_LINK == ret) {
        break;
index bf4a716..5741aff 100644 (file)
@@ -146,13 +146,13 @@ static int olsr_netlink_route(const struct rt_entry *rt, uint8_t family, uint8_t
 
   if (NULL != nexthop->interface) {
     if (AF_INET == family) {
-      if (!ip4equal(&rt->rt_dst.prefix.v4, &nexthop->gateway.v4)) {
+      if (ip4cmp(&rt->rt_dst.prefix.v4, &nexthop->gateway.v4) != 0) {
         olsr_netlink_addreq(&req.n, sizeof(req), RTA_GATEWAY, &nexthop->gateway.v4, sizeof(nexthop->gateway.v4));
         req.r.rtm_scope = RT_SCOPE_UNIVERSE;
       }
       olsr_netlink_addreq(&req.n, sizeof(req), RTA_DST, &rt->rt_dst.prefix.v4, sizeof(rt->rt_dst.prefix.v4));
     } else {
-      if (!ip6equal(&rt->rt_dst.prefix.v6, &nexthop->gateway.v6)) {
+      if (ip6cmp(&rt->rt_dst.prefix.v6, &nexthop->gateway.v6) != 0) {
         olsr_netlink_addreq(&req.n, sizeof(req), RTA_GATEWAY, &nexthop->gateway.v6, sizeof(nexthop->gateway.v6));
         req.r.rtm_scope = RT_SCOPE_UNIVERSE;
       }
index 216904a..7ec3384 100644 (file)
@@ -94,7 +94,7 @@ create_lq_hello(struct lq_hello_message *lq_hello, struct interface *outif)
     struct lq_hello_neighbor *neigh = olsr_malloc_lq_hello_neighbor();
 
     // a) this neighbor interface IS NOT visible via the output interface
-    if(!olsr_ipequal(&walker->local_iface_addr, &outif->ip_addr))
+    if(olsr_ipcmp(&walker->local_iface_addr, &outif->ip_addr) != 0)
       neigh->link_type = UNSPEC_LINK;
 
     // b) this neighbor interface IS visible via the output interface
index 0457495..3cd6eff 100644 (file)
@@ -101,7 +101,7 @@ olsr_lookup_mprs_set(const union olsr_ip_addr *addr)
     return NULL;
   }
   FOR_ALL_MPRS_ENTRIES(mprs) {
-    if (olsr_ipequal(&mprs->MS_main_addr, addr)) {
+    if (olsr_ipcmp(&mprs->MS_main_addr, addr) == 0) {
       return mprs;
     }
   } FOR_ALL_MPRS_ENTRIES_END(mprs);
index 8462b10..8e1fad1 100644 (file)
@@ -122,7 +122,7 @@ olsr_delete_neighbor_2_pointer(struct neighbor_entry *neighbor, union olsr_ip_ad
   nbr2_list = neighbor->neighbor_2_list.next;
 
   while (nbr2_list != &neighbor->neighbor_2_list) {
-    if (olsr_ipequal(&nbr2_list->neighbor_2->neighbor_2_addr, address)) {
+    if (olsr_ipcmp(&nbr2_list->neighbor_2->neighbor_2_addr, address) == 0) {
       olsr_del_nbr2_list(nbr2_list);
       return 1;
     }
@@ -153,7 +153,7 @@ olsr_lookup_my_neighbors(const struct neighbor_entry *neighbor, const union olsr
       entry = entry->next)
     {
 
-      if(olsr_ipequal(&entry->neighbor_2->neighbor_2_addr, neighbor_main_address))
+      if(olsr_ipcmp(&entry->neighbor_2->neighbor_2_addr, neighbor_main_address) == 0)
        return entry;
 
     }
@@ -193,7 +193,7 @@ olsr_delete_neighbor_table(const union olsr_ip_addr *neighbor_addr)
    */
   while(entry != &neighbortable[hash])
     {
-      if(olsr_ipequal(&entry->neighbor_main_addr, neighbor_addr))
+      if(olsr_ipcmp(&entry->neighbor_main_addr, neighbor_addr) == 0)
         break;
 
       entry = entry->next;
@@ -253,7 +253,7 @@ olsr_insert_neighbor_table(const union olsr_ip_addr *main_addr)
       new_neigh != &neighbortable[hash];
       new_neigh = new_neigh->next)
     {
-      if(olsr_ipequal(&new_neigh->neighbor_main_addr, main_addr))
+      if(olsr_ipcmp(&new_neigh->neighbor_main_addr, main_addr) == 0)
         return new_neigh;
     }
 
@@ -320,7 +320,7 @@ olsr_lookup_neighbor_table_alias(const union olsr_ip_addr *dst)
       entry != &neighbortable[hash];
       entry = entry->next)
     {
-      if(olsr_ipequal(&entry->neighbor_main_addr, dst))
+      if(olsr_ipcmp(&entry->neighbor_main_addr, dst) == 0)
         return entry;
     }
 
index 120a5d2..e88c17d 100644 (file)
@@ -73,7 +73,7 @@ ip_prefix_list_remove(struct list_node *ip_prefix_head, const union olsr_ip_addr
   struct ip_prefix_entry *h;
 
   OLSR_FOR_ALL_IPPREFIX_ENTRIES(ip_prefix_head, h) {
-    if (ipequal(ip_version, net, &h->net.prefix) && h->net.prefix_len == prefix_len) {
+    if (ipcmp(ip_version, net, &h->net.prefix) == 0 && h->net.prefix_len == prefix_len) {
       list_remove(&h->node);
       free(h);
       return 1;
@@ -89,7 +89,7 @@ ip_prefix_list_find(struct list_node *ip_prefix_head, const union olsr_ip_addr *
   struct ip_prefix_entry *h;
 
   OLSR_FOR_ALL_IPPREFIX_ENTRIES(ip_prefix_head, h) {
-    if (prefix_len == h->net.prefix_len && ipequal(ip_version, net, &h->net.prefix)) {
+    if (prefix_len == h->net.prefix_len && ipcmp(ip_version, net, &h->net.prefix) == 0) {
       return h;
     }
   }
index 8cbb452..8e50e4c 100644 (file)
@@ -295,7 +295,7 @@ static void parse_packet(struct olsr *olsr, int size, struct interface *in_if, u
      */
 
     /* Should be the same for IPv4 and IPv6 */
-    if (olsr_ipequal((union olsr_ip_addr *)&m->v4.originator, &olsr_cnf->router_id)
+    if (olsr_ipcmp((union olsr_ip_addr *)&m->v4.originator, &olsr_cnf->router_id) == 0
         || !olsr_validate_address((union olsr_ip_addr *)&m->v4.originator)) {
       OLSR_INFO(LOG_PACKET_PARSING, "Not processing message originating from %s!\n",
                   olsr_ip_to_string(&buf,(union olsr_ip_addr *)&m->v4.originator));
index 23113e4..d47b8cc 100644 (file)
@@ -288,8 +288,8 @@ lookup_mpr_status(const struct lq_hello_message *message,
 
   for (neighbors = message->neigh; neighbors; neighbors = neighbors->next) {
     if (olsr_cnf->ip_version == AF_INET
-        ? ip4equal(&neighbors->addr.v4, &in_if->ip_addr.v4)
-        : ip6equal(&neighbors->addr.v6, &in_if->int6_addr.sin6_addr)) {
+        ? ip4cmp(&neighbors->addr.v4, &in_if->ip_addr.v4) == 0
+        : ip6cmp(&neighbors->addr.v6, &in_if->int6_addr.sin6_addr) == 0) {
       return neighbors->link_type == SYM_LINK && neighbors->neigh_type == MPR_NEIGH ? true : false;
     }
   }
@@ -386,7 +386,7 @@ hello_tap(struct lq_hello_message *message,
 
   /* find the input interface in the list of neighbor interfaces */
   for (walker = message->neigh; walker != NULL; walker = walker->next) {
-    if (olsr_ipequal(&walker->addr, &in_if->ip_addr)) {
+    if (olsr_ipcmp(&walker->addr, &in_if->ip_addr) == 0) {
       break;
     }
   }
index b44ce86..2d2a212 100644 (file)
@@ -111,7 +111,7 @@ olsr_enqueue_rt(struct list_node *head_node, struct rt_entry *rt)
    */
   nh = olsr_get_nh(rt);
 
-  if (olsr_ipequal(&rt->rt_dst.prefix, &nh->gateway)) {
+  if (olsr_ipcmp(&rt->rt_dst.prefix, &nh->gateway) == 0) {
     list_add_after(head_node, &rt->rt_change_node);
   } else {
     list_add_before(head_node, &rt->rt_change_node);
index 31ac883..9fb0f6c 100644 (file)
@@ -130,7 +130,7 @@ avl_comp_ipv6_prefix_origin (const void *prefix1, const void *prefix2)
   const struct olsr_ip_prefix *pfx2 = prefix2;
 
   /* prefix */
-  diff = ip6equal(&pfx1->prefix.v6, &pfx2->prefix.v6);
+  diff = ip6cmp(&pfx1->prefix.v6, &pfx2->prefix.v6) == 0;
   if (diff) {
     return diff;
   }
@@ -192,7 +192,7 @@ avl_comp_ipv6_prefix (const void *prefix1, const void *prefix2)
   const struct olsr_ip_prefix *pfx2 = prefix2;
 
   /* prefix */
-  diff = ip6equal(&pfx1->prefix.v6, &pfx2->prefix.v6);
+  diff = ip6cmp(&pfx1->prefix.v6, &pfx2->prefix.v6) == 0;
   if (diff) {
     return diff;
   }
@@ -244,7 +244,7 @@ avl_comp_ipv6_addr_origin (const void *prefix1, const void *prefix2)
   const struct olsr_ip_prefix *pfx2 = prefix2;
 
   /* prefix */
-  diff = ip6equal(&pfx1->prefix.v6, &pfx2->prefix.v6);
+  diff = ip6cmp(&pfx1->prefix.v6, &pfx2->prefix.v6) == 0;
   if (diff) {
     return diff;
   }
index 122f352..8e64244 100644 (file)
@@ -203,7 +203,7 @@ void olsr_rt_best(struct rt_entry *);
  * Check if there is an interface or gateway change.
  */
 static INLINE bool olsr_nh_change(const struct rt_nexthop *nh1, const struct rt_nexthop *nh2) {
-    return !olsr_ipequal(&nh1->gateway, &nh2->gateway) || nh1->interface != nh2->interface;
+    return olsr_ipcmp(&nh1->gateway, &nh2->gateway) != 0 || nh1->interface != nh2->interface;
 }
 
 /**
index 84558f1..d114657 100644 (file)
@@ -136,7 +136,7 @@ olsr_add_tc_entry(const union olsr_ip_addr *adr)
   /*
    * Safety net against loss of the last main IP address.
    */
-  if (olsr_ipequal(&olsr_cnf->router_id, &all_zero)) {
+  if (olsr_ipcmp(&olsr_cnf->router_id, &all_zero) == 0) {
     return NULL;
   }
 
@@ -212,7 +212,7 @@ olsr_change_myself_tc(void)
     /*
      * Check if there was a change.
      */
-    if (olsr_ipequal(&tc_myself->addr, &olsr_cnf->router_id)) {
+    if (olsr_ipcmp(&tc_myself->addr, &olsr_cnf->router_id) == 0) {
       return;
     }
 
index b9dcc76..3a046ef 100644 (file)
@@ -80,7 +80,7 @@ olsr_delete_neighbor_pointer(struct neighbor_2_entry *two_hop_entry, const union
   struct neighbor_list_entry *entry = two_hop_entry->neighbor_2_nblist.next;
   while(entry != &two_hop_entry->neighbor_2_nblist)
     {
-      if(olsr_ipequal(&entry->neighbor->neighbor_main_addr, address))
+      if(olsr_ipcmp(&entry->neighbor->neighbor_main_addr, address) == 0)
        {
          struct neighbor_list_entry *entry_to_delete = entry;
          entry = entry->next;
@@ -168,7 +168,7 @@ olsr_lookup_two_hop_neighbor_table(const union olsr_ip_addr *dest)
       neighbor_2 = neighbor_2->next) {
     struct tc_entry *tc;
 
-    if (olsr_ipequal(&neighbor_2->neighbor_2_addr, dest)) {
+    if (olsr_ipcmp(&neighbor_2->neighbor_2_addr, dest) == 0) {
       return neighbor_2;
     }
     /*
@@ -200,7 +200,7 @@ olsr_lookup_two_hop_neighbor_table_mid(const union olsr_ip_addr *dest)
   for(neighbor_2 = two_hop_neighbortable[hash].next;
       neighbor_2 != &two_hop_neighbortable[hash];
       neighbor_2 = neighbor_2->next) {
-    if (olsr_ipequal(&neighbor_2->neighbor_2_addr, dest))
+    if (olsr_ipcmp(&neighbor_2->neighbor_2_addr, dest) == 0)
       return neighbor_2;
   }
   return NULL;
index 1f54995..c05e9f0 100644 (file)
@@ -214,13 +214,13 @@ chk_if_changed(struct olsr_if_config *iface)
     OLSR_PRINTF(3, "\tAddress: %s\n", ip6_to_string(&buf, &tmp_saddr6.sin6_addr));
 #endif
 
-    if (!ip6equal(&tmp_saddr6.sin6_addr, &ifp->int6_addr.sin6_addr)) {
+    if (ip6cmp(&tmp_saddr6.sin6_addr, &ifp->int6_addr.sin6_addr) != 0) {
       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));
 
       /* Check main addr */
-      if (!olsr_cnf->fixed_origaddr && ip6equal(&olsr_cnf->router_id.v6, &tmp_saddr6.sin6_addr)) {
+      if (!olsr_cnf->fixed_origaddr && ip6cmp(&olsr_cnf->router_id.v6, &tmp_saddr6.sin6_addr) != 0) {
         /* Update main addr */
         olsr_cnf->router_id.v6 = tmp_saddr6.sin6_addr;
       }
@@ -246,14 +246,14 @@ chk_if_changed(struct olsr_if_config *iface)
       OLSR_PRINTF(3, "\tAddress:%s\n", ip4_to_string(&buf, ifp->int_addr.sin_addr));
 #endif
 
-    if (!ip4equal(&ifp->int_addr.sin_addr, &tmp_saddr4->sin_addr)) {
+    if (ip4cmp(&ifp->int_addr.sin_addr, &tmp_saddr4->sin_addr) != 0) {
       /* New address */
       OLSR_PRINTF(1, "IPv4 address changed for %s\n", ifr.ifr_name);
       OLSR_PRINTF(1, "\tOld:%s\n", ip4_to_string(&buf, ifp->int_addr.sin_addr));
       OLSR_PRINTF(1, "\tNew:%s\n", ip4_to_string(&buf, tmp_saddr4->sin_addr));
 
       ifp->int_addr = *(struct sockaddr_in *)&ifr.ifr_addr;
-      if (!olsr_cnf->fixed_origaddr && ip4equal(&olsr_cnf->router_id.v4, &ifp->ip_addr.v4)) {
+      if (!olsr_cnf->fixed_origaddr && ip4cmp(&olsr_cnf->router_id.v4, &ifp->ip_addr.v4) == 0) {
         OLSR_INFO(LOG_NETWORKING, "New main address: %s\n", ip4_to_string(&buf, tmp_saddr4->sin_addr));
         olsr_cnf->router_id.v4 = tmp_saddr4->sin_addr;
       }
@@ -274,7 +274,7 @@ chk_if_changed(struct olsr_if_config *iface)
     OLSR_PRINTF(3, "\tNetmask:%s\n", ip4_to_string(&buf, ((struct sockaddr_in *)&ifr.ifr_netmask)->sin_addr));
 #endif
 
-    if (!ip4equal(&ifp->int_netmask.sin_addr, &((struct sockaddr_in *)&ifr.ifr_netmask)->sin_addr)) {
+    if (ip4cmp(&ifp->int_netmask.sin_addr, &((struct sockaddr_in *)&ifr.ifr_netmask)->sin_addr) != 0) {
       /* 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));
@@ -343,7 +343,7 @@ int add_hemu_if (struct olsr_if_config *iface)
   /* Queue */
   list_add_before(&interface_head, &ifp->int_node);
 
-  if (!olsr_cnf->fixed_origaddr && olsr_ipequal(&all_zero, &olsr_cnf->router_id)) {
+  if (!olsr_cnf->fixed_origaddr && olsr_ipcmp(&all_zero, &olsr_cnf->router_id) == 0) {
     olsr_cnf->router_id = iface->hemu_ip;
     OLSR_INFO(LOG_NETWORKING, "New main address: %s\n", olsr_ip_to_string(&buf, &olsr_cnf->router_id));
   }
@@ -728,7 +728,7 @@ chk_if_up(struct olsr_if_config *iface, int debuglvl __attribute__((unused)))
   /*
    * Set main address if this is the only interface
    */
-  if (!olsr_cnf->fixed_origaddr && olsr_ipequal(&all_zero, &olsr_cnf->router_id)) {
+  if (!olsr_cnf->fixed_origaddr && olsr_ipcmp(&all_zero, &olsr_cnf->router_id) == 0) {
     olsr_cnf->router_id = ifp->ip_addr;
     OLSR_INFO(LOG_NETWORKING, "New main address: %s\n", olsr_ip_to_string(&buf, &olsr_cnf->router_id));
   }
index b2a4bb8..23931a4 100644 (file)
@@ -551,7 +551,7 @@ int add_hemu_if(struct olsr_if_config *iface)
   /* Queue */
   list_add_before(&interface_head, &ifp->int_node);
 
-  if(!olsr_cnf->fixed_origaddr && olsr_ipequal(&all_zero, &olsr_cnf->router_id))
+  if(!olsr_cnf->fixed_origaddr && olsr_ipcmp(&all_zero, &olsr_cnf->router_id) == 0)
     {
       olsr_cnf->router_id = iface->hemu_ip;
       OLSR_PRINTF(1, "New main address: %s\n", olsr_ip_to_string(&buf, &olsr_cnf->router_id));
@@ -927,7 +927,7 @@ int chk_if_up(struct olsr_if_config *IntConf, int DebugLevel __attribute__((unus
 
   memset(&NullAddr, 0, olsr_cnf->ipsize);
 
-  if(!olsr_cnf->fixed_origaddr && olsr_ipequal(&NullAddr, &olsr_cnf->router_id))
+  if(!olsr_cnf->fixed_origaddr && olsr_ipcmp(&NullAddr, &olsr_cnf->router_id) == 0)
   {
     olsr_cnf->router_id = New->ip_addr;
     OLSR_PRINTF(1, "New main address: %s\n", olsr_ip_to_string(&buf, &olsr_cnf->router_id));