From mboxrd@z Thu Jan 1 00:00:00 1970 Received: from mx0a-0031df01.pphosted.com (mx0a-0031df01.pphosted.com [205.220.168.131]) (using TLSv1.2 with cipher ECDHE-RSA-AES256-GCM-SHA384 (256/256 bits)) (No client certificate requested) by smtp.subspace.kernel.org (Postfix) with ESMTPS id 962F879DA for ; Sun, 19 Jul 2026 00:09:25 +0000 (UTC) Authentication-Results: smtp.subspace.kernel.org; arc=none smtp.client-ip=205.220.168.131 ARC-Seal:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784419767; cv=none; b=Yx1Q+g7TeB+8lsFKTNmqiZB8wQXV9zQDu/hfZAx45cvcV/0bMNTZfMu0JDninYY9aR5rwZBKmlYbAAFD6Iig0o6zgyJlb7Oh+das9DvQ9sImmdq/W6jRg3uKgLOduJHbz4gaREGJkr44GlwYyGtnqiwtx0mx0ts7TWwukM+JoJs= ARC-Message-Signature:i=1; a=rsa-sha256; d=subspace.kernel.org; s=arc-20240116; t=1784419767; c=relaxed/simple; bh=Fp1M2fJsMXzbVJa1xj290WI9PSfRcyhTd/zB16zI1wg=; h=Date:From:To:Cc:Subject:Message-ID:In-Reply-To:References: MIME-Version:Content-Type; b=T2Pn7RmoCuwvDWBfYvvJGx4qTzpPwD4+CdpH1f2hmxwWLnRIDkpm6W4Pbya4lDp/s5dKDs2nF/Ti11R5JHLXZ3nRTCj+EnSX6s3n+emFydgQbGF1zvRkrA67T9PJnLYYX+oH+HIl7yuxTYNb4Fuzfj2FI5zTChmvXz3hByQdMa8= ARC-Authentication-Results:i=1; smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com; spf=pass smtp.mailfrom=oss.qualcomm.com; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b=T5Q3Eaeu; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b=h6aV5FFy; arc=none smtp.client-ip=205.220.168.131 Authentication-Results: smtp.subspace.kernel.org; dmarc=pass (p=reject dis=none) header.from=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; spf=pass smtp.mailfrom=oss.qualcomm.com Authentication-Results: smtp.subspace.kernel.org; dkim=pass (2048-bit key) header.d=qualcomm.com header.i=@qualcomm.com header.b="T5Q3Eaeu"; dkim=pass (2048-bit key) header.d=oss.qualcomm.com header.i=@oss.qualcomm.com header.b="h6aV5FFy" Received: from pps.filterd (m0279864.ppops.net [127.0.0.1]) by mx0a-0031df01.pphosted.com (8.18.1.11/8.18.1.11) with ESMTP id 66INT17k1639095 for ; Sun, 19 Jul 2026 00:09:25 GMT DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=qualcomm.com; h= cc:content-transfer-encoding:content-type:date:from:in-reply-to :message-id:mime-version:references:subject:to; s=qcppdkim1; bh= /Ip7BNs+p7U2rvcrd2MAfZwgo2ckBeageifb/ZWfaLE=; b=T5Q3EaeuizOSIlUT nGEb/nxINB+pP40e/s9kvBijWokWKYHTUHUnwgxsA3Zp34NxMPMZS8oln2otGUt3 +9+3MY5WmTXeDPT+vfxE8d41PDpU9DpJjdnJ8VChdPt8ACyYuDqG3XeJ+4sGUCoH c44NnP94lbWp53adUWQLdYAMyphx6A0ZnpLRwrJ7HYzfSiU5dPqnVWSBCVApbycK H/y5Q5j6Dr/rBC1GiAfypO+rT/bM/qL0YfOn7K4fDJ4jQvncithkUqU5h5Z+444O yVBzzc8Kjw7B1tZiyIQBZqBqVBXN1Af6Mc2qeAR1p2j1fax6D9iMxzez5KQSQRXB UIK6dg== Received: from mail-pf1-f199.google.com (mail-pf1-f199.google.com [209.85.210.199]) by mx0a-0031df01.pphosted.com (PPS) with ESMTPS id 4fg2nhhs7n-1 (version=TLSv1.3 cipher=TLS_AES_128_GCM_SHA256 bits=128 verify=NOT) for ; Sun, 19 Jul 2026 00:09:24 +0000 (GMT) Received: by mail-pf1-f199.google.com with SMTP id d2e1a72fcca58-84877b362f6so5636572b3a.2 for ; Sat, 18 Jul 2026 17:09:24 -0700 (PDT) DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=oss.qualcomm.com; s=google; t=1784419764; x=1785024564; darn=vger.kernel.org; h=content-transfer-encoding:content-type:mime-version:references :in-reply-to:message-id:subject:cc:to:from:date:from:to:cc:subject :date:message-id:reply-to:content-type; bh=/Ip7BNs+p7U2rvcrd2MAfZwgo2ckBeageifb/ZWfaLE=; b=h6aV5FFy0IJqf88sbDeP5gVCyWbAHPzT+N7k0UUQ5vEC/ixuzVzTeEC5Ryk4c9IMv9 xl20HGRCP+K8VqnW+p7lO+SD+Nv+mlzdj2DorBMauXWp1ejgrddZ+xfG2I6grKTDHl+b bN6Gau/SPX1rDdh1TaOJw/tfzemgPaAaZ7Q41aDOGncGqnnYMFCf0ngguwqAA0+18Rto 09IBkfG4pggWYozpgw1IkGR70J7/x1YPsBE+4MMG7A2Rkbe0poW1lZJSOz7g+relmH27 24lwqdk0G631xtb6CpEfLnb04dY0CUsQIu2VQiShqmBdWRVr+MkSNazNCarqfMGefaTq 0hww== X-Google-DKIM-Signature: v=1; a=rsa-sha256; c=relaxed/relaxed; d=1e100.net; s=20251104; t=1784419764; x=1785024564; h=content-transfer-encoding:content-type:mime-version:references :in-reply-to:message-id:subject:cc:to:from:date:x-gm-gg :x-gm-message-state:from:to:cc:subject:date:message-id:reply-to :content-type; bh=/Ip7BNs+p7U2rvcrd2MAfZwgo2ckBeageifb/ZWfaLE=; b=i+j5NOFZHQR2illzv6JVLqxqPfcRCahw/cibEQUpWaAvhx/Uq6g+99jnfbfJCzCLxq hjXxaW1idce/mkTRjcGe6Annmxfg6Lk2ky2+9Ldtb0BWW0LKaMr7/F4MGqJvIyr2QMTf 8U58Tv8po/XbWhLpzDO/IM1T8sdChLxUsjkzT6DRRDXmKwBI50+f7XfF0IGEVhBeIYW0 6XeAA4WQ7vqAwdTu13tNs9BeGRDzrDdLasxItpbERDEkizHzXNlTaPuhyJaLLi5duWnJ Ho+0JIFow9FWe+hGE+aBUMvkXvJ9N29BPWsMWPsT/CnLmOL9DXmV3LSNXCv9Fdz+9rJW VEWw== X-Forwarded-Encrypted: i=1; AHgh+RonRhXubWyfDNJgsjbvhfOj9w/iTNtzs7n8zxV485TieVj1dEyC41csJ3SG62YdNVCbh6ZC6MB3IGcm@vger.kernel.org X-Gm-Message-State: AOJu0YzpssN4i9lRfzMTSg0pr6qGKIIZa2GIdL9iYytXrriEG17aLmGQ QA9oY26uOWUVh6FlNrj2TVdW6q3kIvbXYf0xnXxIIZYpBQN/jflYrrCUzPs4KfD/F1TiJW9ZFfo Z+3+yZ4Hb1/5kmaeGURp6ScR4QTNFihPyVnWJQiViRXz/trMxuNGwNsyxYtbJXNn8 X-Gm-Gg: AfdE7cl1ZcEfsdkhrIUu2xcfohwZJhYL0jSKpn0cRcqZ/45KcU5e9aMQJddAmpPqhy6 xxF40PgFRpOSjSeoYxxKzVVA8/vtyGEKZbJvL5gATH6mNzGvwNBtZ36hYQaMBZb6lmbPOgOGM7Y 5jQIzqSefDQ3Q5pTn7qyM9ni1PsqJ95vE27OaFULLBhX/Isfm6thou5aO1Si8KDje4rKRV0oH3q smghskCppYyOTS5uUzNkNCqAyI79iWY/C8Q3kWPKKjCpNufNQhs+rCfMCI37DgtuG1xm9PVWT3V /PoUaAR0qlkEsQHnZYk97CI62iwj8hHC4j0PCgy6MsOag9OoUnhFmXqUJ0QGeEKwv3vxxThOPbk RL+0sfOUDSL2gF8x4 X-Received: by 2002:a05:6a00:400f:b0:848:6ba2:4139 with SMTP id d2e1a72fcca58-84c29487c9amr8216375b3a.36.1784419763999; Sat, 18 Jul 2026 17:09:23 -0700 (PDT) X-Received: by 2002:a05:6a00:400f:b0:848:6ba2:4139 with SMTP id d2e1a72fcca58-84c29487c9amr8216349b3a.36.1784419763537; Sat, 18 Jul 2026 17:09:23 -0700 (PDT) Received: from jic23-huawei ([50.35.46.84]) by smtp.gmail.com with ESMTPSA id d2e1a72fcca58-84c2adc3466sm3491452b3a.15.2026.07.18.17.09.20 (version=TLS1_3 cipher=TLS_AES_256_GCM_SHA384 bits=256/256); Sat, 18 Jul 2026 17:09:22 -0700 (PDT) Date: Sun, 19 Jul 2026 01:09:16 +0100 From: Jonathan Cameron To: Andy Shevchenko Cc: Rodrigo Alencar <455.rodrigo.alencar@gmail.com>, Michael Auchter , linux@analog.com, linux-iio@vger.kernel.org, devicetree@vger.kernel.org, linux-kernel@vger.kernel.org, linux-hardening@vger.kernel.org, Michael Hennerich , David Lechner , Andy Shevchenko , Rob Herring , Krzysztof Kozlowski , Conor Dooley , Philipp Zabel , Kees Cook , "Gustavo A. R. Silva" , sashiko-bot@kernel.org Subject: Re: [PATCH v8 2/8] iio: dac: ad5686: missing NULL check on match data Message-ID: <20260719010916.54de1870@jic23-huawei> In-Reply-To: <20260719005533.41520a27@jic23-huawei> References: <20260716-ad5686-new-features-v8-0-ebb0051af5e5@analog.com> <20260716-ad5686-new-features-v8-2-ebb0051af5e5@analog.com> <20260719005533.41520a27@jic23-huawei> X-Mailer: Claws Mail 4.4.0 (GTK 3.24.52; x86_64-pc-linux-gnu) Precedence: bulk X-Mailing-List: devicetree@vger.kernel.org List-Id: List-Subscribe: List-Unsubscribe: MIME-Version: 1.0 Content-Type: text/plain; charset=US-ASCII Content-Transfer-Encoding: 7bit X-Authority-Analysis: v=2.4 cv=Le0MLDfi c=1 sm=1 tr=0 ts=6a5c15b4 cx=c_pps a=WW5sKcV1LcKqjgzy2JUPuA==:117 a=qC1CW/w66vtJz1P9yTJxNA==:17 a=kj9zAlcOel0A:10 a=RAioF0-LDSMA:10 a=s4-Qcg_JpJYA:10 a=VkNPw1HP01LnGYTKEx00:22 a=u7WPNUs3qKkmUXheDGA7:22 a=DJpcGTmdVt4CTyJn9g5Z:22 a=EUspDBNiAAAA:8 a=QyXUC8HyAAAA:8 a=HvgeF_Dkktg8HcRXRxcA:9 a=CjuIK1q_8ugA:10 a=OpyuDcXvxspvyRM73sMx:22 X-Proofpoint-GUID: e_Qek96QKkVsXR8CRmmOWUGkNX4Pimhh X-Proofpoint-ORIG-GUID: e_Qek96QKkVsXR8CRmmOWUGkNX4Pimhh X-Proofpoint-Spam-Details-Enc: AW1haW4tMjYwNzE4MDI1MyBTYWx0ZWRfX37gNJ0Bi2rMV gaeFXVDyd1GIXWPDlicqXKPddo+dc9JG7/vh4IhyycBVLDkFh0DAZpr+UsZlN2qQXcdyoY4NsMl /kZh+LSKNz6qdWMpgeRJFU7JqLHemKK1QBcTu2EgLqqeqNHlwagvIriEr5IqTFiQfzFHP2SXMHc xGPsvtm/+kQyVWJTMJ5Pp6obyy4157x3CAw8Pmi0XnGkPJUs7SfdfmYU49tdNpohB2fiCJN+bOp Do6mbBC4jyhH6x8MXENLhd84QQc1ic/ar8BDGOaIHjgcmY2Yeb8Ag9TOEh4COnIqlRNNhEMjwlc HkSMJABHB/kjt+vgIfYBjtWuKRGsWw7BnMeAghv1AynoFY9kvVZQTThYtI38Rf39ARCV2QqXQzO qfq9aD57LktlBY5NsC5XviV/U6gPXWyBaxNBhHycXov0i4QHQn4c/rgysGA0go44tEyN1C2wdSi oJV7AmIYsoQRPtlWCIQ== X-Proofpoint-Spam-Info: AW1haW4tMjYwNzE4MDI1MyBTYWx0ZWRfX7SYT2nkk9XOX vpkkbM5uKhyK55ccs/LmDVTYuDXyerjd3SKZnquRE63VtkAGB4DhEZ9PV2aWd6oLrtSSH69fAve ypDg3eqTeY1J0PKogj/nv4CR2cFFxBQ= X-Proofpoint-Virus-Version: vendor=baseguard engine=ICAP:2.0.293,Aquarius:18.0.1143,Hydra:6.1.134,FMLib:17.12.100.49 definitions=2026-07-18_07,2026-07-17_01,2025-10-01_01 X-Proofpoint-Spam-Details: rule=outbound_notspam policy=outbound score=0 suspectscore=0 lowpriorityscore=0 clxscore=1015 adultscore=0 phishscore=0 bulkscore=0 impostorscore=0 priorityscore=1501 malwarescore=0 spamscore=0 classifier=typeunknown authscore=0 authtc= authcc= route=outbound adjust=0 reason=mlx scancount=1 engine=8.22.0-2606150000 definitions=main-2607180253 On Sun, 19 Jul 2026 00:55:44 +0100 Jonathan Cameron wrote: > On Fri, 17 Jul 2026 13:28:52 +0300 > Andy Shevchenko wrote: > > > On Fri, Jul 17, 2026 at 10:31:26AM +0100, Rodrigo Alencar wrote: > > > On 17/07/26 11:15, Andy Shevchenko wrote: > > > > On Fri, Jul 17, 2026 at 07:51:35AM +0100, Rodrigo Alencar wrote: > > > > > On 16/07/26 21:42, Andy Shevchenko wrote: > > > > > > On Thu, Jul 16, 2026 at 01:14:18PM +0100, Rodrigo Alencar via B4 Relay wrote: > > > > ... > > > > > > > > > struct iio_dev *indio_dev; > > > > > > > int ret, i; > > > > > > > > > > > > Move an assignment here as well. > > > > > > > > > > > > chip_info = ... > > > > > > > > > > assignment? this is an input param validation > > > > > > > > Can we make sure we always get a correct one to begin with? > > > > With that an assignment and check can be coupled together. > > > > Also consider use -ENODATA as it's most likely comes from > > > > driver_data. > > > > > > I suppose you are suggesting to move the check to the bus code with: > > > > > > info = i2c_get_match_data(i2c); > > > if (!info) > > > return -ENODATA; > > > > > > rather than here in the core/common code. > > > > Yes! This avoids layering violation and makes the API contract cleaner. > > I applied the following rather than going for a v9 for just this: > Shout if I messed it up. > > diff --git a/drivers/iio/dac/ad5686-spi.c b/drivers/iio/dac/ad5686-spi.c > index 8abfaf8f0c46..591596a68004 100644 > --- a/drivers/iio/dac/ad5686-spi.c > +++ b/drivers/iio/dac/ad5686-spi.c > @@ -98,8 +98,13 @@ static const struct ad5686_bus_ops ad5686_spi_ops = { > > static int ad5686_spi_probe(struct spi_device *spi) > { > - return ad5686_probe(&spi->dev, spi_get_device_match_data(spi), > - spi->modalias, &ad5686_spi_ops); > + const struct ad5686_chip_info *chip_info; Renamed these to info to match the later patch that has this code anyway. > + > + chip_info = spi_get_device_match_data(spi); > + if (!chip_info) > + return -ENODEV; > + > + return ad5686_probe(&spi->dev, chip_info, spi->modalias, &ad5686_spi_ops); > } > > static const struct spi_device_id ad5686_spi_id[] = { > diff --git a/drivers/iio/dac/ad5696-i2c.c b/drivers/iio/dac/ad5696-i2c.c > index d49946adbde3..a6c85d0c33c9 100644 > --- a/drivers/iio/dac/ad5696-i2c.c > +++ b/drivers/iio/dac/ad5696-i2c.c > @@ -68,8 +68,13 @@ static const struct ad5686_bus_ops ad5686_i2c_ops = { > > static int ad5686_i2c_probe(struct i2c_client *i2c) > { > - return ad5686_probe(&i2c->dev, i2c_get_match_data(i2c), > - i2c->name, &ad5686_i2c_ops); > + const struct ad5686_chip_info *chip_info; > + > + chip_info = i2c_get_match_data(i2c); > + if (!chip_info) > + return -ENODEV; > + > + return ad5686_probe(&i2c->dev, chip_info, i2c->name, &ad5686_i2c_ops); > } > > > > > > > > > > > + if (!chip_info) > > > > > > > + return -ENODEV; > > > >