diff options
| author | Mark Johnston <markj@FreeBSD.org> | 2023-12-26 01:41:32 +0000 |
|---|---|---|
| committer | Mark Johnston <markj@FreeBSD.org> | 2024-01-02 01:12:45 +0000 |
| commit | 3aa79121f2b8cb070ea9b8a89e6ca8ca967a16f5 (patch) | |
| tree | 9b2bf710495c69c50442f3c7198f486961b75ff3 /sys/kern/kern_sendfile.c | |
| parent | 8276e3d3f8f3cb5ed5258ea39a9423bfb38e3bdb (diff) | |
sendfile: Explicitly ignore errors from copyout()
There is a documented bug in sendfile.2 which notes that sendfile(2)
does not raise an error if it fails to copy out the number of bytes
written. Explicitly ignore the error from copyout() calls in
preparation for annotating copyout() with __result_use_check.
Reviewed by: glebius, kib
MFC after: 1 week
Differential Revision: https://reviews.freebsd.org/D43129
(cherry picked from commit d0adc2f283ad5db6b568ca533a056c9f635551cd)
Diffstat (limited to 'sys/kern/kern_sendfile.c')
| -rw-r--r-- | sys/kern/kern_sendfile.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sys/kern/kern_sendfile.c b/sys/kern/kern_sendfile.c index 8b7a2e7e0ce7..2a34a34067d1 100644 --- a/sys/kern/kern_sendfile.c +++ b/sys/kern/kern_sendfile.c @@ -1326,7 +1326,7 @@ sendfile(struct thread *td, struct sendfile_args *uap, int compat) fdrop(fp, td); if (uap->sbytes != NULL) - copyout(&sbytes, uap->sbytes, sizeof(off_t)); + (void)copyout(&sbytes, uap->sbytes, sizeof(off_t)); out: free(hdr_uio, M_IOV); |