userspace/tools: prevent segfaults in dump_shmem when pointer is not valid

dump_many_fields shall check if pointer to dump is valid. It is simpler than checking if all wrs_shm_follow were successful. Signed-off-by: Adam Wujek <adam.wujek@cern.ch>

userspace/tools: prevent segfaults in dump_shmem when pointer is not valid
dump_many_fields shall check if pointer to dump is valid. It is simpler than checking if all wrs_shm_follow were successful. Signed-off-by: Adam Wujek <adam.wujek@cern.ch>
73e010f1 · Adam Wujek · 43fb7aa5 · 73e010f1
Commit 73e010f1 authored Feb 06, 2015 by Adam Wujek 💬
Hide whitespace changes
Inline Side-by-side

Showing with 4 additions and 0 deletions

wrs_dump_shmem.c userspace/tools/wrs_dump_shmem.c +4 -0

No files found.
--- a/userspace/tools/wrs_dump_shmem.c
+++ b/userspace/tools/wrs_dump_shmem.c
@@ -171,6 +171,10 @@ void dump_many_fields(void *addr, struct dump_info *info, int ninfo)
 {
 	int i;

+	if (!addr) {
+		fprintf(stderr, "dump: pointer not valid\n");
+		return;
+	}
 	for (i = 0; i < ninfo; i++)
 		dump_one_field(addr, info + i);
 }