ipv6: fix ifnullfree.cocci warnings
authorWu Fengguang <fengguang.wu@intel.com>
Wed, 9 Sep 2015 22:57:12 +0000 (06:57 +0800)
committerDavid S. Miller <davem@davemloft.net>
Thu, 10 Sep 2015 00:21:01 +0000 (17:21 -0700)
net/ipv6/route.c:2946:3-8: WARNING: NULL check before freeing functions like kfree, debugfs_remove, debugfs_remove_recursive or usb_free_urb is not needed. Maybe consider reorganizing relevant code to avoid passing NULL values.

 NULL check before some freeing functions is not needed.

 Based on checkpatch warning
 "kfree(NULL) is safe this check is probably not required"
 and kfreeaddr.cocci by Julia Lawall.

Generated by: scripts/coccinelle/free/ifnullfree.cocci

CC: Roopa Prabhu <roopa@cumulusnetworks.com>
Signed-off-by: Fengguang Wu <fengguang.wu@intel.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
net/ipv6/route.c

index 34539d3b843fd7851ca5b8c93657f91ad18f970d..53617d71518850dfc26220dd15923b8027c5249a 100644 (file)
@@ -2942,8 +2942,7 @@ cleanup:
        list_for_each_entry_safe(nh, nh_safe, &rt6_nh_list, next) {
                if (nh->rt6_info)
                        dst_free(&nh->rt6_info->dst);
-               if (nh->mxc.mx)
-                       kfree(nh->mxc.mx);
+               kfree(nh->mxc.mx);
                list_del(&nh->next);
                kfree(nh);
        }