diff options
| author | Fabian Frederick <fabf@skynet.be> | 2014-06-26 19:14:31 +0200 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2014-07-01 07:13:38 -0400 |
| commit | 3f4d8f78a07dba1cb333ce749bd6a15c1ada362d (patch) | |
| tree | a6ee520b7fd91b5a306fb7cb6e6e8ed8073f8572 /kernel/trace/trace.c | |
| parent | 9096032fbcdcdb80b76f1046346499e20417988e (diff) | |
| download | lwn-3f4d8f78a07dba1cb333ce749bd6a15c1ada362d.tar.gz lwn-3f4d8f78a07dba1cb333ce749bd6a15c1ada362d.zip | |
tracing: Remove unnecessary null test before debugfs_remove()
This fixes checkpatch warning:
"WARNING: debugfs_remove(NULL) is safe this check is probably not required"
Link: http://lkml.kernel.org/p/1403802871-8599-1-git-send-email-fabf@skynet.be
Signed-off-by: Fabian Frederick <fabf@skynet.be>
Signed-off-by: Steven Rostedt <rostedt@goodmis.org>
Diffstat (limited to 'kernel/trace/trace.c')
| -rw-r--r-- | kernel/trace/trace.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/kernel/trace/trace.c b/kernel/trace/trace.c index eeb233cbac4f..4caa814d41c3 100644 --- a/kernel/trace/trace.c +++ b/kernel/trace/trace.c @@ -6071,10 +6071,8 @@ destroy_trace_option_files(struct trace_option_dentry *topts) if (!topts) return; - for (cnt = 0; topts[cnt].opt; cnt++) { - if (topts[cnt].entry) - debugfs_remove(topts[cnt].entry); - } + for (cnt = 0; topts[cnt].opt; cnt++) + debugfs_remove(topts[cnt].entry); kfree(topts); } |