aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorTakashi Iwai <tiwai@suse.de>2016-03-15 12:14:49 +0100
committerAndreas Blaesius <skate4life@gmx.de>2016-11-09 00:23:13 +0100
commit581c5a9030aadf6ab4cf9a1c6b95bedb7e7e6bc8 (patch)
tree6f36c4534b4eaf718198cae5c141ac62d92b96cd
parente101f7e8168fbbfccc9ae76559013a25cb768854 (diff)
downloadkernel_samsung_espresso10-581c5a9030aadf6ab4cf9a1c6b95bedb7e7e6bc8.tar.gz
kernel_samsung_espresso10-581c5a9030aadf6ab4cf9a1c6b95bedb7e7e6bc8.tar.bz2
kernel_samsung_espresso10-581c5a9030aadf6ab4cf9a1c6b95bedb7e7e6bc8.zip
ALSA: usb-audio: Minor code cleanup in create_fixed_stream_quirk()
commit 902eb7fd1e4af3ac69b9b30f8373f118c92b9729 upstream. Just a minor code cleanup: unify the error paths. Change-Id: Idbac50c4d1602ea3b075c747a716cc6eab905b52 Signed-off-by: Takashi Iwai <tiwai@suse.de> [bwh: Backported to 3.2: adjust context] Signed-off-by: Ben Hutchings <ben@decadent.org.uk>
-rw-r--r--sound/usb/quirks.c22
1 files changed, 11 insertions, 11 deletions
diff --git a/sound/usb/quirks.c b/sound/usb/quirks.c
index 62b955b42ac..39e5c776ac6 100644
--- a/sound/usb/quirks.c
+++ b/sound/usb/quirks.c
@@ -152,22 +152,17 @@ static int create_fixed_stream_quirk(struct snd_usb_audio *chip,
stream = (fp->endpoint & USB_DIR_IN)
? SNDRV_PCM_STREAM_CAPTURE : SNDRV_PCM_STREAM_PLAYBACK;
err = snd_usb_add_audio_endpoint(chip, stream, fp);
- if (err < 0) {
- kfree(fp);
- kfree(rate_table);
- return err;
- }
+ if (err < 0)
+ goto error;
if (fp->iface != get_iface_desc(&iface->altsetting[0])->bInterfaceNumber ||
fp->altset_idx >= iface->num_altsetting) {
- kfree(fp);
- kfree(rate_table);
- return -EINVAL;
+ err = -EINVAL;
+ goto error;
}
alts = &iface->altsetting[fp->altset_idx];
if (get_iface_desc(alts)->bNumEndpoints < 1) {
- kfree(fp);
- kfree(rate_table);
- return -EINVAL;
+ err = -EINVAL;
+ goto error;
}
fp->datainterval = snd_usb_parse_datainterval(chip, alts);
@@ -176,6 +171,11 @@ static int create_fixed_stream_quirk(struct snd_usb_audio *chip,
snd_usb_init_pitch(chip, fp->iface, alts, fp);
snd_usb_init_sample_rate(chip, fp->iface, alts, fp, fp->rate_max);
return 0;
+
+ error:
+ kfree(fp);
+ kfree(rate_table);
+ return err;
}
/*