diff options
author | Vasiliy Kulikov <segooon@gmail.com> | 2011-01-07 10:55:53 -0600 |
---|---|---|
committer | James Bottomley <James.Bottomley@suse.de> | 2011-01-24 11:34:05 -0600 |
commit | 938abd8449c27fc67203e1a7c350199cea1158da (patch) | |
tree | 9df0d8a955d7c874010db02bcefa59e73ff941aa /drivers | |
parent | 1d5e2ed0805bf426226b7daa54b3e60af78baa10 (diff) | |
download | linux-938abd8449c27fc67203e1a7c350199cea1158da.tar.gz linux-938abd8449c27fc67203e1a7c350199cea1158da.tar.bz2 linux-938abd8449c27fc67203e1a7c350199cea1158da.zip |
[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')
-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 5828bcb82964..959eeb202d92 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)); |