From mboxrd@z Thu Jan 1 00:00:00 1970 From: Eric Anholt Subject: Re: [PATCH] gpu: drm: pl111: pl111_vexpress.c: Add of_node_put() before return Date: Mon, 15 Jul 2019 17:10:49 -0700 Message-ID: <87muhebzzq.fsf@anholt.net> References: <20190706132742.3250-1-nishkadg.linux@gmail.com> Mime-Version: 1.0 Content-Type: multipart/mixed; boundary="===============2085470747==" Return-path: Received: from anholt.net (anholt.net [50.246.234.109]) by gabe.freedesktop.org (Postfix) with ESMTP id 5341989E59 for ; Tue, 16 Jul 2019 00:10:52 +0000 (UTC) In-Reply-To: <20190706132742.3250-1-nishkadg.linux@gmail.com> List-Unsubscribe: , List-Archive: List-Post: List-Help: List-Subscribe: , Errors-To: dri-devel-bounces@lists.freedesktop.org Sender: "dri-devel" To: airlie@linux.ie, daniel@ffwll.ch, dri-devel@lists.freedesktop.org Cc: Nishka Dasgupta List-Id: dri-devel@lists.freedesktop.org --===============2085470747== Content-Type: multipart/signed; boundary="=-=-="; micalg=pgp-sha512; protocol="application/pgp-signature" --=-=-= Content-Type: text/plain Nishka Dasgupta writes: > Each iteration of for_each_available_child_of_node puts the previous > node, but in the case of a break from the middle of the loop there is > no put, thus causing a memory leak. Hence add an of_node_put before the > break. > Issue found with Coccinelle. Pushed. Thanks! --=-=-= Content-Type: application/pgp-signature; name="signature.asc" -----BEGIN PGP SIGNATURE----- iQIzBAEBCgAdFiEE/JuuFDWp9/ZkuCBXtdYpNtH8nugFAl0tFgkACgkQtdYpNtH8 nuj9xhAAi9b+QoWnxgQRYhs+sulqiY5clJ7Qwlirceo+8+RWbh9CSWMHVJE/dft4 j45UhbctcsLMP7lkqrB+s/UeZjhB2lm+HdFjlhyekE1mGuTL8egKzYaUclybkeNX 9g9SLmjMp+sWZ/MFpxIocIAw8TGdCEA2z1SHjEzk8TTUFIPvprAxiRt4MiXfzMUk o1Lnq93zm64pdbExfQtrEYx82i5s6TfvYREIqngWjfKtIfCkqghQ8Sy37QQ3AboP HJpDBgjZZAhcYDK8e7wrA1RXSjPEHnMneqrzZiO5y3i2HfUo3yodhvaHLLFYhY2A qJj4IfmsetdTHSlkd/8ZPwjiLV3Z68VV3hkdZmmH4/M7h6RQ9R12kRcK884R41Ak yAuEG5N8Y4Mv+zKjhE009gb0zf3SJ5i5WqauiYPjQ/XZ4YzhX2jX2x07qeV4hKlO pGuTtjlCdemvn9Dc8Jkb7vVg3MDZTCZy9UPdPi+17oqcukzi3zTDTIIeMk/Si5U4 MRzDnxSpfrypt3YEqjyd79PQj3oSm5ZxOLF+a9AdaYeMe5tVzuMAV+9C8i7gom1O VyrV38mEE23YwXhUoaiuhrcoHpJHGBEwUyxQC+zA37EQur9LjmC+qn+m+j5CSb35 x++XKl4DNM6u1MEsVrXx1a8edD0N7VVwppNM/t/b0NjdEtvPtbo= =HEHR -----END PGP SIGNATURE----- --=-=-=-- --===============2085470747== Content-Type: text/plain; charset="utf-8" MIME-Version: 1.0 Content-Transfer-Encoding: base64 Content-Disposition: inline X19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX19fX18KZHJpLWRldmVs IG1haWxpbmcgbGlzdApkcmktZGV2ZWxAbGlzdHMuZnJlZWRlc2t0b3Aub3JnCmh0dHBzOi8vbGlz dHMuZnJlZWRlc2t0b3Aub3JnL21haWxtYW4vbGlzdGluZm8vZHJpLWRldmVs --===============2085470747==--