Received: by 2002:a05:6a10:af89:0:0:0:0 with SMTP id iu9csp5879286pxb; Thu, 27 Jan 2022 01:15:27 -0800 (PST) X-Google-Smtp-Source: ABdhPJy6NROs/bTC6FpsVFyASISrunppwQVQbdBmq45xvI6tOChz3qNmABnltOjMTH9hfagqta6K X-Received: by 2002:a05:6a00:198f:: with SMTP id d15mr2302209pfl.78.1643274926735; Thu, 27 Jan 2022 01:15:26 -0800 (PST) ARC-Seal: i=1; a=rsa-sha256; t=1643274926; cv=none; d=google.com; s=arc-20160816; b=Yx4kb/taWieavBWweqpjgrc/4Hgob9LS/7of8KV+8wCBVmm3JD/ia5v3t3TZrjIkBO FI9ciapf8qcGwm/vTi1Dsg9AtOiw4pVKzKE87m5d8N0KtdNtC4dO5nmCT3spKkG6nWPC 8t0EYGGpldnPSPVJcny3jOlT52Ra6VCcOp9OGf7JDNbAkcIpKO+yJ8ryW81ms2aW7t5O ga5yvraT6fw8A6o+Kp6sCC25HOz9hsgBxfGWnN9awfsbxJeUJZNewzWiCAMgwLrAGbT0 O1N4MnHeGafiwRpawC1XC2wNJMpKgvnKV/mirf1y4cm3uS1gd/sJpxtp8gf/3xDBe5PU N9Dw== ARC-Message-Signature: i=1; a=rsa-sha256; c=relaxed/relaxed; d=google.com; s=arc-20160816; h=list-id:precedence:content-transfer-encoding:in-reply-to :organization:from:references:cc:to:content-language:subject :user-agent:mime-version:date:message-id:dkim-signature :wdcironportexception:ironport-sdr:ironport-sdr:dkim-signature; bh=DmPRNjSDEj15+H1h2pCe35Fn+q9N5QoPAl2czb8J36E=; b=X2gwrFUcfFNxI5gCrzq/p2LvQOBRCxxXh50UMKBncUSvUZdmM9puBn3/8zXDepBuA5 420qd18ELiwM3lxTyNdxinaT4onc0KsNlivPAUqjgn8moVOIjwQ5760AUL7/yqZx6t9t W9z8qssAzGpsJmdAHYyMvrqblp+PqUCNeRX/xnwdXKcEuYq0wf7mKO4WwSQeIrKReqrZ I6eMhK1j5+WaPakPBz+1S0dEd5/pURFLSfYqlKlu7Gqz+P201L4ACHU+jiP7KVi0NUb5 lnY7Pse2/rRzvfdOk4ReMjgfNw9NaDx4idk5HhHHrzzcK/qBwIqvkBcO1GJ5lVURWkDJ 4k9w== ARC-Authentication-Results: i=1; mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b=gjPLuk+Y; dkim=pass header.i=@opensource.wdc.com header.s=dkim header.b=tUtb7l5s; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=opensource.wdc.com Return-Path: Received: from vger.kernel.org (vger.kernel.org. [23.128.96.18]) by mx.google.com with ESMTP id 123si1723079pgi.481.2022.01.27.01.15.15; Thu, 27 Jan 2022 01:15:26 -0800 (PST) Received-SPF: pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) client-ip=23.128.96.18; Authentication-Results: mx.google.com; dkim=fail header.i=@wdc.com header.s=dkim.wdc.com header.b=gjPLuk+Y; dkim=pass header.i=@opensource.wdc.com header.s=dkim header.b=tUtb7l5s; spf=pass (google.com: domain of linux-kernel-owner@vger.kernel.org designates 23.128.96.18 as permitted sender) smtp.mailfrom=linux-kernel-owner@vger.kernel.org; dmarc=pass (p=QUARANTINE sp=NONE dis=NONE) header.from=opensource.wdc.com Received: (majordomo@vger.kernel.org) by vger.kernel.org via listexpand id S234764AbiA0CPT (ORCPT + 99 others); Wed, 26 Jan 2022 21:15:19 -0500 Received: from esa4.hgst.iphmx.com ([216.71.154.42]:20863 "EHLO esa4.hgst.iphmx.com" rhost-flags-OK-OK-OK-OK) by vger.kernel.org with ESMTP id S232881AbiA0CPR (ORCPT ); Wed, 26 Jan 2022 21:15:17 -0500 DKIM-Signature: v=1; a=rsa-sha256; c=simple/simple; d=wdc.com; i=@wdc.com; q=dns/txt; s=dkim.wdc.com; t=1643249717; x=1674785717; h=message-id:date:mime-version:subject:to:cc:references: from:in-reply-to:content-transfer-encoding; bh=koizvhqZQhyhzGOf2KY9qXoCJNHtohDiLlRaxO3HbiI=; b=gjPLuk+YPHEs29kOQLJydt1PuoNyMoP92vYDknJgTN5UP1i2QslHcF6o fAg95qYrUHITkJROGiOncIe4JPwpBVihWC2cDjrkn7TzSfIrf7O9EuwHT PgVa8R76ugwssBwsYdkYUS/O+pIpPTrFVhChEieBqWze00ZCuKfZIag+1 Z9hCIQk/f58RTpKMncCWULItiR/rB5J+qHTh7vMUDsRdLpHEPnBgPCd1J MhgEEQ/F5D8MaWSoYvXHFH/ChBMXwXWKYfBwGpSgMwljOSrIwBUSLmzAj K/A1e3eadjtGotRGZrQwQJ5GjAguWtrXx7ntIOKIcv4fUu8mx5Hsvhvig Q==; X-IronPort-AV: E=Sophos;i="5.88,319,1635177600"; d="scan'208";a="190429261" Received: from h199-255-45-14.hgst.com (HELO uls-op-cesaep01.wdc.com) ([199.255.45.14]) by ob1.hgst.iphmx.com with ESMTP; 27 Jan 2022 10:15:15 +0800 IronPort-SDR: x0lrlTbol6wn0ZI2lSNOp7jaZu3sHpZ5IkM/8kRW3D+QOlgLNOXqTTBwUQwWfyCwL3HVv+xfl1 ZheN8pYP91TaALlQ8Jwl8Ije+vXGRJpDap/qZOS2wlNIoMlrA3K2Bhis5ori0/iTE+haRZU75I wBJ/IpGQnIo5F3LBBKHdwEdw+aSIh5cHBA9np9M77dr1p8aNS+1kMgmyoEA+R1mE1E0sjyp92K crKgpkqkGjpKmnxyFYLrwv1CKoosD3hEZFwXfQAttceam1n/rQG+9/3kNirQ28fR8PtBcfgVxD 7CwtVlKaV01UTKa69/qwuGmb Received: from uls-op-cesaip01.wdc.com ([10.248.3.36]) by uls-op-cesaep01.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Jan 2022 17:48:34 -0800 IronPort-SDR: LHssMG3tyYHJclHpErBW1hKGH5xz/zgbVqLaSgE0Z7yEgiRhJ0VEqSVWcnf4SF1uVwPdqE4kjZ sGWRgxwiZtqbGtO6+qQQxRUvm35/7siYB08Mlg3lzzAnoRWbB8LQsNaxnHoZY0zdrnYNrCJTAJ WiyHyyAoYko4aGZK5hfGiEuFdQnwi/Za4YKlxEDNu1deJ9xHcaR560LH0WHsCBHfI3Hi+duKT4 TprSxHjdW11Wej4l15YUpnlh9hBX0HiG7oBUWTUQFEkcmJDn1lf+w8MtEPcsOBG+zmELlpFyQl NxY= WDCIronportException: Internal Received: from usg-ed-osssrv.wdc.com ([10.3.10.180]) by uls-op-cesaip01.wdc.com with ESMTP/TLS/ECDHE-RSA-AES256-GCM-SHA384; 26 Jan 2022 18:15:15 -0800 Received: from usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTP id 4Jkkh25FrKz1SVp1 for ; Wed, 26 Jan 2022 18:15:14 -0800 (PST) Authentication-Results: usg-ed-osssrv.wdc.com (amavisd-new); dkim=pass reason="pass (just generated, assumed good)" header.d=opensource.wdc.com DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/simple; d= opensource.wdc.com; h=content-transfer-encoding:content-type :in-reply-to:organization:from:references:to:content-language :subject:user-agent:mime-version:date:message-id; s=dkim; t= 1643249714; x=1645841715; bh=koizvhqZQhyhzGOf2KY9qXoCJNHtohDiLlR axO3HbiI=; b=tUtb7l5sxiIxEUHZMIQP8rrNk2igozlmTew3oldE4le3QT07xz1 oEECnxmKA2AS3+abbDHJasCILWb4V7r4ATs8YQ5gRoAiwGAzj+9ppF4D9cCrO3fQ XSQF7xmRVwSORKlNs9977DYiEdkrgqewxDNchqFiK2AOCURhHfWVlxquonbd4wE6 Wr4U3QxqkX0WWSigMbJ6zjgRJ11yS6tqzuVG2Gc2X+po8DUmZBAPHt9Ntu16y6yj mNFKXyX6S5T2knT9T3yTjmVG+ikN1wttb2NB4Iy5fDRiM4eq30MqjQhL9X1j5fPH I3TBTTlINt+xYWNoLse4LUXt2mj3ZFhLr3w== X-Virus-Scanned: amavisd-new at usg-ed-osssrv.wdc.com Received: from usg-ed-osssrv.wdc.com ([127.0.0.1]) by usg-ed-osssrv.wdc.com (usg-ed-osssrv.wdc.com [127.0.0.1]) (amavisd-new, port 10026) with ESMTP id gKtSm8-1_pKF for ; Wed, 26 Jan 2022 18:15:14 -0800 (PST) Received: from [10.225.163.56] (unknown [10.225.163.56]) by usg-ed-osssrv.wdc.com (Postfix) with ESMTPSA id 4Jkkh10LLVz1RvlN; Wed, 26 Jan 2022 18:15:12 -0800 (PST) Message-ID: Date: Thu, 27 Jan 2022 11:15:11 +0900 MIME-Version: 1.0 User-Agent: Mozilla/5.0 (X11; Linux x86_64; rv:91.0) Gecko/20100101 Thunderbird/91.5.0 Subject: Re: [PATCH] ata: pata_platform: Fix a NULL pointer dereference in __pata_platform_probe() Content-Language: en-US To: Sergey Shtylyov , Zhou Qingyang Cc: kjlu@umn.edu, Alexander Shiyan , Bartlomiej Zolnierkiewicz , Jens Axboe , linux-ide@vger.kernel.org, linux-kernel@vger.kernel.org References: <20220124164525.53068-1-zhou1615@umn.edu> <578d6a7e-1488-43b9-9bfe-9c99c88da4fd@omp.ru> From: Damien Le Moal Organization: Western Digital Research In-Reply-To: <578d6a7e-1488-43b9-9bfe-9c99c88da4fd@omp.ru> Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 7bit Precedence: bulk List-ID: X-Mailing-List: linux-kernel@vger.kernel.org On 1/25/22 18:46, Sergey Shtylyov wrote: > On 1/24/22 7:45 PM, Zhou Qingyang wrote: > >> In __pata_platform_probe(), devm_kzalloc() is assigned to ap->ops and >> there is a dereference of it right after that, which could introduce a >> NULL pointer dereference bug. >> >> Fix this by adding a NULL check of ap->ops. >> >> This bug was found by a static analyzer. >> >> Builds with 'make allyesconfig' show no new warnings, >> and our static analyzer no longer warns about this code. >> >> Fixes: f3d5e4f18dba ("ata: pata_of_platform: Allow to use 16-bit wide data transfer") >> Signed-off-by: Zhou Qingyang > > Reviewed-by: Sergey Shtylyov Please see my note about the typo in the fix. I kept you RB tag. Let me know if that is OK. > > [...] > > MBR, Sergey -- Damien Le Moal Western Digital Research