[alsa-cvslog] alsa-kernel: usbaudio - Fix Oops with broken usb descriptors
Brought to you by:
perex
From: Takashi I. <ti...@su...> - 2007-01-31 11:28:35
|
changeset: 4838:776cac567090 tag: tip user: tiwai date: Wed Jan 31 12:27:39 2007 +0100 files: usb/usbaudio.c description: usbaudio - Fix Oops with broken usb descriptors This is a patch for ALSA Bug #2724. Some webcams provide bogus settings with no valid rates. With this patch those are skipped. Signed-off-by: Gregor Jasny <gj...@we...> diff -r b4265ee02e26 -r 776cac567090 usb/usbaudio.c --- a/usb/usbaudio.c Wed Jan 31 10:35:19 2007 +0100 +++ b/usb/usbaudio.c Wed Jan 31 12:27:39 2007 +0100 @@ -2463,6 +2463,7 @@ static int parse_audio_format_rates(stru * build the rate table and bitmap flags */ int r, idx, c; + unsigned int nonzero_rates = 0; /* this table corresponds to the SNDRV_PCM_RATE_XXX bit */ static unsigned int conv_rates[] = { 5512, 8000, 11025, 16000, 22050, 32000, 44100, 48000, @@ -2485,6 +2486,7 @@ static int parse_audio_format_rates(stru fp->altsetting == 5 && fp->maxpacksize == 392) rate = 96000; fp->rate_table[r] = rate; + nonzero_rates |= rate; if (rate < fp->rate_min) fp->rate_min = rate; else if (rate > fp->rate_max) @@ -2499,6 +2501,10 @@ static int parse_audio_format_rates(stru } if (!found) fp->needs_knot = 1; + } + if (!nonzero_rates) { + hwc_debug("All rates were zero. Skipping format!\n"); + return -1; } if (fp->needs_knot) fp->rates |= SNDRV_PCM_RATE_KNOT; |