diff options
author | Oliver Neukum <oneukum@suse.de> | 2007-08-31 06:15:27 -0400 |
---|---|---|
committer | Jaroslav Kysela <perex@perex.cz> | 2007-10-16 09:59:36 -0400 |
commit | 5149fe2c15e2dfe349bfde27e1fea5593b40671f (patch) | |
tree | 646b2a4653d83d02545ca6a30a7a1e6339291a5e /sound/usb | |
parent | 15944806e247076c0886aabe160bee391ce0629c (diff) |
[ALSA] missing error check in usb sound driver
usb_set_interface() can fail, even for altsetting 0
Signed-off-by: Oliver Neukum <oneukum@suse.de>
Signed-off-by: Takashi Iwai <tiwai@suse.de>
Signed-off-by: Jaroslav Kysela <perex@suse.cz>
Diffstat (limited to 'sound/usb')
-rw-r--r-- | sound/usb/usbaudio.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/sound/usb/usbaudio.c b/sound/usb/usbaudio.c index e1558a6c42d8..967b823eace0 100644 --- a/sound/usb/usbaudio.c +++ b/sound/usb/usbaudio.c | |||
@@ -1308,7 +1308,11 @@ static int set_format(struct snd_usb_substream *subs, struct audioformat *fmt) | |||
1308 | 1308 | ||
1309 | /* close the old interface */ | 1309 | /* close the old interface */ |
1310 | if (subs->interface >= 0 && subs->interface != fmt->iface) { | 1310 | if (subs->interface >= 0 && subs->interface != fmt->iface) { |
1311 | usb_set_interface(subs->dev, subs->interface, 0); | 1311 | if (usb_set_interface(subs->dev, subs->interface, 0) < 0) { |
1312 | snd_printk(KERN_ERR "%d:%d:%d: return to setting 0 failed\n", | ||
1313 | dev->devnum, fmt->iface, fmt->altsetting); | ||
1314 | return -EIO; | ||
1315 | } | ||
1312 | subs->interface = -1; | 1316 | subs->interface = -1; |
1313 | subs->format = 0; | 1317 | subs->format = 0; |
1314 | } | 1318 | } |