diff options
author | Joe Lawrence <joe.lawrence@stratus.com> | 2014-06-25 17:06:02 -0400 |
---|---|---|
committer | Christoph Hellwig <hch@lst.de> | 2014-07-25 17:16:57 -0400 |
commit | 5074b1b60edd47e0ce4485e57990f7ca75c3e67e (patch) | |
tree | e65147968416fa7b241fbfad44bb3efdc7ee46f9 | |
parent | 5767d25f0d0933ebba111dfb6ae5c820add8d8e4 (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>
-rw-r--r-- | drivers/message/fusion/mptfc.c | 3 | ||||
-rw-r--r-- | drivers/message/fusion/mptspi.c | 3 |
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) | |||
461 | static void | 461 | static void |
462 | mptspi_target_destroy(struct scsi_target *starget) | 462 | mptspi_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 | ||