RE: [PATCH] IB/mad: fix ifnullfree.cocci warnings

2015-02-03 Thread Weiny, Ira
> > > > drivers/infiniband/core/mad.c:2088: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

RE: [PATCH] IB/mad: fix ifnullfree.cocci warnings

2015-02-03 Thread Weiny, Ira
> -Original Message- > From: Wu, Fengguang > Sent: Tuesday, January 27, 2015 1:36 PM > To: Weiny, Ira > Cc: kbuild-...@01.org; Roland Dreier; Hefty, Sean; Hal Rosenstock; Or Gerlitz; > Yan Burman; linux-r...@vger.kernel.org; linux-kernel@vger.kernel.org > Subject

RE: [PATCH] IB/mad: fix ifnullfree.cocci warnings

2015-02-03 Thread Weiny, Ira
drivers/infiniband/core/mad.c:2088: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

RE: [PATCH] IB/mad: fix ifnullfree.cocci warnings

2015-02-03 Thread Weiny, Ira
-Original Message- From: Wu, Fengguang Sent: Tuesday, January 27, 2015 1:36 PM To: Weiny, Ira Cc: kbuild-...@01.org; Roland Dreier; Hefty, Sean; Hal Rosenstock; Or Gerlitz; Yan Burman; linux-r...@vger.kernel.org; linux-kernel@vger.kernel.org Subject: [PATCH] IB/mad: fix

[PATCH] IB/mad: fix ifnullfree.cocci warnings

2015-01-27 Thread kbuild test robot
drivers/infiniband/core/mad.c:2088: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

[PATCH] IB/mad: fix ifnullfree.cocci warnings

2015-01-27 Thread kbuild test robot
drivers/infiniband/core/mad.c:2088: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