From 71a3560bd0e0d5525dbb24d291e31e8a1bfdbcad Mon Sep 17 00:00:00 2001 From: Mahesh Kumar Kalikot Veetil Date: Thu, 6 Feb 2020 16:49:49 -0800 Subject: [PATCH] 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: Mahesh Kumar Kalikot Veetil --- drivers/net/wireless/cnss2/qmi.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/drivers/net/wireless/cnss2/qmi.c b/drivers/net/wireless/cnss2/qmi.c index 6c0c078beb2b..74c07f076a9e 100644 --- a/drivers/net/wireless/cnss2/qmi.c +++ b/drivers/net/wireless/cnss2/qmi.c @@ -898,7 +898,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; } @@ -985,12 +985,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);