From 8ad8a6f9b23dba6eb17dfc58e8238b3b5e96bf31 Mon Sep 17 00:00:00 2001 From: jitiphil Date: Thu, 1 Mar 2018 23:45:05 +0530 Subject: [PATCH] qcacld-3.0: Fix invalid use of %p in ol_txrx_peer_release_ref Currently %p is used in ol_txrx_peer_release_ref function in ol_txrx.c which violates security guidelines To address this issue change %p to %pK. Change-Id: I06fd9eb260d1c53e4de235b710cd0ea05125b976 CRs-Fixed: 2170905 --- core/dp/txrx/ol_txrx.c | 6 +++--- 1 file changed, 3 insertions(+), 3 deletions(-) diff --git a/core/dp/txrx/ol_txrx.c b/core/dp/txrx/ol_txrx.c index 33e5c50b5d2e..28e9381ec600 100644 --- a/core/dp/txrx/ol_txrx.c +++ b/core/dp/txrx/ol_txrx.c @@ -3489,7 +3489,7 @@ int ol_txrx_peer_release_ref(ol_txrx_peer_handle peer, if (!qdf_atomic_read(&peer->access_list[debug_id])) { qdf_spin_unlock_bh(&pdev->peer_ref_mutex); - ol_txrx_err("peer %p ref was not taken by %d", + ol_txrx_err("peer %pK ref was not taken by %d", peer, debug_id); ol_txrx_dump_peer_access_list(peer); QDF_BUG(0); @@ -3581,7 +3581,7 @@ int ol_txrx_peer_release_ref(ol_txrx_peer_handle peer, qdf_spin_unlock_bh(&pdev->peer_ref_mutex); } - ol_txrx_info_high("[%d][%d]: Deleting peer %p ref_cnt -> %d %s", + ol_txrx_info_high("[%d][%d]: Deleting peer %pK ref_cnt -> %d %s", debug_id, qdf_atomic_read(&peer->access_list[debug_id]), peer, rc, @@ -3607,7 +3607,7 @@ int ol_txrx_peer_release_ref(ol_txrx_peer_handle peer, &peer->access_list[debug_id]); qdf_spin_unlock_bh(&pdev->peer_ref_mutex); if (!ref_silent) - ol_txrx_info_high("[%d][%d]: ref delete peer %p ref_cnt -> %d", + ol_txrx_info_high("[%d][%d]: ref delete peer %pK ref_cnt -> %d", debug_id, access_list, peer, rc);