diff mbox series

[iproute2] libnetlink: linkdump_req: AF_PACKET family also expects ext_filter_mask

Message ID 1548412627-6006-1-git-send-email-chrism@mellanox.com
State Accepted
Delegated to: stephen hemminger
Headers show
Series [iproute2] libnetlink: linkdump_req: AF_PACKET family also expects ext_filter_mask | expand

Commit Message

Chris Mi Jan. 25, 2019, 10:37 a.m. UTC
Without this fix, the VF info can't be showed using command
"ip link".

146: ens1f0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq state UP mode DEFAULT group default qlen 1000
    link/ether 24:8a:07:ad:78:52 brd ff:ff:ff:ff:ff:ff
    vf 0 MAC 02:25:d0:12:01:01, spoof checking off, link-state auto, trust off, query_rss off
    vf 1 MAC 02:25:d0:12:01:02, spoof checking off, link-state auto, trust off, query_rss off

Fixes: d97b16b2c906 ("libnetlink: linkdump_req: Only AF_UNSPEC family expects an ext_filter_mask")

Signed-off-by: Chris Mi <chrism@mellanox.com>
---
 lib/libnetlink.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

David Ahern Jan. 25, 2019, 3:54 p.m. UTC | #1
On 1/25/19 3:37 AM, Chris Mi wrote:
> Without this fix, the VF info can't be showed using command
> "ip link".
> 
> 146: ens1f0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq state UP mode DEFAULT group default qlen 1000
>     link/ether 24:8a:07:ad:78:52 brd ff:ff:ff:ff:ff:ff
>     vf 0 MAC 02:25:d0:12:01:01, spoof checking off, link-state auto, trust off, query_rss off
>     vf 1 MAC 02:25:d0:12:01:02, spoof checking off, link-state auto, trust off, query_rss off
> 
> Fixes: d97b16b2c906 ("libnetlink: linkdump_req: Only AF_UNSPEC family expects an ext_filter_mask")
> 
> Signed-off-by: Chris Mi <chrism@mellanox.com>
> ---
>  lib/libnetlink.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 

Acked-by: David Ahern <dsahern@gmail.com>

Thanks for the fix
Stephen Hemminger Feb. 5, 2019, 11:26 p.m. UTC | #2
On Fri, 25 Jan 2019 10:37:07 +0000
Chris Mi <chrism@mellanox.com> wrote:

> Without this fix, the VF info can't be showed using command
> "ip link".
> 
> 146: ens1f0: <BROADCAST,MULTICAST,UP,LOWER_UP> mtu 1500 qdisc mq state UP mode DEFAULT group default qlen 1000
>     link/ether 24:8a:07:ad:78:52 brd ff:ff:ff:ff:ff:ff
>     vf 0 MAC 02:25:d0:12:01:01, spoof checking off, link-state auto, trust off, query_rss off
>     vf 1 MAC 02:25:d0:12:01:02, spoof checking off, link-state auto, trust off, query_rss off
> 
> Fixes: d97b16b2c906 ("libnetlink: linkdump_req: Only AF_UNSPEC family expects an ext_filter_mask")
> 
> Signed-off-by: Chris Mi <chrism@mellanox.com>

Applied
diff mbox series

Patch

diff --git a/lib/libnetlink.c b/lib/libnetlink.c
index 110f47b..3beb434 100644
--- a/lib/libnetlink.c
+++ b/lib/libnetlink.c
@@ -476,7 +476,7 @@  int rtnl_linkdump_req_filter(struct rtnl_handle *rth, int family,
 int rtnl_linkdump_req_filter_fn(struct rtnl_handle *rth, int family,
 				req_filter_fn_t filter_fn)
 {
-	if (family == AF_UNSPEC) {
+	if (family == AF_UNSPEC || family == AF_PACKET) {
 		struct {
 			struct nlmsghdr nlh;
 			struct ifinfomsg ifm;