From mboxrd@z Thu Jan 1 00:00:00 1970 From: Wolfram Sang Date: Sat, 20 Aug 2011 13:44:39 +0000 Subject: Re: [PATCH 3/4] sound/soc/mxs/mxs-saif.c: add missing kfree Message-Id: <20110820134439.GA2266@pengutronix.de> MIME-Version: 1 Content-Type: multipart/mixed; boundary="LQksG6bCIzRHxTLp" List-Id: References: <1313820761-12042-3-git-send-email-julia@diku.dk> In-Reply-To: <1313820761-12042-3-git-send-email-julia@diku.dk> To: Julia Lawall Cc: alsa-devel@alsa-project.org, Takashi Iwai , Mark Brown , kernel-janitors@vger.kernel.org, linux-kernel@vger.kernel.org, Dong Aisheng , Liam Girdwood --LQksG6bCIzRHxTLp Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Sat, Aug 20, 2011 at 08:12:40AM +0200, Julia Lawall wrote: > diff --git a/sound/soc/mxs/mxs-saif.c b/sound/soc/mxs/mxs-saif.c > index 0b3adae..4a17bc8 100644 > --- a/sound/soc/mxs/mxs-saif.c > +++ b/sound/soc/mxs/mxs-saif.c > @@ -523,8 +523,10 @@ static int mxs_saif_probe(struct platform_device *pd= ev) > if (!saif) > return -ENOMEM; > =20 > - if (pdev->id >=3D ARRAY_SIZE(mxs_saif)) > - return -EINVAL; > + if (pdev->id >=3D ARRAY_SIZE(mxs_saif)) { > + ret =3D -EINVAL; > + goto failed; > + } > mxs_saif[pdev->id] =3D saif; Thanks, Julia. May I suggest to simple move the id-check before the allocation? Regards, Wolfram --=20 Pengutronix e.K. | Wolfram Sang | Industrial Linux Solutions | http://www.pengutronix.de/ | --LQksG6bCIzRHxTLp Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature Content-Disposition: inline -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.10 (GNU/Linux) iEYEARECAAYFAk5PukcACgkQD27XaX1/VRs7/gCglf5+sIRVoH2jDAqCqjEVVzTh UX0AoKRND9QLZLtDrUYLrtJ1bczyZx3y =Fu0f -----END PGP SIGNATURE----- --LQksG6bCIzRHxTLp--