From patchwork Sat May 27 09:39:01 2023 Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: 7bit X-Patchwork-Submitter: Helge Deller X-Patchwork-Id: 13257616 Return-Path: X-Spam-Checker-Version: SpamAssassin 3.4.0 (2014-02-07) on aws-us-west-2-korg-lkml-1.web.codeaurora.org Received: from gabe.freedesktop.org (gabe.freedesktop.org [131.252.210.177]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.lore.kernel.org (Postfix) with ESMTPS id BB1D2C77B73 for ; Sat, 27 May 2023 09:39:07 +0000 (UTC) Received: from gabe.freedesktop.org (localhost [127.0.0.1]) by gabe.freedesktop.org (Postfix) with ESMTP id D24CF10E235; Sat, 27 May 2023 09:39:06 +0000 (UTC) Received: from mout.gmx.net (mout.gmx.net [212.227.15.15]) by gabe.freedesktop.org (Postfix) with ESMTPS id 8993510E235 for ; Sat, 27 May 2023 09:39:04 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d=gmx.de; s=s31663417; t=1685180342; i=deller@gmx.de; bh=R9RX/dwotnnh7v3zQFXOlRjv2fsImL0ckXxAGe4BPxI=; h=X-UI-Sender-Class:From:To:Subject:Date; b=I7u5RVu7XL4PrNPrcgdeVeWEQy8gW1kRcxw1KjSLNcwnen5CcvTMxvrsOqZcYl2TC mNOi50x0TsMJy7S8yMhQBbtDWyaIEWk9bTRySGbJfAyahfl+GAtj7p23MLz8qomVB3 H7Tg3DU5h5+nSxzsnbLHwFnB6oh+jK8hEP5xjtwzfoNiK5GnUZGMWy/E4ONns1GM+L ePEiu5JbaoxHNVcb7gr+c8LP7R9BmXSkpuxQS7IfTMn8G8+eVFWOmG/FXWWoLa84AP qEmLgt9oe+XniuOgQrMK41dVhB9TFbZfoBLFfZUArenIt4Sh6LByuyIFQt7ZCTGoiJ DbQuJ/bmXi9mg== X-UI-Sender-Class: 724b4f7f-cbec-4199-ad4e-598c01a50d3a Received: from ls3530.fritz.box ([94.134.159.182]) by mail.gmx.net (mrgmx004 [212.227.17.190]) with ESMTPSA (Nemesis) id 1M4axg-1q4UHQ1FaQ-001jbn; Sat, 27 May 2023 11:39:02 +0200 From: Helge Deller To: linux-fbdev@vger.kernel.org, dri-devel@lists.freedesktop.org Subject: [PATCH] fbdev: imsttfb: Fix error path of imsttfb_probe() Date: Sat, 27 May 2023 11:39:01 +0200 Message-Id: <20230527093901.487483-1-deller@gmx.de> X-Mailer: git-send-email 2.40.1 MIME-Version: 1.0 X-Provags-ID: V03:K1:HbCP5ui6rhvfxxfYGzrxBDTt/tfEDORI3+wHMZffEPk8DonkymS IsQh+zGRDs5V/TkGpOQnxDOQie104hKrNIb/ONpOkabHvtIkHlaDQ/TxAxnCyyfGwY7Peby KFXI7P2HKMOLL9W7zwsDrxXHYF22McPzUrD24HCh36LkJZY+D/OujX0Ej2VXV/qEZCn0oRt HXOyrsyikjTOm/tDmhIbA== UI-OutboundReport: notjunk:1;M01:P0:+zBvTWCIjcQ=;h/5p0EkhWplg/royKpxq00Hv+nE AnhdghfOOZxfZcOaE5cr33UW0yPfQFd0LrszQkTxR2g8zIk07/I8z+YAbLSM4j/txM0ExxM6+ B+cMMTYiY22kqz+nDVFjgDFdkLoNlcZj+5b7LF/IVRnWxmFncf0arFmBJvPPqZZmh9n86EoKN Nrf54UNt0QC0CbK5Ja7sSmBakY3SDnxvL+ProUOatohWGmwrHJfBXqb4Nyl/gAHKxZM4eAjJl +Uo325+MJu6WKqWxZXn6tEMaEsFUmFp47jAVdWtnaS3wpJB3whUR4LJDl1dR7MByH6HSBDQ/B cDxQyaC6IDBy1We8CtojngZVbwQi3rYTLSV8S0DXmaYa8yBbZ3ZQ0qzdL8IoF/Z/Q8I8gyVZo saDoLU6t3NYQpYPQJyEKGix9s442RGXGlT7Krt2ZVFpmOBJLw1htyRz9kn6+AT4fsCTZFIwSz souDbWceFe65KHIzfrFO8DbuJvMoEG56VwXziIl7bpPXMvAm4Zf37eacdYNcR+c72vQ8mucEr bugVwQhwPvptIPZHru8nNbItGJHsYqKhYCJG2LNH/CiVN8o2sXxn5j84txIPUNHhhv2EpIRF9 SA9TGPbiU1hJ3fsw4X12Z3wC+Wqz8SURw4KGwyT8YZOgtPZZ3k6ISUxlkhgc3OUlrDdDREAK8 04LLRuavn5GJFlK5cH9cz8ehaCfnpG0CoKf/+2KW1MBkR8CKWWDjxLRPlnCViUNFyxRK6qUU9 dlC7VGOBSY9MnDkaP7dP9nRUdNiSUhEdJWUVaiWFcl6szvbxoxc+fcm7VFlVOZofS2BJQdOCq nz4ISqbK8zsZ+6/aKyNQa0kN077fWpqoNfPkZqVXdIZzQwoNWYFnMYn/nIrLTyLlkv6tXugpB 6pXimI8kcOQgHgseuYXjvN0FuWZBpa0HZfPP1XsDVxTynwJ2RhKx1nu+25PcjruS/XG46kbF/ vpwTUDguPMJhqPMheH/KRuwXyQk= X-BeenThere: dri-devel@lists.freedesktop.org X-Mailman-Version: 2.1.29 Precedence: list List-Id: Direct Rendering Infrastructure - Development List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" Release ressources when init_imstt() returns failure. Signed-off-by: Helge Deller --- drivers/video/fbdev/imsttfb.c | 6 ++++-- 1 file changed, 4 insertions(+), 2 deletions(-) -- 2.40.1 diff --git a/drivers/video/fbdev/imsttfb.c b/drivers/video/fbdev/imsttfb.c index 075f11991281..ee7d01ad1406 100644 --- a/drivers/video/fbdev/imsttfb.c +++ b/drivers/video/fbdev/imsttfb.c @@ -1535,8 +1535,10 @@ static int imsttfb_probe(struct pci_dev *pdev, const struct pci_device_id *ent) goto error; info->pseudo_palette = par->palette; ret = init_imstt(info); - if (!ret) - pci_set_drvdata(pdev, info); + if (ret) + goto error; + + pci_set_drvdata(pdev, info); return ret; error: