diff options
| author | Mark Kettenis <kettenis@gnu.org> | 2014-02-12 14:51:19 +0100 |
|---|---|---|
| committer | Mark Kettenis <kettenis@gnu.org> | 2014-02-12 14:51:19 +0100 |
| commit | 493443a47f514251f12e08223b2c56f0fed69015 (patch) | |
| tree | 13b9bf19a9039cd7d735e0ae955960bb97a72433 /gdb/inf-ptrace.c | |
| parent | 706d088346930eeee11befa93330375164e175b9 (diff) | |
FIX EOF detection in PT_IO-based to_xfer_partial implementation.
At least on OpenBSD PT_IO/PIOD_READ_AUXV can return sucessfully without
transferring any bytes. Arguably a kernel bug, but interpreting this as EOF
seems sensible.
gdb/ChangeLog:
* inf-ptrace.c (inf_ptrace_xfer_partial): Return TARGET_XFER_EOF
if a PT_IO ptrace request returns sucessfully but indicates that 0
bytes were transferred.
Diffstat (limited to 'gdb/inf-ptrace.c')
| -rw-r--r-- | gdb/inf-ptrace.c | 8 |
1 files changed, 4 insertions, 4 deletions
diff --git a/gdb/inf-ptrace.c b/gdb/inf-ptrace.c index 1ab6b0beee..1269d24b65 100644 --- a/gdb/inf-ptrace.c +++ b/gdb/inf-ptrace.c @@ -489,9 +489,9 @@ inf_ptrace_xfer_partial (struct target_ops *ops, enum target_object object, errno = 0; if (ptrace (PT_IO, pid, (caddr_t)&piod, 0) == 0) { - *xfered_len = piod.piod_len; /* Return the actual number of bytes read or written. */ - return TARGET_XFER_OK; + *xfered_len = piod.piod_len; + return (piod.piod_len == 0) ? TARGET_XFER_EOF : TARGET_XFER_OK; } /* If the PT_IO request is somehow not supported, fallback on using PT_WRITE_D/PT_READ_D. Otherwise we will return zero @@ -595,9 +595,9 @@ inf_ptrace_xfer_partial (struct target_ops *ops, enum target_object object, errno = 0; if (ptrace (PT_IO, pid, (caddr_t)&piod, 0) == 0) { - *xfered_len = piod.piod_len; /* Return the actual number of bytes read or written. */ - return TARGET_XFER_OK; + *xfered_len = piod.piod_len; + return (piod.piod_len == 0) ? TARGET_XFER_EOF : TARGET_XFER_OK; } } #endif |