diff options
author | Dan Carpenter <error27@gmail.com> | 2011-01-28 16:03:31 -0800 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2011-02-12 10:57:54 -0600 |
commit | e4a9a98289606392f2b1b24b4ca4e29154ec4d15 (patch) | |
tree | 2f2b31f1dddebe7a0bc6d8d97034272314861833 | |
parent | 28a4af1e43047531ab612564e32ab9969c9dd965 (diff) | |
download | linux-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.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) * * scsi-eh will escalate for when either happens. */ - goto out; + return; } if (fc_fcp_lock_pkt(fsp)) |