From f2f309d5e265deb273693815f16fa6c41cedf9dc Mon Sep 17 00:00:00 2001 From: Divya Ponnusamy Date: Wed, 15 Jun 2016 16:22:11 +0530 Subject: staging: android: Change %p to %pK in debug messages The format specifier %p can leak kernel addresses while not valuing the kptr_restrict system settings. Use %pK instead of %p, which also evaluates whether kptr_restrict is set. Bug: 30148243 Change-Id: Ib1adf14e9620ad7b1bd3e962001c852610210d46 Signed-off-by: Divya Ponnusamy Ticket: PORRIDGE-499 --- drivers/staging/android/sync.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-) diff --git a/drivers/staging/android/sync.c b/drivers/staging/android/sync.c index 89c72797a..b473f883c 100644 --- a/drivers/staging/android/sync.c +++ b/drivers/staging/android/sync.c @@ -652,7 +652,7 @@ int sync_fence_wait(struct sync_fence *fence, long timeout) return err; if (fence->status < 0) { - pr_info("fence error %d on [%p]\n", fence->status, fence); + pr_info("fence error %d on [%pK]\n", fence->status, fence); sync_dump(); // [MTK] {{{ sync_fence_dump_info(fence); @@ -662,7 +662,7 @@ int sync_fence_wait(struct sync_fence *fence, long timeout) if (fence->status == 0) { if (timeout > 0) { - pr_info("fence timeout on %s [%p] after %dms\n", fence->name, fence, + pr_info("fence timeout on %s [%pK] after %dms\n", fence->name, fence, jiffies_to_msecs(timeout)); sync_dump(); // [MTK] {{{ -- cgit v1.2.3