diff options
author | Dan Carpenter <error27@gmail.com> | 2011-01-28 19:03:31 -0500 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2011-02-12 11:57:54 -0500 |
commit | e4a9a98289606392f2b1b24b4ca4e29154ec4d15 (patch) | |
tree | 2f2b31f1dddebe7a0bc6d8d97034272314861833 /drivers/scsi/libfc | |
parent | 28a4af1e43047531ab612564e32ab9969c9dd965 (diff) |
[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>
Diffstat (limited to 'drivers/scsi/libfc')
-rw-r--r-- | drivers/scsi/libfc/fc_fcp.c | 2 |
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) | |||
1323 | * | 1323 | * |
1324 | * scsi-eh will escalate for when either happens. | 1324 | * scsi-eh will escalate for when either happens. |
1325 | */ | 1325 | */ |
1326 | goto out; | 1326 | return; |
1327 | } | 1327 | } |
1328 | 1328 | ||
1329 | if (fc_fcp_lock_pkt(fsp)) | 1329 | if (fc_fcp_lock_pkt(fsp)) |