diff options
author | Tom de Vries <tdevries@suse.de> | 2024-11-22 17:44:29 +0100 |
---|---|---|
committer | Tom de Vries <tdevries@suse.de> | 2024-11-22 17:44:29 +0100 |
commit | 658a03e9e85698fa8815915d1a858371ea51da83 (patch) | |
tree | 902c3dec08c0ece2695f535c565eefbb8da790d3 /gdbserver | |
parent | 26522e34802f32406eb653d91a3bbb509f919b30 (diff) | |
download | binutils-658a03e9e85698fa8815915d1a858371ea51da83.zip binutils-658a03e9e85698fa8815915d1a858371ea51da83.tar.gz binutils-658a03e9e85698fa8815915d1a858371ea51da83.tar.bz2 |
[gdbsupport] Add gdb::{waitpid,read,write,close}
We have gdb::handle_eintr, which allows us to rewrite:
...
ssize_t ret;
do
{
errno = 0;
ret = ::write (pipe[1], "+", 1);
}
while (ret == -1 && errno == EINTR);
...
into:
...
ssize_t ret = gdb::handle_eintr (-1, ::write, pipe[1], "+", 1);
...
However, the call to write got a bit mangled, requiring effort to decode it
back to its original form.
Instead, add a new function gdb::write that allows us to write:
...
ssize_t ret = gdb::write (pipe[1], "+", 1);
...
Likewise for waitpid, read and close.
Tested on x86_64-linux.
Diffstat (limited to 'gdbserver')
-rw-r--r-- | gdbserver/netbsd-low.cc | 10 |
1 files changed, 5 insertions, 5 deletions
diff --git a/gdbserver/netbsd-low.cc b/gdbserver/netbsd-low.cc index 4e459e6..04e1035 100644 --- a/gdbserver/netbsd-low.cc +++ b/gdbserver/netbsd-low.cc @@ -222,7 +222,7 @@ netbsd_waitpid (ptid_t ptid, struct target_waitstatus *ourstatus, int options = (target_options & TARGET_WNOHANG) ? WNOHANG : 0; pid_t pid - = gdb::handle_eintr (-1, ::waitpid, ptid.pid (), &status, options); + = gdb::waitpid (ptid.pid (), &status, options); if (pid == -1) perror_with_name (_("Child process unexpectedly missing")); @@ -432,7 +432,7 @@ netbsd_process_target::kill (process_info *process) return -1; int status; - if (gdb::handle_eintr (-1, ::waitpid, pid, &status, 0) == -1) + if (gdb::waitpid (pid, &status, 0) == -1) return -1; mourn (process); return 0; @@ -1123,15 +1123,15 @@ netbsd_qxfer_libraries_svr4 (const pid_t pid, const char *annex, static bool elf_64_file_p (const char *file) { - int fd = gdb::handle_eintr (-1, ::open, file, O_RDONLY); + int fd = gdb::open (file, O_RDONLY); if (fd < 0) perror_with_name (("open")); Elf64_Ehdr header; - ssize_t ret = gdb::handle_eintr (-1, ::read, fd, &header, sizeof (header)); + ssize_t ret = gdb::read (fd, &header, sizeof (header)); if (ret == -1) perror_with_name (("read")); - gdb::handle_eintr (-1, ::close, fd); + gdb::close (fd); if (ret != sizeof (header)) error ("Cannot read ELF file header: %s", file); |