pflow: observation domain is an unsigned integer

Ensure we print it as such, rather than as a signed integer, as that
would lead to confusion.

Reported by:	Jim Pingle <jimp@netgate.com>
Sponsored by:	Rubicon Communications, LLC ("Netgate")
This commit is contained in:
Kristof Provost 2024-01-22 17:34:44 +01:00
parent 8297ff13fb
commit 484e977f24
2 changed files with 26 additions and 1 deletions

View file

@ -340,7 +340,7 @@ get(int id)
if (! snl_parse_nlmsg(&ss, hdr, &get_parser, &g))
continue;
printf("pflow%d: version %d domain %d", g.id, g.version, g.obs_dom);
printf("pflow%d: version %d domain %u", g.id, g.version, g.obs_dom);
print_sockaddr(" src ", &g.src.storage);
print_sockaddr(" dst ", &g.dst.storage);
printf("\n");

View file

@ -282,6 +282,30 @@ rule_cleanup()
pft_cleanup
}
atf_test_case "obs_dom" "cleanup"
obs_dom_head()
{
atf_set descr 'Test configuring observation domain values'
atf_set require.user root
}
obs_dom_body()
{
pflow_init
vnet_mkjail alcatraz
pflow=$(jexec alcatraz pflowctl -c)
jexec alcatraz pflowctl -s ${pflow} domain 2300000000
atf_check -o match:".*domain 2300000000.*" -s exit:0 \
jexec alcatraz pflowctl -l
}
obs_dom_cleanup()
{
pft_cleanup
}
atf_init_test_cases()
{
atf_add_test_case "basic"
@ -289,4 +313,5 @@ atf_init_test_cases()
atf_add_test_case "v6"
atf_add_test_case "nat"
atf_add_test_case "rule"
atf_add_test_case "obs_dom"
}