upstream/bind9
1
0
Fork 0
mirror of https://github.com/isc-projects/bind9.git synced 2026-05-28 04:34:54 -04:00
Code Issues Projects Releases Packages Wiki Activity Actions

fix: test: Handle large query IDs in xfer/ans5 properly

Browse source 
Previously, the server would crash if it received a query with an ID
close to 65535 in the badmessageid case, as adding 50 to it would not
fit in uint16.

This was an oversight in porting it from Perl to Python in
f9ed3650ac.

Fixes #6025.

Merge branch 'stepan/fix-xfer-large-qid' into 'main'

See merge request isc-projects/bind9!12097
This commit is contained in:
Štěpán Balážik 2026-05-26 13:49:04 +00:00
commit cdad654ef8
1 changed files with 1 additions and 1 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

2
bin/tests/system/xfer/ans5/ans.py
View file

@ -154,7 +154,7 @@ class Add50ToMessageIdFromSecondResponse(ResponseHandlerWrapper):
assert isinstance(
response_action, DnsResponseSend
), "Add50ToMessageIdFromSecondResponse can only wrap handlers that yield DnsResponseSend from the second response onward"
response_action.response.id += 50
response_action.response.id = (response_action.response.id + 50) % 65536
else:
self._first_yielded = True