oss-sec mailing list archives
Re: CVE request - kernel: prevent heap corruption in snd_ctl_new()
From: Marcus Meissner <meissner () suse de>
Date: Wed, 29 Sep 2010 09:01:15 +0200
On Wed, Sep 29, 2010 at 02:49:52PM +0800, Eugene Teo wrote:
Reported by Dan Rosenberg. The snd_ctl_new() function in sound/core/control.c allocates space for a snd_kcontrol struct by performing arithmetic operations on a user-provided size without checking for integer overflow. If a user provides a large enough size, an overflow will occur, the allocated chunk will be too small, and a second user-influenced value will be written repeatedly past the bounds of this chunk. This code is reachable by unprivileged users who have permission to open a /dev/snd/controlC* device (on many distros, this is group "audio") via the SNDRV_CTL_IOCTL_ELEM_ADD and SNDRV_CTL_IOCTL_ELEM_REPLACE ioctls. Upstream commit: http://git.kernel.org/linus/5591bf07225523600450edd9e6ad258bb877b779
Doesnt seem to be valid. There is also no change in sounds/core/control.c since April in current mainline git. Ciao, Marcus
Current thread:
- CVE request - kernel: prevent heap corruption in snd_ctl_new() Eugene Teo (Sep 28)
- Re: CVE request - kernel: prevent heap corruption in snd_ctl_new() Marcus Meissner (Sep 29)
- Re: CVE request - kernel: prevent heap corruption in snd_ctl_new() Eugene Teo (Sep 29)
- Re: CVE request - kernel: prevent heap corruption in snd_ctl_new() Josh Bressers (Sep 29)
- Re: CVE request - kernel: prevent heap corruption in snd_ctl_new() Eugene Teo (Sep 29)
- Re: CVE request - kernel: prevent heap corruption in snd_ctl_new() Marcus Meissner (Sep 29)