diff options
| author | Fabian Frederick <fabf@skynet.be> | 2014-06-26 13:14:31 -0400 |
|---|---|---|
| committer | Steven Rostedt <rostedt@goodmis.org> | 2014-07-01 07:13:38 -0400 |
| commit | 3f4d8f78a07dba1cb333ce749bd6a15c1ada362d (patch) | |
| tree | a6ee520b7fd91b5a306fb7cb6e6e8ed8073f8572 /kernel/trace | |
| parent | 9096032fbcdcdb80b76f1046346499e20417988e (diff) | |
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')
| -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) | |||
| 6071 | if (!topts) | 6071 | if (!topts) |
| 6072 | return; | 6072 | return; |
| 6073 | 6073 | ||
| 6074 | for (cnt = 0; topts[cnt].opt; cnt++) { | 6074 | for (cnt = 0; topts[cnt].opt; cnt++) |
| 6075 | if (topts[cnt].entry) | 6075 | debugfs_remove(topts[cnt].entry); |
| 6076 | debugfs_remove(topts[cnt].entry); | ||
| 6077 | } | ||
| 6078 | 6076 | ||
| 6079 | kfree(topts); | 6077 | kfree(topts); |
| 6080 | } | 6078 | } |
