[Pkg-xen-devel] Bug#437127: [Xen-devel] [PATCH] hotplug: fix ip_of for systems using peer-to-peer link

[Ian Campbell]

Anyone object to or ack this patch?

On Tue, 2012-07-10 at 15:05 +0100, Ian Campbell wrote:

> 8<---------------------------------
> 
> # HG changeset patch
> # User Ian Campbell <ian.campbell at citrix.com>
> # Date 1341929173 -3600
> # Node ID ca5c306052791edf6d96da3f80aecd750b86a5e4
> # Parent  54384951de02e2db909116d64aa6a65d06ffa708
> hotplug: fix ip_of for systems using a peer-to-peer link
> 
> This is from an old Debian bug at
> http://bugs.debian.org/cgi-bin/bugreport.cgi?bug=437127
> 
> I'm slightly inferring what the configuration which trigger this issue is, the
> bug report says:
> 
>     This code however fails if no slash is present in the address, which is the
>     case in my configuration:
> 
>     $ ip addr show dev eth0
>     2: eth0: <BROADCAST,MULTICAST,UP,10000> mtu 1500 qdisc pfifo_fast qlen 1000
>         link/ether 00:16:17:90:b4:35 brd ff:ff:ff:ff:ff:ff
>         inet 88.198.12.XXX peer 88.198.12.XXX/32 brd 88.255.255.255 scope global eth0
> 
> I hope either Marc or Stefan can correct me if I have surmised wrongly what
> this configuration represents.
> 
> This function is used in the vif-route configuration.
> 
> Signed-off-by: Ian Campbell <ian.campbell at citrix.com>
> 
> diff -r 54384951de02 -r ca5c30605279 tools/hotplug/Linux/vif-common.sh
> --- a/tools/hotplug/Linux/vif-common.sh	Tue Jul 10 00:07:20 2012 +0100
> +++ b/tools/hotplug/Linux/vif-common.sh	Tue Jul 10 15:06:13 2012 +0100
> @@ -175,7 +175,7 @@ handle_iptable()
>  #
>  ip_of()
>  {
> -  ip addr show "$1" | awk "/^.*inet.*$1\$/{print \$2}" | sed -n '1 s,/.*,,p'
> +  ip -4 -o addr show primary dev "$1" | awk '$3 == "inet" {split($4,i,"/"); print i[1]; exit}'
>  }
>  
> 
> 
> 
> 
> _______________________________________________
> Xen-devel mailing list
> Xen-devel at lists.xen.org
> http://lists.xen.org/xen-devel