mDNS: trivial debug print fix and code readibility improvement
authorSaverio Proto <zioproto@gmail.com>
Sun, 12 Aug 2012 12:35:42 +0000 (14:35 +0200)
committerSaverio Proto <zioproto@gmail.com>
Mon, 13 Aug 2012 22:46:08 +0000 (00:46 +0200)
lib/mdns/src/mdns.c

index 1e9f34d..3608952 100644 (file)
@@ -171,9 +171,10 @@ PacketReceivedFromOLSR(unsigned char *encapsulationUdpData, int len)
        * in that case. */
       memset(dest.sll_addr, 0xFF, IFHWADDRLEN);
       
-      if(walker->isActive == 0)             //Don't forward packet if isn't master router
-        return;
-
+      if(walker->isActive == 0) {            //Don't forward packet if isn't master router
+       OLSR_PRINTF(1,"Not forwarding mDNS packet to interface %s because this router is not master\n",walker->ifName);
+       return;
+       }
       nBytesWritten = sendto(walker->capturingSkfd, encapsulationUdpData, stripped_len, 0, (struct sockaddr *)&dest, sizeof(dest));
       if (nBytesWritten != stripped_len) {
         BmfPError("sendto() error forwarding unpacked encapsulated pkt on \"%s\"", walker->ifName);
@@ -460,11 +461,15 @@ BmfPacketCaptured(
     if (destPort != 5353) {
       return;
     }
-    if(my_TTL_Check)
-       if(((u_int8_t) ipHeader->ip_ttl) <= ((u_int8_t) 1))    // Discard mdns packet with TTL limit 1 or less
-               return;
+    if(my_TTL_Check) {
+       if(((u_int8_t) ipHeader->ip_ttl) <= ((u_int8_t) 1)) {   // Discard mdns packet with TTL limit 1 or less
+       OLSR_PRINTF(1,"Discarding packet captured with TTL = 1\n");
+       return;
+       }
+    }
 
     if (isInFilteredList(&src)) {
+      OLSR_PRINTF(1,"Discarding packet captured because of filtered hosts ACL\n");
       return;
     }
 
@@ -492,11 +497,15 @@ BmfPacketCaptured(
     if (destPort != 5353) {
       return;
     }
-    if(my_TTL_Check)
-       if(((uint8_t) ipHeader6->ip6_hops) <= ((uint8_t) 1))  // Discard mdns packet with hop limit 1 or less
-               return;
+    if(my_TTL_Check) {
+       if(((uint8_t) ipHeader6->ip6_hops) <= ((uint8_t) 1)) { // Discard mdns packet with hop limit 1 or less
+       OLSR_PRINTF(1,"Discarding packet captured with TTL = 1\n");
+       return;
+       }
+       }
   
     if (isInFilteredList(&src)) {
+      OLSR_PRINTF(1,"Discarding packet captured because of filtered hosts ACL\n");
       return;
     }
 
@@ -560,8 +569,10 @@ DoMDNS(int skfd, void *data __attribute__ ((unused)), unsigned int flags __attri
     }
 
     for(walker = BmfInterfaces; walker != NULL; walker = walker->next){        //if the router isn't the master for this interface
-      if(skfd == walker->capturingSkfd && walker->isActive == 0)       //discard mdns packets
-        return;
+      if(skfd == walker->capturingSkfd && walker->isActive == 0) {     //discard mdns packets
+         OLSR_PRINTF(1,"Not capturing mDNS packet from interface %s because this router is not master\n",walker->ifName);
+        return;
+      }
     }
 
     if (pktAddr.sll_pkttype == PACKET_OUTGOING ||