Donate to e Foundation | Murena handsets with /e/OS | Own a part of Murena! Learn more

Commit 95c6c483 authored by Mahesh Kumar Kalikot Veetil's avatar Mahesh Kumar Kalikot Veetil Committed by Gerrit - the friendly Code Review server
Browse files

cnss2: Replace "%p" with "%pK" 



The format specifier %p can leak kernel addresses. Use "%pK" instead.
The behaviour of "%pK" depends on the 'kptr_restrict sysctl'.

Change-Id: I65882067783100e7b9ecdc1e697b93948945b4ea
CRs-Fixed: 2616581
Signed-off-by: default avatarMahesh Kumar Kalikot Veetil <mkalikot@codeaurora.org>
parent cfab2293

drivers/net/wireless/cnss2/qmi.c

+3 −3
Original line number Diff line number Diff line
@@ -1029,7 +1029,7 @@ int cnss_wlfw_athdiag_read_send_sync(struct cnss_plat_data *plat_priv, 
		return -ENODEV;



	if (!data || data_len == 0 || data_len > QMI_WLFW_MAX_DATA_SIZE_V01) {

		cnss_pr_err("Invalid parameters for athdiag read: data %p, data_len %u\n",

		cnss_pr_err("Invalid parameters for athdiag read: data %pK, data_len %u\n",

			    data, data_len);

		return -EINVAL;

	}
@@ -1116,12 +1116,12 @@ int cnss_wlfw_athdiag_write_send_sync(struct cnss_plat_data *plat_priv, 
		return -ENODEV;



	if (!data || data_len == 0 || data_len > QMI_WLFW_MAX_DATA_SIZE_V01) {

		cnss_pr_err("Invalid parameters for athdiag write: data %p, data_len %u\n",

		cnss_pr_err("Invalid parameters for athdiag write: data %pK, data_len %u\n",

			    data, data_len);

		return -EINVAL;

	}



	cnss_pr_dbg("athdiag write: state 0x%lx, offset %x, mem_type %x, data_len %u, data %p\n",

	cnss_pr_dbg("athdiag write: state 0x%lx, offset %x, mem_type %x, data_len %u, data %pK\n",

		    plat_priv->driver_state, offset, mem_type, data_len, data);



	req = kzalloc(sizeof(*req), GFP_KERNEL);