diff options
| author | Antonio Quartulli <ordex@autistici.org> | 2012-11-29 01:03:31 +0100 |
|---|---|---|
| committer | Antonio Quartulli <ordex@autistici.org> | 2013-01-12 20:58:22 +1000 |
| commit | 3f87c4239fa1df61beffbee2ca00e8ffbe31a5e3 (patch) | |
| tree | a41fb2169986f164fffbce1d760ca02876d40d86 /net/batman-adv | |
| parent | 46d160ef887963f26fee2df3d3b99eab54637014 (diff) | |
| download | lwn-3f87c4239fa1df61beffbee2ca00e8ffbe31a5e3.tar.gz lwn-3f87c4239fa1df61beffbee2ca00e8ffbe31a5e3.zip | |
batman-adv: remove useless NULL check
debugfs_remove_recursive() checks whether its argument is not null
on its own, therefore it is possible to remove the external check.
Signed-off-by: Antonio Quartulli <ordex@autistici.org>
Signed-off-by: Marek Lindner <lindner_marek@yahoo.de>
Diffstat (limited to 'net/batman-adv')
| -rw-r--r-- | net/batman-adv/debugfs.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/net/batman-adv/debugfs.c b/net/batman-adv/debugfs.c index 0049837e1f2a..5136d32bc9b9 100644 --- a/net/batman-adv/debugfs.c +++ b/net/batman-adv/debugfs.c @@ -396,10 +396,8 @@ err: void batadv_debugfs_destroy(void) { - if (batadv_debugfs) { - debugfs_remove_recursive(batadv_debugfs); - batadv_debugfs = NULL; - } + debugfs_remove_recursive(batadv_debugfs); + batadv_debugfs = NULL; } int batadv_debugfs_add_meshif(struct net_device *dev) |