3900
58%
master: 66%

Ran 21 Feb 2019 04:35PM UTC

Jobs 1

Files 422

Run time 23min

Badge

Embed ▾

pending completion

Build # 3900

Build Type

push

travis-ci

Committed by

nmathewson

Commit Message

kist: Don't write above the highwater outbuf mark

KIST works by computing how much should be allowed to write to the kernel for
a given socket, and then it writes that amount to the outbuf.

The problem is that it could be possible that the outbuf already has lots of
data in it from a previous scheduling round (because the kernel is full/busy
and Tor was not able to flush the outbuf yet). KIST ignores that the outbuf
has been filling (is above its "highwater") and writes more anyway. The end
result is that the outbuf length would exceed INT_MAX, hence causing an
assertion error and a corresponding "Bug()" message to get printed to the
logs.

This commit makes it for KIST to take into account the outbuf length when
computing the available space.

Bug found and patch by Rob Jansen.

Closes #29168. TROVE-2019-001.

Signed-off-by: David Goulet <dgoulet@torproject.org>

Run Details

40285 of 69526 relevant lines covered (57.94%)

27593.37 hits per line

Jobs

ID	Job ID	Ran	Files	Coverage
5	3900.5 (COVERAGE_OPTIONS="--enable-coverage" HARDENING_OPTIONS="")	21 Feb 2019 04:35PM UTC	0	57.94	Travis Job 3900.5

Source Files on build 3900

Detailed source file information is not available for this build.

torproject / tor / 3900
58%
master: 66%

README BADGES
x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 3900

torproject / tor / 3900 58% master: 66%

README BADGES x

Markdown

Textile

RDoc

HTML

Rst

Jobs

Source Files on build 3900

torproject / tor / 3900
58%
master: 66%

README BADGES
x