[PATCH] bug 8926 - Arping dropping leading 0 from mac address

Vito Mulè mule.vito at gmail.com
Wed May 25 15:24:36 UTC 2016


friendly ping

---------- Forwarded message ----------
From: Vito Mulè <mule.vito at gmail.com>
Date: 16 May 2016 at 18:16
Subject: Fwd: bug 8926 - Arping dropping leading 0 from mac address
To: busybox at busybox.net


Hello,
not sure if this is necessary since I've opened the bug already, apologies
in case it's not.
Bug link: https://bugs.busybox.net/show_bug.cgi?id=8926


Not sure if this is intende but busybox arping is not consistent with
arping on linux, printing MAC addressed.


root at agent4:/home/vmule# arping 192.168.1.159
ARPING 192.168.1.159
60 bytes from 08:00:27:86:47:9d (192.168.1.159): index=0 time=1.002 sec

and this is busybox arping
root at agent4:/home/vmule# busybox arping 192.168.1.159
ARPING to 192.168.1.159 from 192.168.1.157 via eth0
Unicast reply from 192.168.1.159 [8:0:27:86:47:9d] 0.314ms

I wrote a small patch to fix it:


diff --git a/networking/arping.c b/networking/arping.c
index 6b0de4d..2b22451 100644
--- a/networking/arping.c
+++ b/networking/arping.c
@@ -230,12 +230,15 @@ static void recv_pack(unsigned char *buf, int
len, struct sockaddr_ll *FROM)
        }
        if (!(option_mask32 & QUIET)) {
                int s_printed = 0;
+               struct ether_addr* mac = (struct ether_addr *) p;

-               printf("%scast re%s from %s [%s]",
+               printf("%scast re%s from %s [%02x:%02x:%02x:%02x:%02x:%02x] ",
                        FROM->sll_pkttype == PACKET_HOST ? "Uni" : "Broad",
                        ah->ar_op == htons(ARPOP_REPLY) ? "ply" : "quest",
                        inet_ntoa(src_ip),
-                       ether_ntoa((struct ether_addr *) p));
+                       mac->ether_addr_octet[0], mac->ether_addr_octet[1],
+                       mac->ether_addr_octet[2], mac->ether_addr_octet[3],
+                       mac->ether_addr_octet[4], mac->ether_addr_octet[5]);
                if (dst_ip.s_addr != src.s_addr) {
                        printf("for %s ", inet_ntoa(dst_ip));
                        s_printed = 1;
@@ -243,8 +246,11 @@ static void recv_pack(unsigned char *buf, int
len, struct sockaddr_ll *FROM)
                if (memcmp(p + ah->ar_hln + 4, me.sll_addr, ah->ar_hln)) {
                        if (!s_printed)
                                printf("for ");
-                       printf("[%s]",
-                               ether_ntoa((struct ether_addr *) p +
ah->ar_hln + 4));
+                       struct ether_addr* mac2 = mac + ah->ar_hln + 4;
+                       printf("[%02x:%02x:%02x:%02x:%02x:%02x]",
+                               mac2->ether_addr_octet[0],
mac2->ether_addr_octet[1],
+                               mac2->ether_addr_octet[2],
mac2->ether_addr_octet[3],
+                               mac2->ether_addr_octet[4],
mac2->ether_addr_octet[5]);
                }

                if (last) {


Ideas?


Cheers
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.busybox.net/pipermail/busybox/attachments/20160525/4326892a/attachment.html>


More information about the busybox mailing list