diff options
author | markj <markj@FreeBSD.org> | 2015-10-03 19:37:41 +0000 |
---|---|---|
committer | markj <markj@FreeBSD.org> | 2015-10-03 19:37:41 +0000 |
commit | f73def85e631574d0501f0eeba5a69fb1c20f6a8 (patch) | |
tree | 36aef3c08a4018101db47ebb776c34296c00e4a6 /sys | |
parent | 6ec46fe67c08981d9534fd290a91ee302743c226 (diff) | |
download | FreeBSD-src-f73def85e631574d0501f0eeba5a69fb1c20f6a8.zip FreeBSD-src-f73def85e631574d0501f0eeba5a69fb1c20f6a8.tar.gz |
The return value of posix_fadvise(2) is just an error status, so
sys_posix_fadvise() should simply return the errno (or 0) to syscallenter()
rather than setting a return value.
MFC after: 1 week
Diffstat (limited to 'sys')
-rw-r--r-- | sys/kern/vfs_syscalls.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/sys/kern/vfs_syscalls.c b/sys/kern/vfs_syscalls.c index ce4436a..725a7fd 100644 --- a/sys/kern/vfs_syscalls.c +++ b/sys/kern/vfs_syscalls.c @@ -4663,7 +4663,6 @@ int sys_posix_fadvise(struct thread *td, struct posix_fadvise_args *uap) { - td->td_retval[0] = kern_posix_fadvise(td, uap->fd, uap->offset, - uap->len, uap->advice); - return (0); + return (kern_posix_fadvise(td, uap->fd, uap->offset, uap->len, + uap->advice)); } |