once cvs's flow-control mechanism has kicked in, stop reading from
authorjcs <jcs@openbsd.org>
Fri, 14 Apr 2017 19:39:59 +0000 (19:39 +0000)
committerjcs <jcs@openbsd.org>
Fri, 14 Apr 2017 19:39:59 +0000 (19:39 +0000)
commit94a18cb59f9d5469fbc350d54305ac5e37bf9043
tree04cb7986284991c7972471569298839deb65a776
parent97970a6243cf0d263a65c4fd1bd4c20caa2ef554
once cvs's flow-control mechanism has kicked in, stop reading from
its local reader until memory usage goes back down below its
watermark.

during a checkout of a tree with big files (like www) from a fast
reader (disk) and a slow writer (net), the amount of data being
buffered can back up enough that cvs tries to allocate too many
buffers and hits its resource limit, causing death.

cvs's flow-control mechanism properly detects this early on, but the
message sent to the reader process to stop sending data takes too
long to process.

take more aggressive action and just stop reading from the reader
until the writer has ejected enough data that it can start re-using
its already-allocated buffers instead of allocating new ones.

ok deraadt
gnu/usr.bin/cvs/src/server.c