From 982dfed192b9bcdf24aec84bad7639b28833bc0e Mon Sep 17 00:00:00 2001 From: Alejandro Colomar Date: Sat, 12 Sep 2020 01:14:11 +0200 Subject: [PATCH] mprotect.2: Use "%p" rather than casting to 'long' when printing pointer values Signed-off-by: Alejandro Colomar Signed-off-by: Michael Kerrisk --- man2/mprotect.2 | 5 ++--- 1 file changed, 2 insertions(+), 3 deletions(-) diff --git a/man2/mprotect.2 b/man2/mprotect.2 index 47d250303..885238a8d 100644 --- a/man2/mprotect.2 +++ b/man2/mprotect.2 @@ -326,8 +326,7 @@ handler(int sig, siginfo_t *si, void *unused) Nevertheless, we use printf() here as a simple way of showing that the handler was called. */ - printf("Got SIGSEGV at address: %#lx\en", - (long) si\->si_addr); + printf("Got SIGSEGV at address: %p\en", si\->si_addr); exit(EXIT_FAILURE); } @@ -354,7 +353,7 @@ main(int argc, char *argv[]) if (buffer == NULL) handle_error("memalign"); - printf("Start of region: %#lx\en", (long) buffer); + printf("Start of region: %p\en", buffer); if (mprotect(buffer + pagesize * 2, pagesize, PROT_READ) == \-1)