netlink_delinearize: remove obsolete fixme

The FIXME was related to exclusion of string types from cmp length checks.
Since with fixed sized helper names the last case where this could happen
is gone, remove the FIXME and perform length checks on strings as well.

Signed-off-by: Patrick McHardy <kaber@trash.net>

diff --git a/src/netlink_delinearize.c b/src/netlink_delinearize.c
index ec1a9646eabfcf3978fcd1a52751a5006e0acfbd..b23d5875a8d8314040ef4b48d70c441b101d62ad 100644 (file)
--- a/src/netlink_delinearize.c
+++ b/src/netlink_delinearize.c
@@ -166,9 +166,7 @@ static void netlink_parse_cmp(struct netlink_parse_ctx *ctx,
        nld.value = nft_rule_expr_get(nle, NFT_EXPR_CMP_DATA, &nld.len);
        right = netlink_alloc_value(loc, &nld);
 
-       // FIXME
-       if (left->len && left->dtype && left->dtype->type != TYPE_STRING &&
-           left->len != right->len)
+       if (left->len != right->len)
                return netlink_error(ctx, loc,
                                     "Relational expression size mismatch");