diff options
| author | Vasiliy Kulikov <segooon@gmail.com> | 2011-01-07 16:55:53 (GMT) |
|---|---|---|
| committer | James Bottomley <James.Bottomley@suse.de> | 2011-01-24 17:34:05 (GMT) |
| commit | 938abd8449c27fc67203e1a7c350199cea1158da (patch) | |
| tree | 9df0d8a955d7c874010db02bcefa59e73ff941aa /drivers/scsi | |
| parent | 1d5e2ed0805bf426226b7daa54b3e60af78baa10 (diff) | |
| download | linux-fsl-qoriq-938abd8449c27fc67203e1a7c350199cea1158da.tar.xz | |
[SCSI] hpsa: avoid leaking stack contents to userland
memset arg64 to zero in the passthrough ioctls to avoid leaking contents
of kernel stack memory to userland via uninitialized padding fields
inserted by the compiler for alignment reasons.
Signed-off-by: Stephen M. Cameron <scameron@beardog.cce.hp.com>
Signed-off-by: James Bottomley <James.Bottomley@suse.de>
Diffstat (limited to 'drivers/scsi')
| -rw-r--r-- | drivers/scsi/hpsa.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/drivers/scsi/hpsa.c b/drivers/scsi/hpsa.c index 5828bcb..959eeb2 100644 --- a/drivers/scsi/hpsa.c +++ b/drivers/scsi/hpsa.c @@ -2310,6 +2310,7 @@ static int hpsa_ioctl32_passthru(struct scsi_device *dev, int cmd, void *arg) int err; u32 cp; + memset(&arg64, 0, sizeof(arg64)); err = 0; err |= copy_from_user(&arg64.LUN_info, &arg32->LUN_info, sizeof(arg64.LUN_info)); @@ -2346,6 +2347,7 @@ static int hpsa_ioctl32_big_passthru(struct scsi_device *dev, int err; u32 cp; + memset(&arg64, 0, sizeof(arg64)); err = 0; err |= copy_from_user(&arg64.LUN_info, &arg32->LUN_info, sizeof(arg64.LUN_info)); |