fix: test: unit test tsig_badsig was not being called

Additionally, the test was badly implemented. This has been fixed.

Closes #5712

Merge branch '5712-unit-test-tsig_badsig-was-not-being-called' into 'main'

See merge request isc-projects/bind9!11478
This commit is contained in:
Mark Andrews 2026-01-24 02:27:34 +11:00
commit 7878281048

View file

@ -419,7 +419,16 @@ tsig_tcp(isc_stdtime_t now, isc_result_t expected_result, bool mangle_sig) {
assert_int_equal(msg->verified_sig, 1);
assert_int_equal(msg->tsigstatus, dns_tsigerror_badtime);
break;
case DNS_R_TSIGVERIFYFAILURE:
assert_int_equal(result, DNS_R_TSIGVERIFYFAILURE);
assert_int_equal(msg->verified_sig, 0);
assert_int_equal(msg->tsigstatus, dns_tsigerror_badsig);
break;
default:
if (debug) {
fprintf(stderr, "# result = %s\n",
isc_result_totext(result));
}
UNREACHABLE();
}
@ -471,7 +480,7 @@ ISC_RUN_TEST_IMPL(tsig_badtime) {
}
ISC_RUN_TEST_IMPL(tsig_badsig) {
tsig_tcp(isc_stdtime_now(), DNS_R_TSIGERRORSET, true);
tsig_tcp(isc_stdtime_now(), DNS_R_TSIGVERIFYFAILURE, true);
}
/* Tests the dns__tsig_algvalid function */
@ -490,9 +499,10 @@ ISC_RUN_TEST_IMPL(algvalid) {
}
ISC_TEST_LIST_START
ISC_TEST_ENTRY(tsig_tcp)
ISC_TEST_ENTRY(tsig_badtime)
ISC_TEST_ENTRY(algvalid)
ISC_TEST_ENTRY(tsig_badsig)
ISC_TEST_ENTRY(tsig_badtime)
ISC_TEST_ENTRY(tsig_tcp)
ISC_TEST_LIST_END
ISC_TEST_MAIN