From 33f34cd0215cd49b4f036b86e4724a1afe8e0af2 Mon Sep 17 00:00:00 2001 From: Manish Poddar Date: Fri, 9 Dec 2016 11:25:31 +0530 Subject: [PATCH] msm: camera: fix Information leak In msm_buf_mngr_sd_shutdown bufs is a pointer defined in kernel, and it is printed to userspace using %lx. changed it to %pK. Change-Id: Ic2cb1a2ff109364ac2da1bbb3253a2253a0c6be9 Signed-off-by: Manish Poddar --- .../media/platform/msm/camera_v2/msm_buf_mgr/msm_generic_buf_mgr.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/media/platform/msm/camera_v2/msm_buf_mgr/msm_generic_buf_mgr.c b/drivers/media/platform/msm/camera_v2/msm_buf_mgr/msm_generic_buf_mgr.c index ac9a4b2048d1..0c73eace8729 100644 --- a/drivers/media/platform/msm/camera_v2/msm_buf_mgr/msm_generic_buf_mgr.c +++ b/drivers/media/platform/msm/camera_v2/msm_buf_mgr/msm_generic_buf_mgr.c @@ -299,8 +299,8 @@ static void msm_buf_mngr_sd_shutdown(struct msm_buf_mngr_device *dev, if (!list_empty(&dev->buf_qhead)) { list_for_each_entry_safe(bufs, save, &dev->buf_qhead, entry) { - pr_info("%s: Delete invalid bufs =%lx, session_id=%u, bufs->ses_id=%d, str_id=%d, idx=%d\n", - __func__, (unsigned long)bufs, session->session, + pr_info("%s: Delete invalid bufs =%pK, session_id=%u, bufs->ses_id=%d, str_id=%d, idx=%d\n", + __func__, (void *)bufs, session->session, bufs->session_id, bufs->stream_id, bufs->vb2_v4l2_buf->vb2_buf.index); if (session->session == bufs->session_id) { -- 2.11.0