Received: by 2002:ac0:e350:0:0:0:0:0 with SMTP id g16csp312718imn; Fri, 29 Jul 2022 07:38:52 -0700 (PDT) X-Google-Smtp-Source: AA6agR7RiyXXy38OhaDqwrt3tncc80cZvopJ8U1DOFbaRJ9jr2Z+hA+A5qjwKTnSRRp0F1P138aS X-Received: by 2002:a05:6a00:114f:b0:528:2c7a:634c with SMTP id b15-20020a056a00114f00b005282c7a634cmr4119652pfm.41.1659105532219; Fri, 29 Jul 2022 07:38:52 -0700 (PDT) ARC-Seal: i=1; a=rsa-sha256; t=1659105532; cv=none; d=google.com; s=arc-20160816; b=Ipn/lZwudGHDaBO6nmClNmZJvlReEBcWz5Tfz54ecf7K3Ww7QVvh5K0PVX7ZGHQ53e +XdG3wn5hSu7BjGnxmPv1PZquElDmUso6sSDqZR22fFl/vlVDq6ei7FO12N+0Obm4cNi PUdoBuDvCwge4Lt5JcuA/YRCNoYV3mF6bJH3h2nRd6pLOZF8WSzcw9kucMARsbvmP1WM 6sYJ+2dYtfdIIvHIxu6oAQ1ssDOQ+0+QCV7O6Atn0KqE4gkcZ2hgd7Rz7rRbmARGQZzb oMbBnVEDhBSa0e+tZTst4Mt7NgUVpHoPbtNkUjLaEtzemipklDQOnBgxpCd87FR3eTYN 5Osw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:in-reply-to:references:cc:to:from :content-language:subject:user-agent:mime-version:date:message-id :dkim-signature:dkim-signature; bh=brxv0Il7aXuaoEfSMewGeVS/r64cW73YuICcYwVyi7I=; b=sa8lSO8J1jzcivuLy9xnH2KO6t+S7Nm6F0Vgx/qVl1C4Wi8cg/zwy/46u9D4M2Kkwn bg7q+5LDx0uJaMCBRyBSIjCkC/IMa0h/tqSFHtsirFeMjW1Xi6037lcYrwQ0OZmqs36k Yh5sDF5T/UttZ42C/jPtFYAzVZdZ5pyP1esPQlL6NoU2Eh+8XeDsjDjlRZciLf9gPpsK PQe2tGS5TEW5gr7FVnnTCx/KoZwLkyyTsZWZuGDHoSkZ5r+kEAki47F/VgzzMnHgvftD Edl+a+zXrO2bE47EhqK/0OpwaVeiMM4y9hBu/TzItmMrKm7BeCBr3qaEIGoYeXNGN2jB QFKw== ARC-Authentication-Results: i=1; mx.google.com; dkim=pass header.i=@suse.de header.s=susede2_rsa header.b=SO9f3fOu; dkim=neutral (no key) header.i=@suse.de header.b="1WXFxXC/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=suse.de Return-Path: Received: from out1.vger.email (out1.vger.email. [2620:137:e000::1:20]) by mx.google.com with ESMTP id d72-20020a63364b000000b00414de676c21si4472551pga.649.2022.07.29.07.38.36; Fri, 29 Jul 2022 07:38:52 -0700 (PDT) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) client-ip=2620:137:e000::1:20; Authentication-Results: mx.google.com; dkim=pass header.i=@suse.de header.s=susede2_rsa header.b=SO9f3fOu; dkim=neutral (no key) header.i=@suse.de header.b="1WXFxXC/"; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 2620:137:e000::1:20 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=NONE sp=NONE dis=NONE) header.from=suse.de Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S237025AbiG2O0C (ORCPT + 99 others); Fri, 29 Jul 2022 10:26:02 -0400 Received: from lindbergh.monkeyblade.net ([23.128.96.19]:34734 "EHLO lindbergh.monkeyblade.net" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S237038AbiG2OZw (ORCPT ); Fri, 29 Jul 2022 10:25:52 -0400 Received: from smtp-out2.suse.de (smtp-out2.suse.de [IPv6:2001:67c:2178:6::1d]) by lindbergh.monkeyblade.net (Postfix) with ESMTPS id 579B07C183 for ; Fri, 29 Jul 2022 07:25:48 -0700 (PDT) Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by smtp-out2.suse.de (Postfix) with ESMTPS id 04EED2105E; Fri, 29 Jul 2022 14:25:47 +0000 (UTC) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_rsa; t=1659104747; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=brxv0Il7aXuaoEfSMewGeVS/r64cW73YuICcYwVyi7I=; b=SO9f3fOuAnftNQ8R8q2Cic7KqH+8EHRwWbJBZHLPMJgGCKNW4E6pyuN+jEF4N3HPy/jSFR BOJvH9SRTbSVS8eTJCCBa6hb74D4chqQFkLrcU89mQcjlJBG+57e/VQl/egioC06bBfq2m dUdwfeVIj510NLlX5tlseZVipnI3bTw= DKIM-Signature: v=1; a=ed25519-sha256; c=relaxed/relaxed; d=suse.de; s=susede2_ed25519; t=1659104747; h=from:from:reply-to:date:date:message-id:message-id:to:to:cc:cc: mime-version:mime-version:content-type:content-type: in-reply-to:in-reply-to:references:references; bh=brxv0Il7aXuaoEfSMewGeVS/r64cW73YuICcYwVyi7I=; b=1WXFxXC/fpCEBNrNKBk8J9H2mFvWmwFkyPh0Jz8YW03D1SjPCm9S9ptcsCShPyAHyHeoWP q6GLBhr/AKFQKcAw== Received: from imap2.suse-dmz.suse.de (imap2.suse-dmz.suse.de [192.168.254.74]) (using TLSv1.3 with cipher TLS_AES_256_GCM_SHA384 (256/256 bits) key-exchange X25519 server-signature ECDSA (P-521) server-digest SHA512) (No client certificate requested) by imap2.suse-dmz.suse.de (Postfix) with ESMTPS id BEACC13A1B; Fri, 29 Jul 2022 14:25:46 +0000 (UTC) Received: from dovecot-director2.suse.de ([192.168.254.65]) by imap2.suse-dmz.suse.de with ESMTPSA id x26ELert42JQZQAAMHmgww (envelope-from ); Fri, 29 Jul 2022 14:25:46 +0000 Message-ID: Date: Fri, 29 Jul 2022 16:25:46 +0200 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.11.0 Subject: Re: [PATCH] drm/kmb: fix dereference before NULL check in kmb_plane_atomic_update() Content-Language: en-US From: Thomas Zimmermann To: Zeng Jingxiang , anitha.chrisanthus@intel.com, edmund.j.dea@intel.com, airlied@linux.ie, daniel@ffwll.ch, laurent.pinchart@ideasonboard.com, maxime@cerno.tech, ville.syrjala@linux.intel.com Cc: Zeng Jingxiang , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org References: <20220729030711.2117849-1-zengjx95@gmail.com> In-Reply-To: Content-Type: multipart/signed; micalg=pgp-sha256; protocol="application/pgp-signature"; boundary="------------8XYdnNgb7h3hVjAfqSJlMFec" X-Spam-Status: No, score=-2.1 required=5.0 tests=BAYES_00,DKIM_SIGNED, DKIM_VALID,DKIM_VALID_AU,DKIM_VALID_EF,NICE_REPLY_A,SPF_HELO_NONE, SPF_PASS autolearn=ham autolearn_force=no version=3.4.6 X-Spam-Checker-Version: SpamAssassin 3.4.6 (2021-04-09) on lindbergh.monkeyblade.net Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org This is an OpenPGP/MIME signed message (RFC 4880 and 3156) --------------8XYdnNgb7h3hVjAfqSJlMFec Content-Type: multipart/mixed; boundary="------------QsDYZ0Y0WtwqwbmmpqeTelLL"; protected-headers="v1" From: Thomas Zimmermann To: Zeng Jingxiang , anitha.chrisanthus@intel.com, edmund.j.dea@intel.com, airlied@linux.ie, daniel@ffwll.ch, laurent.pinchart@ideasonboard.com, maxime@cerno.tech, ville.syrjala@linux.intel.com Cc: Zeng Jingxiang , linux-kernel@vger.kernel.org, dri-devel@lists.freedesktop.org Message-ID: Subject: Re: [PATCH] drm/kmb: fix dereference before NULL check in kmb_plane_atomic_update() References: <20220729030711.2117849-1-zengjx95@gmail.com> In-Reply-To: --------------QsDYZ0Y0WtwqwbmmpqeTelLL Content-Type: text/plain; charset=UTF-8; format=flowed Content-Transfer-Encoding: base64 SGkNCg0KQW0gMjkuMDcuMjIgdW0gMTY6MjMgc2NocmllYiBUaG9tYXMgWmltbWVybWFubjoN Cj4gSGkNCj4gDQo+IEFtIDI5LjA3LjIyIHVtIDA1OjA3IHNjaHJpZWIgWmVuZyBKaW5neGlh bmc6DQo+PiBGcm9tOiBaZW5nIEppbmd4aWFuZyA8bGludXN6ZW5nQHRlbmNlbnQuY29tPg0K Pj4NCj4+IFRoZSAicGxhbmUiIHBvaW50ZXIgd2FzIGFjY2VzcyBiZWZvcmUgY2hlY2tpbmcg aWYgaXQgd2FzIE5VTEwuDQo+Pg0KPj4gVGhlIGRybV9hdG9taWNfZ2V0X29sZF9wbGFuZV9z dGF0ZSgpIGZ1bmN0aW9uIHdpbGwgZGVyZWZlcmVuY2UNCj4+IHRoZSBwb2ludGVyICJwbGFu ZSIuDQo+PiAzNDXCoMKgwqAgc3RydWN0IGRybV9wbGFuZV9zdGF0ZSAqb2xkX3BsYW5lX3N0 YXRlID0NCj4+IMKgwqDCoMKgwqDCoMKgIGRybV9hdG9taWNfZ2V0X29sZF9wbGFuZV9zdGF0 ZShzdGF0ZSwgcGxhbmUpOw0KPj4gMzQ2wqDCoMKgIHN0cnVjdCBkcm1fcGxhbmVfc3RhdGUg Km5ld19wbGFuZV9zdGF0ZSA9DQo+PiDCoMKgwqDCoMKgwqDCoCBkcm1fYXRvbWljX2dldF9u ZXdfcGxhbmVfc3RhdGUoc3RhdGUsIHBsYW5lKTsNCj4+DQo+PiBBIE5VTEwgY2hlY2sgZm9y ICJwbGFuZSIgaW5kaWNhdGVzIHRoYXQgaXQgbWF5IGJlIE5VTEwNCj4+IDM2M8KgwqDCoCBp ZiAoIXBsYW5lIHx8ICFuZXdfcGxhbmVfc3RhdGUgfHwgIW9sZF9wbGFuZV9zdGF0ZSkNCj4g DQo+IElzIHRoaXMgYW4gYWN0dWFsIGJ1ZyB0aGF0IGhhcHBlbnM/DQo+IA0KPiBBbGwgcGxh bmVzIHNob3VsZCBhbHdheXMgaGF2ZSBhIHN0YXRlLiBUaGVyZWZvcmUgdGhlIHRlc3RzIGZv ciANCj4gIW5ld19wbGFuZV9zdGF0ZSBhbmQgIW9sZF9wbGFuZV9zdGF0ZSBjYW4gYmUgcmVt b3ZlZCwgSSdkIHNheS4NCg0KSnVzdCB0byBjbGFyaWZ5OiBtb3ZpbmcgdGhlIHRlc3QgZm9y ICFwbGFuZSBiZWZvcmUgdXNpbmcgb25lIG9mIHRoZSANCmdldF9wbGFuZV9zdGF0ZSBmdW5j dGlvbnMgaXMgYSBjb3JyZWN0IGJ1Z2ZpeC4NCg0KQmVzdCByZWdhcmRzDQpUaG9tYXMNCg0K PiANCj4gQmVzdCByZWdhcmRzDQo+IFRob21hcw0KPiANCj4+DQo+PiBGaXhlczogOTc3Njk3 ZTIwYjNkICgiZHJtL2F0b21pYzogUGFzcyB0aGUgZnVsbCBzdGF0ZSB0byBwbGFuZXMgYXRv bWljIA0KPj4gZGlzYWJsZSBhbmQgdXBkYXRlIikNCj4+IEZpeGVzOiAzNzQxOGJmMTRjMTMg KCJkcm06IFVzZSBzdGF0ZSBoZWxwZXIgaW5zdGVhZCBvZiB0aGUgcGxhbmUgc3RhdGUgDQo+ PiBwb2ludGVyIikNCj4+IFNpZ25lZC1vZmYtYnk6IFplbmcgSmluZ3hpYW5nIDxsaW51c3pl bmdAdGVuY2VudC5jb20+DQo+PiAtLS0NCj4+IMKgIGRyaXZlcnMvZ3B1L2RybS9rbWIva21i X3BsYW5lLmMgfCAxMyArKysrKysrKy0tLS0tDQo+PiDCoCAxIGZpbGUgY2hhbmdlZCwgOCBp bnNlcnRpb25zKCspLCA1IGRlbGV0aW9ucygtKQ0KPj4NCj4+IGRpZmYgLS1naXQgYS9kcml2 ZXJzL2dwdS9kcm0va21iL2ttYl9wbGFuZS5jIA0KPj4gYi9kcml2ZXJzL2dwdS9kcm0va21i L2ttYl9wbGFuZS5jDQo+PiBpbmRleCAyNzM1YjhlYjM1MzcuLmQyYmM5OThiNjVjZSAxMDA2 NDQNCj4+IC0tLSBhL2RyaXZlcnMvZ3B1L2RybS9rbWIva21iX3BsYW5lLmMNCj4+ICsrKyBi L2RyaXZlcnMvZ3B1L2RybS9rbWIva21iX3BsYW5lLmMNCj4+IEBAIC0zNDIsMTAgKzM0Miw3 IEBAIHN0YXRpYyB2b2lkIGttYl9wbGFuZV9zZXRfYWxwaGEoc3RydWN0IA0KPj4ga21iX2Ry bV9wcml2YXRlICprbWIsDQo+PiDCoCBzdGF0aWMgdm9pZCBrbWJfcGxhbmVfYXRvbWljX3Vw ZGF0ZShzdHJ1Y3QgZHJtX3BsYW5lICpwbGFuZSwNCj4+IMKgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoMKgwqDCoMKgwqDCoCBzdHJ1Y3QgZHJtX2F0b21pY19zdGF0ZSAqc3RhdGUp DQo+PiDCoCB7DQo+PiAtwqDCoMKgIHN0cnVjdCBkcm1fcGxhbmVfc3RhdGUgKm9sZF9wbGFu ZV9zdGF0ZSA9IA0KPj4gZHJtX2F0b21pY19nZXRfb2xkX3BsYW5lX3N0YXRlKHN0YXRlLA0K Pj4gLcKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKg wqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgIHBsYW5lKTsNCj4+IC3CoMKgwqAgc3Ry dWN0IGRybV9wbGFuZV9zdGF0ZSAqbmV3X3BsYW5lX3N0YXRlID0gDQo+PiBkcm1fYXRvbWlj X2dldF9uZXdfcGxhbmVfc3RhdGUoc3RhdGUsDQo+PiAtwqDCoMKgwqDCoMKgwqDCoMKgwqDC oMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDCoMKgwqDC oMKgwqAgcGxhbmUpOw0KPj4gK8KgwqDCoCBzdHJ1Y3QgZHJtX3BsYW5lX3N0YXRlICpvbGRf cGxhbmVfc3RhdGUsICpuZXdfcGxhbmVfc3RhdGU7DQo+PiDCoMKgwqDCoMKgIHN0cnVjdCBk cm1fZnJhbWVidWZmZXIgKmZiOw0KPj4gwqDCoMKgwqDCoCBzdHJ1Y3Qga21iX2RybV9wcml2 YXRlICprbWI7DQo+PiDCoMKgwqDCoMKgIHVuc2lnbmVkIGludCB3aWR0aDsNCj4+IEBAIC0z NjAsNyArMzU3LDEzIEBAIHN0YXRpYyB2b2lkIGttYl9wbGFuZV9hdG9taWNfdXBkYXRlKHN0 cnVjdCANCj4+IGRybV9wbGFuZSAqcGxhbmUsDQo+PiDCoMKgwqDCoMKgIHN0YXRpYyBkbWFf YWRkcl90IGFkZHJbTUFYX1NVQl9QTEFORVNdOw0KPj4gwqDCoMKgwqDCoCBzdHJ1Y3QgZGlz cF9jZmcgKmluaXRfZGlzcF9jZmc7DQo+PiAtwqDCoMKgIGlmICghcGxhbmUgfHwgIW5ld19w bGFuZV9zdGF0ZSB8fCAhb2xkX3BsYW5lX3N0YXRlKQ0KPj4gK8KgwqDCoCBpZiAoIXBsYW5l KQ0KPj4gK8KgwqDCoMKgwqDCoMKgIHJldHVybjsNCj4+ICsNCj4+ICvCoMKgwqAgb2xkX3Bs YW5lX3N0YXRlID0gZHJtX2F0b21pY19nZXRfb2xkX3BsYW5lX3N0YXRlKHN0YXRlLCBwbGFu ZSk7DQo+PiArwqDCoMKgIG5ld19wbGFuZV9zdGF0ZSA9IGRybV9hdG9taWNfZ2V0X25ld19w bGFuZV9zdGF0ZShzdGF0ZSwgcGxhbmUpOw0KPj4gKw0KPj4gK8KgwqDCoCBpZiAoIW5ld19w bGFuZV9zdGF0ZSB8fCAhb2xkX3BsYW5lX3N0YXRlKQ0KPj4gwqDCoMKgwqDCoMKgwqDCoMKg IHJldHVybjsNCj4+IMKgwqDCoMKgwqAgZmIgPSBuZXdfcGxhbmVfc3RhdGUtPmZiOw0KPiAN Cg0KLS0gDQpUaG9tYXMgWmltbWVybWFubg0KR3JhcGhpY3MgRHJpdmVyIERldmVsb3Blcg0K U1VTRSBTb2Z0d2FyZSBTb2x1dGlvbnMgR2VybWFueSBHbWJIDQpNYXhmZWxkc3RyLiA1LCA5 MDQwOSBOw7xybmJlcmcsIEdlcm1hbnkNCihIUkIgMzY4MDksIEFHIE7DvHJuYmVyZykNCkdl c2Now6RmdHNmw7xocmVyOiBJdm8gVG90ZXYNCg== --------------QsDYZ0Y0WtwqwbmmpqeTelLL-- --------------8XYdnNgb7h3hVjAfqSJlMFec Content-Type: application/pgp-signature; name="OpenPGP_signature.asc" Content-Description: OpenPGP digital signature Content-Disposition: attachment; filename="OpenPGP_signature" -----BEGIN PGP SIGNATURE----- wsF5BAABCAAjFiEExndm/fpuMUdwYFFolh/E3EQov+AFAmLj7eoFAwAAAAAACgkQlh/E3EQov+CX ahAAsVg3sE20LkP7C2n34XDLir22oTldIpn7EwNbpONH1oEg/yOy3QjHZ+pt+rz6k4k7fsWqOlIk LG8LGFdhJX/cS5V/4uWv4QctHrT1w1QcU6l5GkgHyB7kSgtxOBCnbfY8wyO/NoZ5+Z8xJ2+/SDFn nFgkOdb2D14N7iNZvagD8l0gZB4WssQxXXwdjExZQEd2Uj+xiWyRnNf2crbrp1XmhV18p/zMtZ7t AvsSNM/ZfxrxZ1msdKJlT+G+roPSLcfDYyv5rLk9+rCRairK+hWH2TGxaBr8WV1Z28T6IWdQYAyV 9urzlTlT7+Q2UZXkWTN7pZOWVyaGk4GObcwn4sWr3J5cUTVatSeLEVGGSZFzKRsmwDGp8JRCTsEj UGFHN4zqNIK6ZGHR/5rcigR3CZoxn+vT6bV/QiNXkghN0qUQVw6cz2+Ey8jOEDARoOkzTy28nUjP Ft9dB9f6YW6OioCN1w4yhyvph84Jy74B8AP5f6vcVkkLl8GYuQ22FNFnMW4V1PpBEl0Gx7tn1gJn MzReHC04J6bU77zUyFVcf9Z+zIZy6EMeyRRs5vwdQgGDrtfZ9XEUkOM87NcPOLo8DNZAyKc6OhDY XjWcUGPxUlgFVd/KoCWAv71OcFK4FDV5L2VwTKZ1QBqxJ4PZ8FWXF/mklE+CbyIwoVimd7qSWknO U7A= =YDPy -----END PGP SIGNATURE----- --------------8XYdnNgb7h3hVjAfqSJlMFec--