Add fix for arping broadcast handling #337049 by Aleksander Machniak.

Package-Manager: portage-2.2_rc67/cvs/Linux x86_64

1 files changed, 50 insertions, 0 deletions

diff --git a/net-misc/iputils/files/iputils-20100418-arping-broadcast.patch b/net-misc/iputils/files/iputils-20100418-arping-broadcast.patch
new file mode 100644
index 000000000000..a5c906346bda
--- /dev/null
+++ b/net-misc/iputils/files/iputils-20100418-arping-broadcast.patch
@@ -0,0 +1,50 @@
+http://bugs.gentoo.org/337049
+http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=585591
+
+From: Jesper Dangaard Brouer <hawk@comx.dk>
+Subject: [PATCH] iputils: arping fails to set correct broadcast address
+
+From: Paul Martin <pm@debian.org>
+
+There seems to have been introduced a bug in iputils release s20100418.
+
+This patch is based upon git://www.linux-ipv6.org/gitroot/iputils.git
+(git commit fe342ca3148)
+
+The regression is caused by commit 56018bf1b3
+ arping: Support link-layer type with larger link-layer address.
+
+As reported by: Paul Martin <pm@debian.org> in Debian bugreport #585591.
+
+ There's a logic error in the function that parses the interface's
+ broadcast address, causing it not to fill the broadcast address array
+ correctly.
+
+Please apply.
+
+Reported-by: Paul Martin <pm@debian.org>
+Tested-by: Jesper Dangaard Brouer <hawk@comx.dk>
+Signed-off-by: Jesper Dangaard Brouer <hawk@comx.dk>
+---
+ arping.c |    2 +-
+ 1 files changed, 1 insertions(+), 1 deletions(-)
+
+diff --git a/arping.c b/arping.c
+index 9bd6927..2613a12 100644
+--- a/arping.c
++++ b/arping.c
+@@ -336,7 +336,7 @@ void set_device_broadcast(char *device, unsigned char *ba, size_t balen)
+ 	}
+ 
+ 	for (p = ba, ch = 0; p < ba + balen; p++, ch += 3)
+-		*p++ = strtoul(brdcast->value + ch * 3, NULL, 16);
++		*p = strtoul(brdcast->value + ch, NULL, 16);
+ 
+ 	return;
+ }
+
+
+
+
+
+