diff options
| author | Dan Carpenter <dan.carpenter@linaro.org> | 2023-11-06 17:05:04 +0300 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2023-11-08 21:42:26 -0500 |
| commit | 037fbd3fcfbd99145f9310d93f6637012807cfd0 (patch) | |
| tree | 0c15ba0bbd0f432e90e9be89067bd871afd3a065 /tools/perf/scripts/python | |
| parent | 860c3d03bbc3f17aef8600662c488f27fd093142 (diff) | |
scsi: scsi_debug: Delete some bogus error checking
Smatch complains that "dentry" is never initialized. These days everyone
initializes all their stack variables to zero so this means that it will
trigger a warning every time this function is run.
Really, debugfs functions are not supposed to be checked for errors in
normal code. For example, if we updated this code to check the correct
variable then it would print a warning if CONFIG_DEBUGFS was disabled. We
don't want that. Just delete the check.
Fixes: f084fe52c640 ("scsi: scsi_debug: Add debugfs interface to fail target reset")
Signed-off-by: Dan Carpenter <dan.carpenter@linaro.org>
Link: https://lore.kernel.org/r/c602c9ad-5e35-4e18-a47f-87ed956a9ec2@moroto.mountain
Reviewed-by: Wenchao Hao <haowenchao2@huawei.com>
Signed-off-by: Martin K. Petersen <martin.petersen@oracle.com>
Diffstat (limited to 'tools/perf/scripts/python')
0 files changed, 0 insertions, 0 deletions