diff mbox series

mixer: Fix memory leak for more than 16 file descriptor case

Message ID 20200107151810.pzthoz3w2y3wqae7@spacedout.fries.net (mailing list archive)
State New, archived
Headers show
Series mixer: Fix memory leak for more than 16 file descriptor case | expand

Commit Message

David Fries Jan. 7, 2020, 3:18 p.m. UTC 
Signed-off-by: David Fries <David@Fries.net>
Reviewed-by: Takashi Iwai <tiwai@suse.de>
---
Added signed off.

I saw this when digging through the source code, wrote up a test case,
and verified with valgrind. I don't have that many sound cards so I
modified mixer.c to always take the conditional. alloca seemed like
the least effort way to fix this. Other thoughts were to test pfds !=
spfds and free, but given how little bytes would be allocated putting
on the stack seems reasonable.

I also created the github pull request 20.
https://github.com/alsa-project/alsa-lib/pull/20

 src/mixer/mixer.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

Comments

Takashi Iwai Jan. 8, 2020, 8:50 a.m. UTC | #1 
On Tue, 07 Jan 2020 16:18:10 +0100,
David Fries wrote:
> 
> Signed-off-by: David Fries <David@Fries.net>
> Reviewed-by: Takashi Iwai <tiwai@suse.de>

Applied now.  Thanks.


Takashi
diff mbox series

Patch

diff --git a/src/mixer/mixer.c b/src/mixer/mixer.c
index 82056474..b1af9945 100644
--- a/src/mixer/mixer.c
+++ b/src/mixer/mixer.c
@@ -767,7 +767,7 @@  int snd_mixer_wait(snd_mixer_t *mixer, int timeout)
 	if (count < 0)
 		return count;
 	if ((unsigned int) count > sizeof(spfds) / sizeof(spfds[0])) {
-		pfds = malloc(count * sizeof(*pfds));
+		pfds = alloca(count * sizeof(*pfds));
 		if (!pfds)
 			return -ENOMEM;
 		err = snd_mixer_poll_descriptors(mixer, pfds,