diff options
| author | Tsukasa OI <research_trasio@irq.a4lg.com> | 2022-10-12 12:22:12 +0000 |
|---|---|---|
| committer | Tsukasa OI <research_trasio@irq.a4lg.com> | 2022-10-12 13:21:31 +0000 |
| commit | 4bd531c7ffda1df83b6e54ce11f28a0821e3d59e (patch) | |
| tree | c701f0e83b9188c93eba854ebeac3b15f41158ab /sim/ppc | |
| parent | 182421c9d2eea8c4877d983a2124e591f0aca710 (diff) | |
| download | binutils-gdb-4bd531c7ffda1df83b6e54ce11f28a0821e3d59e.tar.gz | |
sim/ppc: Fix core_find_mapping diagnostics
Because "%p" is the pointer conversion specifier to print a pointer in an
implementation-defined manner, the result with format string containing
"0x%p" can be strange. For instance, core_map_find_mapping prints error
containing "0x0x...." (processor is not NULL) or "0x(null)" (processor is
NULL) on glibc.
This commit replaces "0x%p" with "%p" to prevent unpredictable behavior.
Diffstat (limited to 'sim/ppc')
| -rw-r--r-- | sim/ppc/corefile.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sim/ppc/corefile.c b/sim/ppc/corefile.c index b2faef58c5b..ee4e2cf733e 100644 --- a/sim/ppc/corefile.c +++ b/sim/ppc/corefile.c @@ -292,7 +292,7 @@ core_map_find_mapping(core_map *map, mapping = mapping->next; } if (abort) - error("core_find_mapping() - access to unmaped address, attach a default map to handle this - addr=0x%x nr_bytes=0x%x processor=0x%p cia=0x%x\n", + error("core_find_mapping() - access to unmaped address, attach a default map to handle this - addr=0x%x nr_bytes=0x%x processor=%p cia=0x%x\n", addr, nr_bytes, (void *) processor, cia); return NULL; } |