]> git.ipfire.org Git - thirdparty/bind9.git/commit
TCP: remove wrong INSIST(csock->recv_cb != NULL)
authorArtem Boldariev <artem@boldariev.com>
Mon, 15 Jan 2024 15:03:50 +0000 (17:03 +0200)
committerArtem Boldariev <artem@boldariev.com>
Tue, 16 Jan 2024 13:01:26 +0000 (15:01 +0200)
commitdffb11f2c09becac47b7ace628891abdcde19e00
tree6e05e9573948198a1327db4eabdae8212f77171b
parent84e7e5d5df7a4fcb71f51ba229104b03cfe439c9
TCP: remove wrong INSIST(csock->recv_cb != NULL)

This commit removes wrong INSIST() condition as the assumption that if
'csock->recv_cb != NULL' iff 'csock->statichandle != NULL' is wrong.

There is no direct relation between 'csock->statichandle' and
'csock->recv_cb', as 'csock->statichandle' gets set when allocating a
handle regardless of 'csock->recv_cb' not being NULL, as it is
possible to attach to the handle without starting a read operation (at
the very least, it is correct to start writing before reading).

That condition made `cipher-suites` system test fail with crash on
some platforms in FIPS mode (namely, Oracle Linux 9) despite not being
related to FIPS at all.
lib/isc/netmgr/tcp.c