diff options
author | zhong jiang <zhongjiang@huawei.com> | 2018-09-08 15:35:06 +0200 |
---|---|---|
committer | David S. Miller <davem@davemloft.net> | 2018-09-12 08:01:33 +0200 |
commit | 0527097ce81cad20529e4f2c37a092774d713d36 (patch) | |
tree | 67c0f3b03a35e02a914324307f253ec6f6166353 /drivers/net/xen-netback | |
parent | net: dsa: b53: Uninitialized variable in b53_adjust_link() (diff) | |
download | linux-0527097ce81cad20529e4f2c37a092774d713d36.tar.xz linux-0527097ce81cad20529e4f2c37a092774d713d36.zip |
net: xenbus: remove redundant condition check before debugfs_remove_recursive
debugfs_remove_recursive has taken the IS_ERR_OR_NULL into account. Just
remove the unnecessary condition check.
Signed-off-by: zhong jiang <zhongjiang@huawei.com>
Reviewed-by: Paul Durrant <paul.durrant@citrix.com>
Signed-off-by: David S. Miller <davem@davemloft.net>
Diffstat (limited to 'drivers/net/xen-netback')
-rw-r--r-- | drivers/net/xen-netback/xenbus.c | 3 |
1 files changed, 1 insertions, 2 deletions
diff --git a/drivers/net/xen-netback/xenbus.c b/drivers/net/xen-netback/xenbus.c index cd51492ae6c2..fe1d52247bbe 100644 --- a/drivers/net/xen-netback/xenbus.c +++ b/drivers/net/xen-netback/xenbus.c @@ -254,8 +254,7 @@ static void xenvif_debugfs_delif(struct xenvif *vif) if (IS_ERR_OR_NULL(xen_netback_dbg_root)) return; - if (!IS_ERR_OR_NULL(vif->xenvif_dbg_root)) - debugfs_remove_recursive(vif->xenvif_dbg_root); + debugfs_remove_recursive(vif->xenvif_dbg_root); vif->xenvif_dbg_root = NULL; } #endif /* CONFIG_DEBUG_FS */ |