diff options
| author | Divya Ponnusamy <pdivya@codeaurora.org> | 2016-06-15 16:22:11 +0530 |
|---|---|---|
| committer | Moyster <oysterized@gmail.com> | 2016-11-07 13:44:31 +0100 |
| commit | f2f309d5e265deb273693815f16fa6c41cedf9dc (patch) | |
| tree | 79ad6c65b42cd8f62e03d55b68345c7e6b88e63d | |
| parent | f5f76c228a564fa6f3c9eeb709538116d428d14a (diff) | |
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 <pdivya@codeaurora.org>
Ticket: PORRIDGE-499
| -rw-r--r-- | drivers/staging/android/sync.c | 4 |
1 files 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] {{{ |