diff options
| author | Jesper Juhl <jj@chaosbits.net> | 2010-12-26 16:14:01 -0500 |
|---|---|---|
| committer | James Bottomley <James.Bottomley@suse.de> | 2011-01-24 12:42:10 -0500 |
| commit | efee0bd9b272b0dae1c7fb1bb0be75606c8fa6f8 (patch) | |
| tree | e7c31143eb82b218d34cb2fb254e3d7c914ac569 /drivers/message | |
| parent | 70e14722718ea3fc66400924f7a99c7d8c62b8ff (diff) | |
[SCSI] mptfusion: Fix memory leak in mptctl_getiocinfo()
A 'kfree(karg)' is missing in a failure path in
mptctl.c::mptctl_getiocinfo() which can cause a memory leak.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Acked-by: "Desai, Kashyap" <Kashyap.Desai@lsi.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/message')
| -rw-r--r-- | drivers/message/fusion/mptctl.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/drivers/message/fusion/mptctl.c b/drivers/message/fusion/mptctl.c index a3856ed90ae..1dcc13a52c7 100644 --- a/drivers/message/fusion/mptctl.c +++ b/drivers/message/fusion/mptctl.c | |||
| @@ -1307,8 +1307,10 @@ mptctl_getiocinfo (unsigned long arg, unsigned int data_size) | |||
| 1307 | else | 1307 | else |
| 1308 | karg->adapterType = MPT_IOCTL_INTERFACE_SCSI; | 1308 | karg->adapterType = MPT_IOCTL_INTERFACE_SCSI; |
| 1309 | 1309 | ||
| 1310 | if (karg->hdr.port > 1) | 1310 | if (karg->hdr.port > 1) { |
| 1311 | kfree(karg); | ||
| 1311 | return -EINVAL; | 1312 | return -EINVAL; |
| 1313 | } | ||
| 1312 | port = karg->hdr.port; | 1314 | port = karg->hdr.port; |
| 1313 | 1315 | ||
| 1314 | karg->port = port; | 1316 | karg->port = port; |
