dcb_doit: remove redundant skb check
authorGaurav Singh <gaurav1086@gmail.com>
Tue, 23 Jun 2020 02:50:39 +0000 (22:50 -0400)
committerDavid S. Miller <davem@davemloft.net>
Wed, 24 Jun 2020 03:27:09 +0000 (20:27 -0700)
skb cannot be NULL here since its already being accessed
before: sock_net(skb->sk). Remove the redundant null check.

Signed-off-by: Gaurav Singh <gaurav1086@gmail.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/dcb/dcbnl.c

index d2a4553..84dde5a 100644 (file)
@@ -1736,7 +1736,7 @@ static int dcb_doit(struct sk_buff *skb, struct nlmsghdr *nlh,
        struct net_device *netdev;
        struct dcbmsg *dcb = nlmsg_data(nlh);
        struct nlattr *tb[DCB_ATTR_MAX + 1];
-       u32 portid = skb ? NETLINK_CB(skb).portid : 0;
+       u32 portid = NETLINK_CB(skb).portid;
        int ret = -EINVAL;
        struct sk_buff *reply_skb;
        struct nlmsghdr *reply_nlh = NULL;