diff options
| author | Tom Tromey <tom@tromey.com> | 2025-09-04 14:12:52 -0600 |
|---|---|---|
| committer | Tom Tromey <tom@tromey.com> | 2025-09-08 08:44:48 -0600 |
| commit | 8b50cc76f19d129b3887dbcb58f8286a7b36099f (patch) | |
| tree | c59f10b87df27fb3ec5493414190f382a037895e /gdb | |
| parent | e447f3a122cd90d99582475c9a18f7b4c23468dd (diff) | |
| download | binutils-8b50cc76f19d129b3887dbcb58f8286a7b36099f.zip binutils-8b50cc76f19d129b3887dbcb58f8286a7b36099f.tar.gz binutils-8b50cc76f19d129b3887dbcb58f8286a7b36099f.tar.bz2 | |
Change return value of _bfd_mmap_temporary
_bfd_mmap_temporary will return MAP_FAILED in some cases, but only one
caller was prepared for this. This caused a couple of issues in gdb.
This patch chnages _bfd_mmap_temporary to always return NULL on
failure. It also changes the logic to fall back to bfd_malloc if mmap
fails.
I took the opportunity to change some "(void *) -1" to MAP_FAILED
where appropriate.
Tested by "make check" in binutils, and also by applying a hack from
bug 33354 to gdb and testing something approximating the original
report.
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=32542
Bug: https://sourceware.org/bugzilla/show_bug.cgi?id=33354
Diffstat (limited to 'gdb')
0 files changed, 0 insertions, 0 deletions