diff options
| author | Bart Van Assche <bvanassche@acm.org> | 2019-08-08 20:01:38 -0700 |
|---|---|---|
| committer | Martin K. Petersen <martin.petersen@oracle.com> | 2019-08-12 21:34:05 -0400 |
| commit | 8b21900b09a55ccee1aaddaabee729484f641997 (patch) | |
| tree | 02e5d955c9097acc559183692a5c1d1cce787e4f /tools/perf/scripts/python | |
| parent | scsi: qla2xxx: Remove a superfluous pointer check (diff) | |
| download | linux-8b21900b09a55ccee1aaddaabee729484f641997.tar.gz linux-8b21900b09a55ccee1aaddaabee729484f641997.zip | |
scsi: qla2xxx: Remove two superfluous tests
Since qlt_remove_target() only calls qlt_release() if
vha->vha_tgt.qla_tgt != NULL, checking that pointer inside qlt_release()
is not necessary. This patch fixes the following Coverity complaint:
CID 188348 (#1 of 1): Dereference after null check (FORWARD_NULL)
var_deref_model: Passing null pointer &vha->vha_tgt.qla_tgt->tgt_list_entry
to list_del, which dereferences it.
Cc: Himanshu Madhani <hmadhani@marvell.com>
Signed-off-by: Bart Van Assche <bvanassche@acm.org>
Tested-by: Himanshu Madhani <hmadhani@marvell.com>
Reviewed-by: Himanshu Madhani <hmadhani@marvell.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