Changes in uspace/srv/net/inetsrv/sroute.c [a1a101d:a2e3ee6] in mainline

File:

• uspace/srv/net/inetsrv/sroute.c (modified) (1 diff)

uspace/srv/net/inetsrv/sroute.c

@@ -97 +97 @@
 inet_sroute_t *inet_sroute_find(inet_addr_t *addr)
 {
-        uint32_t mask;
-        inet_sroute_t *best;
-
-        log_msg(LOG_DEFAULT, LVL_DEBUG, "inet_sroute_find(%x)", (unsigned)addr->ipv4);
-
-        fibril_mutex_lock(&sroute_list_lock);
-
-        best = NULL;
-
-        list_foreach(sroute_list, link) {
-                inet_sroute_t *sroute = list_get_instance(link,
-                    inet_sroute_t, sroute_list);
-
+        uint32_t addr_addr;
+        int rc = inet_addr_pack(addr, &addr_addr);
+        if (rc != EOK)
+                return NULL;
+        
+        inet_sroute_t *best = NULL;
+        uint8_t best_bits = 0;
+        
+        fibril_mutex_lock(&sroute_list_lock);
+        
+        list_foreach(sroute_list, link) {
+                inet_sroute_t *sroute = list_get_instance(link,
+                    inet_sroute_t, sroute_list);
+                
+                uint32_t dest_addr;
+                uint8_t dest_bits;
+                rc = inet_naddr_pack(&sroute->dest, &dest_addr, &dest_bits);
+                if (rc != EOK)
+                        continue;
+                
                 /* Look for the most specific route */
-                if (best != NULL && best->dest.bits >= sroute->dest.bits)
+                if ((best != NULL) && (best_bits >= dest_bits))
                         continue;
-
-                mask = inet_netmask(sroute->dest.bits);
-                if ((sroute->dest.ipv4 & mask) == (addr->ipv4 & mask)) {
-                        fibril_mutex_unlock(&sroute_list_lock);
-                        log_msg(LOG_DEFAULT, LVL_DEBUG, "inet_sroute_find: found %p",
+                
+                uint32_t mask = inet_netmask(dest_bits);
+                if ((dest_addr & mask) == (addr_addr & mask)) {
+                        log_msg(LOG_DEFAULT, LVL_DEBUG, "inet_sroute_find: found candidate %p",
                             sroute);
-                        return sroute;
+                        
+                        best = sroute;
+                        best_bits = dest_bits;
                 }
         }
-
-        log_msg(LOG_DEFAULT, LVL_DEBUG, "inet_sroute_find: Not found");
-        fibril_mutex_unlock(&sroute_list_lock);
-
-        return NULL;
+        
+        if (best == NULL)
+                log_msg(LOG_DEFAULT, LVL_DEBUG, "inet_sroute_find: Not found");
+        
+        fibril_mutex_unlock(&sroute_list_lock);
+        
+        return best;
 }