Most recent kernel where this bug did not occur: none Distribution: N/A Hardware Environment: any Software Environment: any Problem Description: The RTNLGRP_ND_USEROPT NetLink message introduced in 2.6.24 does not seem to include any information as to which netif the user options come from: struct nduseroptmsg { unsigned char nduseropt_family; unsigned char nduseropt_pad1; unsigned short nduseropt_opts_len; /* Total length of options */ __u8 nduseropt_icmp_type; __u8 nduseropt_icmp_code; unsigned short nduseropt_pad2; /* Followed by one or more ND options */ }; Considering this is meant to carry data that is heavily link-layer involved, this seems like a feature bug. Could an ifindex field be added there?? Steps to reproduce:
Reply-To: akpm@linux-foundation.org On Sun, 11 Nov 2007 10:52:41 -0800 (PST) bugme-daemon@bugzilla.kernel.org wrote: > http://bugzilla.kernel.org/show_bug.cgi?id=9349 > > Summary: RTNLGRP_ND_USEROPT does not report ifindex > Product: Networking > Version: 2.5 > KernelVersion: 2.6.24-rc > Platform: All > OS/Version: Linux > Tree: Mainline > Status: NEW > Severity: normal > Priority: P1 > Component: IPV6 > AssignedTo: yoshfuji@linux-ipv6.org > ReportedBy: rdenis@simphalempin.com > > > Most recent kernel where this bug did not occur: none > Distribution: N/A > Hardware Environment: any > Software Environment: any > Problem Description: > The RTNLGRP_ND_USEROPT NetLink message introduced in 2.6.24 does not seem to > include any information as to which netif the user options come from: > > struct nduseroptmsg > { > unsigned char nduseropt_family; > unsigned char nduseropt_pad1; > unsigned short nduseropt_opts_len; /* Total length of options */ > __u8 nduseropt_icmp_type; > __u8 nduseropt_icmp_code; > unsigned short nduseropt_pad2; > /* Followed by one or more ND options */ > }; > > Considering this is meant to carry data that is heavily link-layer involved, > this seems like a feature bug. Could an ifindex field be added there?? >
it would be nice to include a commit id for the fix :)
This issue was fixed by: commit dbb2ed24851a290616d66212dc75373fd863d636 Author: Pierre Ynard <linkfanel@yahoo.fr> Date: Mon Nov 12 17:58:35 2007 -0800 [IPV6]: Add ifindex field to ND user option messages. Userland neighbor discovery options are typically heavily involved with the interface on which thay are received: add a missing ifindex field to the original struct. Thanks to Rémi Denis-Courmont. Signed-off-by: Pierre Ynard <linkfanel@yahoo.fr> Signed-off-by: David S. Miller <davem@davemloft.net>