]> CyberLeo.Net >> Repos - FreeBSD/FreeBSD.git/commit
In kern_sendfile() adjust byte accounting of the file sending loop to
authorAndre Oppermann <andre@FreeBSD.org>
Sat, 19 May 2007 20:50:59 +0000 (20:50 +0000)
committerAndre Oppermann <andre@FreeBSD.org>
Sat, 19 May 2007 20:50:59 +0000 (20:50 +0000)
commit1c9dbd15679d52f84e31936165b99e7823841be4
tree06d34d5d05c502d5ba8f23a3547c4d6d87549e26
parent98a0900ab4692298e62eb196bf847dfa3aadfbc6
In kern_sendfile() adjust byte accounting of the file sending loop to
ignore the size of any headers that were passed with the sendfile(2)
system call.  Otherwise the file sent will be truncated by the header
size if the nbytes parameter was provided.  The bug doesn't show up
when either nbytes is zero, meaning send the whole file, or no header
iovec is provided.

Resolve a potential error aliasing of errors from the VM and sf_buf
parts and the protocol send parts where an error of the latter over-
writes one of the former.

Update comments.

The byte accounting bug wasn't seen in earlier because none of the popular
sendfile(2) consumers, Apache, lighttpd and our ftpd(8) use it in modes
that trigger it.  The varnish HTTP proxy makes full use of it and exposed
the problem.

Bug found by: phk
Tested by: phk
sys/kern/uipc_syscalls.c