summaryrefslogtreecommitdiffstats
path: root/sound
diff options
context:
space:
mode:
authorVlad Tsyrklevich <vlad@tsyrklevich.net>2016-10-13 14:36:41 +0200
committerTakashi Iwai <tiwai@suse.de>2016-10-18 14:08:25 +0200
commitd69bb92e402ff948bdcd39f19c9067874fb86873 (patch)
tree7bbc369e84e7466fdfb8b669db191d2cc88ac7c5 /sound
parentALSA: hda - Adding a new group of pin cfg into ALC295 pin quirk table (diff)
downloadlinux-d69bb92e402ff948bdcd39f19c9067874fb86873.tar.xz
linux-d69bb92e402ff948bdcd39f19c9067874fb86873.zip
ALSA: asihpi: fix kernel memory disclosure
Some elements in hr are not cleared before being copied to user space, leaking kernel heap memory to user space. For example, this happens in the error handling code for the HPI_ADAPTER_DELETE case. Zero the memory before it's copied. Signed-off-by: Vlad Tsyrklevich <vlad@tsyrklevich.net> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound')
-rw-r--r--sound/pci/asihpi/hpioctl.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/pci/asihpi/hpioctl.c b/sound/pci/asihpi/hpioctl.c
index d17937b92331..7e3aa50b21f9 100644
--- a/sound/pci/asihpi/hpioctl.c
+++ b/sound/pci/asihpi/hpioctl.c
@@ -111,7 +111,7 @@ long asihpi_hpi_ioctl(struct file *file, unsigned int cmd, unsigned long arg)
return -EINVAL;
hm = kmalloc(sizeof(*hm), GFP_KERNEL);
- hr = kmalloc(sizeof(*hr), GFP_KERNEL);
+ hr = kzalloc(sizeof(*hr), GFP_KERNEL);
if (!hm || !hr) {
err = -ENOMEM;
goto out;