From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from rcsinet15.oracle.com ([148.87.113.117]) by merlin.infradead.org with esmtps (Exim 4.76 #1 (Red Hat Linux)) id 1RTVeT-0005br-7O for linux-mtd@lists.infradead.org; Thu, 24 Nov 2011 09:31:29 +0000 Date: Thu, 24 Nov 2011 12:31:20 +0300 From: Dan Carpenter To: walter harms Subject: Re: [patch] mtd/docg3: fix error handling in docg3_probe() Message-ID: <20111124093119.GH3258@mwanda> References: <20111124072117.GC14122@elgon.mountain> <4ECE083D.4030707@bfs.de> MIME-Version: 1.0 Content-Type: multipart/signed; micalg=pgp-sha1; protocol="application/pgp-signature"; boundary="bpVaumkpfGNUagdU" Content-Disposition: inline In-Reply-To: <4ECE083D.4030707@bfs.de> Cc: Robert Jarzmik , kernel-janitors@vger.kernel.org, David Woodhouse , linux-mtd@lists.infradead.org List-Id: Linux MTD discussion mailing list List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , --bpVaumkpfGNUagdU Content-Type: text/plain; charset=us-ascii Content-Disposition: inline Content-Transfer-Encoding: quoted-printable On Thu, Nov 24, 2011 at 10:02:53AM +0100, walter harms wrote: > the code nicely shows that docg3_floors[floor] will be foobar if mtd=3D= =3DNULL and floor > 0 > I guess that can cause problems with the "doc_release_device(docg3_floor= s[floor]);" > =09 > perhaps a "docg3_floors[floor] =3D NULL;" can help here ? >=20 It's allocated with kzalloc() so it already is. regards, dan carpenter --bpVaumkpfGNUagdU Content-Type: application/pgp-signature; name="signature.asc" Content-Description: Digital signature -----BEGIN PGP SIGNATURE----- Version: GnuPG v1.4.11 (GNU/Linux) iQIcBAEBAgAGBQJOzg7nAAoJEOnZkXI/YHqRhxEQAKM0rCjCY4nS93Y+ah4mrIiN 30P/6mJP/b9yiO/QrdNhHCdTVTCPRvWp0BF+aucNncljDwSDd4S6snbflKNVz62A 1tdaGlGRIg7OxRwMotuzU+xJwGkD5x4zYHJnZhWZ5CudJDrt3cp0y1gZNKZmZRZB FSHySrr8/r0CYNCDG/QliRpBuo8CPKUbHJTGIGhRalb5h95DYBnH6Ir07t//1JSL KIGZGaUTITNTn/uzp7NcCtQxomR3b0Lm8Nzco8NNyG9NmXp1Fn9QcLU9Q6MDwI6u jg0lcg9iRVJ3tk8eykubohBudlykGSquB15Sk20bY1oHb8QNsO9q1kwY+pAVuHkm mHAdBt9WEf2NAR0t/GMsx51FtanmvDCQi+LgpLA8zF4/Qk4E/D+KQGfarUYGRuIp 5vF/iYCvl4xBRXWdAoW242slYbrbVloBHR8YcPCxY3ObtyKBvrxrUoJNR/OpmYo+ WIyanyFNmJbL+qws1tln/Owben3rJpjlwa8R70mInLkyxgd+ZCcNY6GKp9nzkN+C 6rIgqxrEuZeNj4KF1pZb7mofgHtjAyhvFO9jWJwTbJNXPrfxhxVIOoASrXw2Mfbb DaPJJykgH5CjN5MfNPosZkbNpqXrDR6XPov+AR5sDp/89oXFX2RUubnLDlPZ5PKl Ph2SZKmVwXO4MDjLQbMb =Pjt3 -----END PGP SIGNATURE----- --bpVaumkpfGNUagdU--