summaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorDan Carpenter <error27@gmail.com>2011-01-28 16:03:31 -0800
committerJames Bottomley <James.Bottomley@suse.de>2011-02-12 10:57:54 -0600
commite4a9a98289606392f2b1b24b4ca4e29154ec4d15 (patch)
tree2f2b31f1dddebe7a0bc6d8d97034272314861833
parent28a4af1e43047531ab612564e32ab9969c9dd965 (diff)
downloadlinux-e4a9a98289606392f2b1b24b4ca4e29154ec4d15.tar.gz
linux-e4a9a98289606392f2b1b24b4ca4e29154ec4d15.tar.bz2
linux-e4a9a98289606392f2b1b24b4ca4e29154ec4d15.zip
[SCSI] libfc:prevent dereferencing ERR_PTR in fc_tm_done()
If we goto out, then it tries to call kfree_skb() on an ERR_PTR which will oops. Just return directly. Signed-off-by: Dan Carpenter <error27@gmail.com> Signed-off-by: Robert Love <robert.w.love@intel.com> Signed-off-by: James Bottomley <James.Bottomley@suse.de>
-rw-r--r--drivers/scsi/libfc/fc_fcp.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/scsi/libfc/fc_fcp.c b/drivers/scsi/libfc/fc_fcp.c
index ba639fad0a1c..f4eb1ab7b288 100644
--- a/drivers/scsi/libfc/fc_fcp.c
+++ b/drivers/scsi/libfc/fc_fcp.c
@@ -1323,7 +1323,7 @@ static void fc_tm_done(struct fc_seq *seq, struct fc_frame *fp, void *arg)
*
* scsi-eh will escalate for when either happens.
*/
- goto out;
+ return;
}
if (fc_fcp_lock_pkt(fsp))