summaryrefslogtreecommitdiffstats
path: root/sys/dev/isci/isci_task_request.c
diff options
context:
space:
mode:
authorjhb <jhb@FreeBSD.org>2016-11-04 04:01:59 +0000
committerjhb <jhb@FreeBSD.org>2016-11-04 04:01:59 +0000
commit199d3f7cf9e40bc51e2187678ca6a03bf83b2a95 (patch)
tree975845009e0d835608fc3a877ed68c534b5b0c4e /sys/dev/isci/isci_task_request.c
parent642b486bea5128a6288c2023c0d57d9a78720184 (diff)
downloadFreeBSD-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
OpenPOWER on IntegriCloud