summaryrefslogtreecommitdiffstats
path: root/sound/pci
diff options
context:
space:
mode:
authorJulia Lawall <julia@diku.dk>2010-04-02 14:47:59 +0200
committerTakashi Iwai <tiwai@suse.de>2010-04-04 12:21:42 +0200
commita0fd4345f928d72a56e27b23e4cd28c94bf36be5 (patch)
treec4eb9cbfebfb66bd68ab4070285f9f1920dd4627 /sound/pci
parentf11947c7c5b8abffd328739996dfdffef2b3e03f (diff)
downloadop-kernel-dev-a0fd4345f928d72a56e27b23e4cd28c94bf36be5.zip
op-kernel-dev-a0fd4345f928d72a56e27b23e4cd28c94bf36be5.tar.gz
ALSA: echoaudio - Eliminate use after free
Use the call to snd_card_free in the error handling code at the end of the function, as in the other error cases. A simplified version of the semantic patch that finds this problem is as follows: (http://coccinelle.lip6.fr/) // <smpl> @@ expression E,E2; @@ snd_card_free(E) ... ( E = E2 | * E ) // </smpl> Signed-off-by: Julia Lawall <julia@diku.dk> Signed-off-by: Takashi Iwai <tiwai@suse.de>
Diffstat (limited to 'sound/pci')
-rw-r--r--sound/pci/echoaudio/echoaudio.c5
1 files changed, 2 insertions, 3 deletions
diff --git a/sound/pci/echoaudio/echoaudio.c b/sound/pci/echoaudio/echoaudio.c
index 8dab82d..668a5ec 100644
--- a/sound/pci/echoaudio/echoaudio.c
+++ b/sound/pci/echoaudio/echoaudio.c
@@ -2184,10 +2184,9 @@ static int __devinit snd_echo_probe(struct pci_dev *pci,
goto ctl_error;
#endif
- if ((err = snd_card_register(card)) < 0) {
- snd_card_free(card);
+ err = snd_card_register(card);
+ if (err < 0)
goto ctl_error;
- }
snd_printk(KERN_INFO "Card registered: %s\n", card->longname);
pci_set_drvdata(pci, chip);
OpenPOWER on IntegriCloud