[alsa-devel] mixer: Fix memory leak for more than 16 file descriptor case

GitHub pull_request - opened github at alsa-project.org
Mon Jan 6 22:39:05 CET 2020


alsa-project/alsa-lib pull request #20 was opened from dfries:

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.

Request URL   : https://github.com/alsa-project/alsa-lib/pull/20
Patch URL     : https://github.com/alsa-project/alsa-lib/pull/20.patch
Repository URL: https://github.com/alsa-project/alsa-lib


More information about the Alsa-devel mailing list