diff options
author | Jesper Juhl <jj@chaosbits.net> | 2012-04-12 22:11:25 +0200 |
---|---|---|
committer | Takashi Iwai <tiwai@suse.de> | 2012-04-13 07:35:57 +0200 |
commit | 7d7eb9ea314e992413620610b4d09c9cd5fa8959 (patch) | |
tree | 159a2cc18ad80868af18d7a0675a940c457e1536 /drivers/i2c | |
parent | 29ebe40284c75a5888c601872059fca7e258528d (diff) | |
download | op-kernel-dev-7d7eb9ea314e992413620610b4d09c9cd5fa8959.zip op-kernel-dev-7d7eb9ea314e992413620610b4d09c9cd5fa8959.tar.gz |
ALSA: hda/realtek - Fix mem leak (and rid us of trailing whitespace).
In sound/pci/hda/patch_realtek.c::alc_auto_fill_dac_nids(), in the
'for (;;)' loop, if the 'badness' value returned from
fill_and_eval_dacs() is negative, then we'll return from the function
without freeing the memory we allocated for 'best_cfg', thus leaking.
Fix the leak by kfree()'ing the memory when badness is negative.
While I was there I also noticed some trailing whitespace in the
function that I removed (along with all other trailing whitespace in
the file) - it didn't seem worth-while to do that as two patches, so I
hope it's OK that I just did it all as one patch.
Signed-off-by: Jesper Juhl <jj@chaosbits.net>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'drivers/i2c')
0 files changed, 0 insertions, 0 deletions