diff options
author | David Gibson <david@gibson.dropbear.id.au> | 2023-03-27 14:56:33 +1100 |
---|---|---|
committer | Stefano Brivio <sbrivio@redhat.com> | 2023-03-29 13:46:58 +0200 |
commit | 085672f77cce3e91a116ac39cff581bca64ba50a (patch) | |
tree | 84e996dc845f193f57fd54857137081f35c56d84 /test/build/clang_tidy | |
parent | 1ee2f7cada9e6f739a00d39bb9821f1ce3493d92 (diff) | |
download | passt-085672f77cce3e91a116ac39cff581bca64ba50a.tar passt-085672f77cce3e91a116ac39cff581bca64ba50a.tar.gz passt-085672f77cce3e91a116ac39cff581bca64ba50a.tar.bz2 passt-085672f77cce3e91a116ac39cff581bca64ba50a.tar.lz passt-085672f77cce3e91a116ac39cff581bca64ba50a.tar.xz passt-085672f77cce3e91a116ac39cff581bca64ba50a.tar.zst passt-085672f77cce3e91a116ac39cff581bca64ba50a.zip |
tcp: Clarify allowed state for tcp_data_from_tap()
Comments suggest that this should only be called for an ESTABLISHED
connection. However, it's non-trivial to ascertain that from the actual
control flow in the caller. Add an ASSERT() to make it very clear that
this is only called in ESTABLISHED state.
In fact, there were some circumstances where it could be called on a CLOSED
connection. In a sense that is "established", but with that assert this
does require specific (trivial) handling to avoid a spurious abort().
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
Signed-off-by: Stefano Brivio <sbrivio@redhat.com>
Diffstat (limited to 'test/build/clang_tidy')
0 files changed, 0 insertions, 0 deletions