path: root/src/monitor.c
diff options
authorFlorian Westphal <>2019-07-16 19:03:55 +0200
committerFlorian Westphal <>2019-12-16 17:06:18 +0100
commit343a51702656a6476e37cfb84609a82155c7fc5e (patch)
tree10ce1fa0bf256982709d8a89c4a52cb1a8f9cfe3 /src/monitor.c
parent10f114806ccd9d64f9d72eaa813babb04d719688 (diff)
src: store expr, not dtype to track data in sets
This will be needed once we add support for the 'typeof' keyword to handle maps that could e.g. store 'ct helper' "type" values. Instead of: set foo { type ipv4_addr . mark; this would allow set foo { typeof(ip saddr) . typeof(ct mark); (exact syntax TBD). This would be needed to allow sets that store variable-sized data types (string, integer and the like) that can't be used at at the moment. Adding special data types for everything is problematic due to the large amount of different types needed. For anonymous sets, e.g. "string" can be used because the needed size can be inferred from the statement, e.g. 'osf name { "Windows", "Linux }', but in case of named sets that won't work because 'type string' lacks the context needed to derive the size information. With 'typeof(osf name)' the context is there, but at the moment it won't help because the expression is discarded instantly and only the data type is retained. Signed-off-by: Florian Westphal <>
Diffstat (limited to 'src/monitor.c')
1 files changed, 1 insertions, 1 deletions
diff --git a/src/monitor.c b/src/monitor.c
index ea0393cd..d586cfa3 100644
--- a/src/monitor.c
+++ b/src/monitor.c
@@ -401,7 +401,7 @@ static int netlink_events_setelem_cb(const struct nlmsghdr *nlh, int type,
dummyset = set_alloc(monh->loc);
dummyset->key = expr_clone(set->key);
- dummyset->datatype = set->datatype;
+ dummyset->data = set->data;
dummyset->flags = set->flags;
dummyset->init = set_expr_alloc(monh->loc, set);