From 228e8b1746270ee5c4a41b671a01369c75622587 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Sat, 26 Dec 2020 19:44:48 +0100 Subject: set_elem: Fix printing of verdict map elements Elements' data was printed as type DATA_VALUE no matter the actual type. For verdicts, this meant no printing at all (because reg->len is either zero or garbage). To fix this, nftnl_set_elem_snprintf_default() needs type info held in struct nftnl_set. Pass it via parameter to that function, make it non-static and call it from nftnl_set_snprintf_default() instead of the generic nftnl_set_elem_snprintf(). This way no changes have to be done to exported functions, also the output type is already defined when nftnl_set_snprintf_default() runs so checking type value again is pointless. Signed-off-by: Phil Sutter --- include/set_elem.h | 4 ++++ src/set.c | 4 ++-- src/set_elem.c | 12 ++++++++---- 3 files changed, 14 insertions(+), 6 deletions(-) diff --git a/include/set_elem.h b/include/set_elem.h index 9239557..503dced 100644 --- a/include/set_elem.h +++ b/include/set_elem.h @@ -20,4 +20,8 @@ struct nftnl_set_elem { } user; }; +int nftnl_set_elem_snprintf_default(char *buf, size_t size, + const struct nftnl_set_elem *e, + enum nft_data_types dtype); + #endif diff --git a/src/set.c b/src/set.c index 8c5025d..a21df1f 100644 --- a/src/set.c +++ b/src/set.c @@ -829,8 +829,8 @@ static int nftnl_set_snprintf_default(char *buf, size_t size, ret = snprintf(buf + offset, remain, "\t"); SNPRINTF_BUFFER_SIZE(ret, remain, offset); - ret = nftnl_set_elem_snprintf(buf + offset, remain, elem, type, - flags); + ret = nftnl_set_elem_snprintf_default(buf + offset, remain, + elem, s->data_type); SNPRINTF_BUFFER_SIZE(ret, remain, offset); } diff --git a/src/set_elem.c b/src/set_elem.c index 8f634e7..ad528e2 100644 --- a/src/set_elem.c +++ b/src/set_elem.c @@ -698,9 +698,12 @@ int nftnl_set_elem_parse_file(struct nftnl_set_elem *e, enum nftnl_parse_type ty return -1; } -static int nftnl_set_elem_snprintf_default(char *buf, size_t size, - const struct nftnl_set_elem *e) +int nftnl_set_elem_snprintf_default(char *buf, size_t size, + const struct nftnl_set_elem *e, + enum nft_data_types dtype) { + int dregtype = (dtype == NFT_DATA_VERDICT) ? DATA_VERDICT : DATA_VALUE; + int ret, remain = size, offset = 0, i; ret = snprintf(buf, remain, "element "); @@ -726,7 +729,7 @@ static int nftnl_set_elem_snprintf_default(char *buf, size_t size, ret = nftnl_data_reg_snprintf(buf + offset, remain, &e->data, NFTNL_OUTPUT_DEFAULT, - DATA_F_NOPFX, DATA_VALUE); + DATA_F_NOPFX, dregtype); SNPRINTF_BUFFER_SIZE(ret, remain, offset); ret = snprintf(buf + offset, remain, "%u [end]", e->set_elem_flags); @@ -760,7 +763,8 @@ static int nftnl_set_elem_cmd_snprintf(char *buf, size_t size, switch(type) { case NFTNL_OUTPUT_DEFAULT: - ret = nftnl_set_elem_snprintf_default(buf + offset, remain, e); + ret = nftnl_set_elem_snprintf_default(buf + offset, remain, e, + NFT_DATA_VALUE); SNPRINTF_BUFFER_SIZE(ret, remain, offset); break; case NFTNL_OUTPUT_XML: -- cgit v1.2.3