From mboxrd@z Thu Jan 1 00:00:00 1970 Return-Path: Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S1751125AbdKLVks (ORCPT ); Sun, 12 Nov 2017 16:40:48 -0500 Received: from mout.web.de ([212.227.15.4]:54655 "EHLO mout.web.de" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S1750761AbdKLVkq (ORCPT ); Sun, 12 Nov 2017 16:40:46 -0500 Subject: [PATCH 2/2] ALSA: au88x0: Use common error handling code in two functions From: SF Markus Elfring To: alsa-devel@alsa-project.org, Arnd Bergmann , Arvind Yadav , Bhumika Goyal , Colin Ian King , Jaroslav Kysela , Takashi Iwai Cc: LKML , kernel-janitors@vger.kernel.org References: Message-ID: <1ff028cc-eac8-f5c8-9149-5df6e44cfba8@users.sourceforge.net> Date: Sun, 12 Nov 2017 22:40:04 +0100 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:52.0) Gecko/20100101 Thunderbird/52.4.0 MIME-Version: 1.0 In-Reply-To: Content-Type: text/plain; charset=utf-8 Content-Language: en-GB Content-Transfer-Encoding: 8bit X-Provags-ID: V03:K0:fc+QSFDv1bgSEoWzexGT5dHi6x3MjQfyB7b602Z7UsFaLwSIP4q KObfMEjL03k79tPu+6HpMjTJYGY5PlOZqDxoTQ0eQAiknZa4pIkoU4dYGIo1+YE5ZCIhGzu jjH5sW+9wdQV1dKl1it3w9xAIMFa+pbKwFT2pso3aAiDvhXgIsxwdeZpirz4FXC5Tkz9lop cwTbITSDnAUIOMjvXXZ5A== X-UI-Out-Filterresults: notjunk:1;V01:K0:hKfS2LR3H2w=:YKlMdtEvni/rBidLYxzf9r CiLpELF1pgVVbV44vMOmXnquWzmSCK+7iQazoqbPHOm/DDSswGSLlVIBDUOGV7w3/oLfuRHVv s/Tbr/3ip4WRREz+8Qp/7j+ji3d0WGDtQlMzXpu5OmloRk6aG+5VUOGiy6+NUpdEceMgllxm2 Tw8NUKnmGbuyuD9ojiVoG2mV0RH0arZ57Ul24NjTUG2iHCJuvhLL3f6T/jQius9lkw8hJNI9U 2IprexUEEceem94/LEz/PCypgIAsu8tTNqrkvth79raTHWY0C10pvfVMevthNmfnJ5VRDHf41 iCI5Ds4deqJ7QMj5tXtKPLrAlK5DJvO1rv1eVAXNaJrk63NJpG5+PZlI7UpGNAOy0Rn4VqqHZ gFwbjGIVhGCUflfIorMHvpcGwjNVkyWSM4Gg2sVJLg/Arj1QSf0w98LRLuJ/0l9ZFyb65tlZr lnuMCdkiQj8HMkHU4RO4NNHzf+50gR8Y33MEFmJD2Blse//StEbDJa/BhbnIDJgnn98jkZKPG OcRGtVr9EkW87h4OibRStRGY21MaWMhd1DJrTlqhbo9Ufdr5LEmSKp3/NGQElmVo9h4XP+48W +W/Dgei3gSU1XzvDSi0YZ1vN59xy18mK7mVw5fHVF6peHLiWZUhpdbRsgVpyqPeor37532DIB u1SttIs47EMwcKpbK7OgjS8N7iQjTJRnUKjq2NKQ8a5TEZUpGm5FodZ2pOdvIvlFDR5Q2ze9c Hg4lhdG6WZWsW+J504hmdexCAzpoR51t/3IWJjzFQyHKLIagxyiqOh9PNRusTXU4s4ym2OjBK v62mQvgWGGRSOFflKaxigOz05iZGryZExq97Xer5tzobTpsDFU= Sender: linux-kernel-owner@vger.kernel.org List-ID: X-Mailing-List: linux-kernel@vger.kernel.org From: Markus Elfring Date: Sun, 12 Nov 2017 22:22:15 +0100 Add jump targets so that a bit of exception handling can be better reused at the end of these functions. This issue was detected by using the Coccinelle software. Signed-off-by: Markus Elfring --- sound/pci/au88x0/au88x0.c | 72 ++++++++++++++++++------------------------ sound/pci/au88x0/au88x0_core.c | 34 ++++++++------------ 2 files changed, 43 insertions(+), 63 deletions(-) diff --git a/sound/pci/au88x0/au88x0.c b/sound/pci/au88x0/au88x0.c index fc5c47ace64f..aa0dbd4e152e 100644 --- a/sound/pci/au88x0/au88x0.c +++ b/sound/pci/au88x0/au88x0.c @@ -257,10 +257,9 @@ snd_vortex_probe(struct pci_dev *pci, const struct pci_device_id *pci_id) // (3) err = snd_vortex_create(card, pci, &chip); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; + snd_vortex_workaround(pci, pcifix[dev]); // Card details needed in snd_vortex_midi @@ -271,29 +270,23 @@ snd_vortex_probe(struct pci_dev *pci, const struct pci_device_id *pci_id) // (4) Alloc components. err = snd_vortex_mixer(chip); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; + // ADB pcm. err = snd_vortex_new_pcm(chip, VORTEX_PCM_ADB, NR_PCM); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; #ifndef CHIP_AU8820 // ADB SPDIF err = snd_vortex_new_pcm(chip, VORTEX_PCM_SPDIF, 1); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; + // A3D err = snd_vortex_new_pcm(chip, VORTEX_PCM_A3D, NR_A3D); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; #endif /* // ADB I2S @@ -305,16 +298,12 @@ snd_vortex_probe(struct pci_dev *pci, const struct pci_device_id *pci_id) #ifndef CHIP_AU8810 // WT pcm. err = snd_vortex_new_pcm(chip, VORTEX_PCM_WT, NR_WT); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; #endif err = snd_vortex_midi(chip); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; vortex_gameport_register(chip); @@ -337,16 +326,13 @@ snd_vortex_probe(struct pci_dev *pci, const struct pci_device_id *pci_id) // (5) err = pci_read_config_word(pci, PCI_DEVICE_ID, &(chip->device)); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; err = pci_read_config_word(pci, PCI_VENDOR_ID, &(chip->vendor)); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; + chip->rev = pci->revision; #ifdef CHIP_AU8830 if ((chip->rev) != 0xfe && (chip->rev) != 0xfa) { @@ -355,24 +341,26 @@ snd_vortex_probe(struct pci_dev *pci, const struct pci_device_id *pci_id) chip->rev); dev_alert(card->dev, "Please email the results of 'lspci -vv' to openvortex-dev@nongnu.org.\n"); - snd_card_free(card); err = -ENODEV; - return err; + goto free_card; } #endif // (6) err = snd_card_register(card); - if (err < 0) { - snd_card_free(card); - return err; - } + if (err < 0) + goto free_card; + // (7) pci_set_drvdata(pci, card); dev++; vortex_connect_default(chip, 1); vortex_enable_int(chip); return 0; + +free_card: + snd_card_free(card); + return err; } // destructor -- see "Destructor" sub-section diff --git a/sound/pci/au88x0/au88x0_core.c b/sound/pci/au88x0/au88x0_core.c index cdb1c227442b..6615b0933801 100644 --- a/sound/pci/au88x0/au88x0_core.c +++ b/sound/pci/au88x0/au88x0_core.c @@ -2151,11 +2151,9 @@ vortex_adb_allocroute(vortex_t *vortex, int dma, int nr_ch, int dir, ->resources, en, VORTEX_RESOURCE_SRC); - if (src[i] < 0) { - memset(stream->resources, 0, - sizeof(stream->resources)); - return -EBUSY; - } + if (src[i] < 0) + goto reset_memory; + if (stream->type != VORTEX_PCM_A3D) { mix[i] = vortex_adb_checkinout(vortex, @@ -2163,12 +2161,8 @@ vortex_adb_allocroute(vortex_t *vortex, int dma, int nr_ch, int dir, ->resources, en, VORTEX_RESOURCE_MIXIN); - if (mix[i] < 0) { - memset(stream->resources, - 0, - sizeof(stream->resources)); - return -EBUSY; - } + if (mix[i] < 0) + goto reset_memory; } } } @@ -2293,20 +2287,14 @@ vortex_adb_allocroute(vortex_t *vortex, int dma, int nr_ch, int dir, mix[i] = vortex_adb_checkinout(vortex, stream->resources, en, VORTEX_RESOURCE_MIXOUT); - if (mix[i] < 0) { - memset(stream->resources, 0, - sizeof(stream->resources)); - return -EBUSY; - } + if (mix[i] < 0) + goto reset_memory; src[i] = vortex_adb_checkinout(vortex, stream->resources, en, VORTEX_RESOURCE_SRC); - if (src[i] < 0) { - memset(stream->resources, 0, - sizeof(stream->resources)); - return -EBUSY; - } + if (src[i] < 0) + goto reset_memory; } /* Make capture routes. */ @@ -2350,6 +2338,10 @@ vortex_adb_allocroute(vortex_t *vortex, int dma, int nr_ch, int dir, } #endif return dma; + +reset_memory: + memset(stream->resources, 0, sizeof(stream->resources)); + return -EBUSY; } /* -- 2.15.0