Commit 6230295d authored by Giovanni Cabiddu's avatar Giovanni Cabiddu Committed by Aichun Shi
Browse files

crypto: qat - add NULL pointer check 

mainline inclusion
from mainline-v6.8-rc1
commit a643212c9f28d09225c3792c316bc4aaf6be4a68
category: feature
bugzilla: https://gitee.com/openeuler/intel-kernel/issues/I9A5BW
CVE: N/A
Reference: https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/commit/?id=a643212c9f28d09225c3792c316bc4aaf6be4a68



-------------------------------------

There is a possibility that the function adf_devmgr_pci_to_accel_dev()
might return a NULL pointer.
Add a NULL pointer check in the function rp2srv_show().

Intel-SIG: commit a643212c9f28 crypto: qat - add NULL pointer check
Backport to support QAT in-tree driver

Fixes: dbc8876dd873 ("crypto: qat - add rp2svc sysfs attribute")
Signed-off-by: default avatarGiovanni Cabiddu <giovanni.cabiddu@intel.com>
Reviewed-by: default avatarAhsan Atta <ahsan.atta@intel.com>
Reviewed-by: default avatarDavid Guckian <david.guckian@intel.com>
Signed-off-by: default avatarHerbert Xu <herbert@gondor.apana.org.au>
[ Aichun Shi: amend commit log ]
Signed-off-by: default avatarAichun Shi <aichun.shi@intel.com>
parent cb642950

drivers/crypto/intel/qat/qat_common/adf_sysfs.c

+3 −0
Original line number Diff line number Diff line
@@ -215,6 +215,9 @@ static ssize_t rp2srv_show(struct device *dev, struct device_attribute *attr, 
	enum adf_cfg_service_type svc;



	accel_dev = adf_devmgr_pci_to_accel_dev(to_pci_dev(dev));

	if (!accel_dev)

		return -EINVAL;



	hw_data = GET_HW_DATA(accel_dev);



	if (accel_dev->sysfs.ring_num == UNSET_RING_NUM)