diff options
author | jhb <jhb@FreeBSD.org> | 2016-11-04 04:01:59 +0000 |
---|---|---|
committer | jhb <jhb@FreeBSD.org> | 2016-11-04 04:01:59 +0000 |
commit | 199d3f7cf9e40bc51e2187678ca6a03bf83b2a95 (patch) | |
tree | 975845009e0d835608fc3a877ed68c534b5b0c4e /sys/dev/isci/isci_task_request.c | |
parent | 642b486bea5128a6288c2023c0d57d9a78720184 (diff) | |
download | FreeBSD-src-199d3f7cf9e40bc51e2187678ca6a03bf83b2a95.zip FreeBSD-src-199d3f7cf9e40bc51e2187678ca6a03bf83b2a95.tar.gz |
MFC 301932: Use sbused() instead of sbspace() to avoid signed issues.
Inserting a full mbuf with an external cluster into the socket buffer
resulted in sbspace() returning -MLEN. However, since sb_hiwat is
unsigned, the -MLEN value was converted to unsigned in comparisons. As a
result, the socket buffer was never autosized. Note that sb_lowat is signed
to permit direct comparisons with sbspace(), but sb_hiwat is unsigned.
Follow suit with what tcp_output() does and compare the value of sbused()
with sb_hiwat instead.
Note: Since stable/10 does not include sbused(), this uses sb->sb_cc
instead.
Sponsored by: Chelsio Communications
Diffstat (limited to 'sys/dev/isci/isci_task_request.c')
0 files changed, 0 insertions, 0 deletions