aboutsummaryrefslogtreecommitdiffstats
path: root/drivers/message
diff options
context:
space:
mode:
authorJoe Lawrence <joe.lawrence@stratus.com>2014-06-25 17:06:02 -0400
committerChristoph Hellwig <hch@lst.de>2014-07-25 17:16:57 -0400
commit5074b1b60edd47e0ce4485e57990f7ca75c3e67e (patch)
treee65147968416fa7b241fbfad44bb3efdc7ee46f9 /drivers/message
parent5767d25f0d0933ebba111dfb6ae5c820add8d8e4 (diff)
mptfusion: remove redundant kfree checks
Fixes the following smatch warnings: drivers/message/fusion/mptfc.c:529 mptfc_target_destroy() info: redundant null check on starget->hostdata calling kfree() drivers/message/fusion/mptspi.c:465 mptspi_target_destroy() info: redundant null check on starget->hostdata calling kfree() Signed-off-by: Joe Lawrence <joe.lawrence@stratus.com> Acked-by: Sreekanth Reddy <Sreekanth.Reddy@avagotech.com> Signed-off-by: Christoph Hellwig <hch@lst.de>
Diffstat (limited to 'drivers/message')
-rw-r--r--drivers/message/fusion/mptfc.c3
-rw-r--r--drivers/message/fusion/mptspi.c3
2 files changed, 2 insertions, 4 deletions
diff --git a/drivers/message/fusion/mptfc.c b/drivers/message/fusion/mptfc.c
index bf2a2cef562b..d8bf84aef602 100644
--- a/drivers/message/fusion/mptfc.c
+++ b/drivers/message/fusion/mptfc.c
@@ -525,8 +525,7 @@ mptfc_target_destroy(struct scsi_target *starget)
525 if (ri) /* better be! */ 525 if (ri) /* better be! */
526 ri->starget = NULL; 526 ri->starget = NULL;
527 } 527 }
528 if (starget->hostdata) 528 kfree(starget->hostdata);
529 kfree(starget->hostdata);
530 starget->hostdata = NULL; 529 starget->hostdata = NULL;
531} 530}
532 531
diff --git a/drivers/message/fusion/mptspi.c b/drivers/message/fusion/mptspi.c
index 7b4db9acf7d7..787933d43d32 100644
--- a/drivers/message/fusion/mptspi.c
+++ b/drivers/message/fusion/mptspi.c
@@ -461,8 +461,7 @@ static int mptspi_target_alloc(struct scsi_target *starget)
461static void 461static void
462mptspi_target_destroy(struct scsi_target *starget) 462mptspi_target_destroy(struct scsi_target *starget)
463{ 463{
464 if (starget->hostdata) 464 kfree(starget->hostdata);
465 kfree(starget->hostdata);
466 starget->hostdata = NULL; 465 starget->hostdata = NULL;
467} 466}
468 467