From d3d9966d79cc7d6d11124302dd06b7d7522e7305 Mon Sep 17 00:00:00 2001 From: Phil Sutter Date: Tue, 29 Jan 2019 18:12:15 +0100 Subject: [PATCH] flowtable: Add missing break Bugzilla: https://bugzilla.redhat.com/show_bug.cgi?id=1661327 Upstream Status: libnftnl commit 404ef7222d055 commit 404ef7222d055aacdbd4d73dc0d8731fa8f6cbe4 Author: Phil Sutter Date: Thu Dec 20 21:03:28 2018 +0100 flowtable: Add missing break In nftnl_flowtable_set_data(), when setting flowtable size, the switch() case fell through and the same value was copied into ft_flags field. This can't be right. Fixes: 41fe3d38ba34b ("flowtable: support for flags") Signed-off-by: Phil Sutter Signed-off-by: Pablo Neira Ayuso --- src/flowtable.c | 1 + 1 file changed, 1 insertion(+) diff --git a/src/flowtable.c b/src/flowtable.c index c1ddae4..aa6ce59 100644 --- a/src/flowtable.c +++ b/src/flowtable.c @@ -163,6 +163,7 @@ int nftnl_flowtable_set_data(struct nftnl_flowtable *c, uint16_t attr, break; case NFTNL_FLOWTABLE_SIZE: memcpy(&c->size, data, sizeof(c->size)); + break; case NFTNL_FLOWTABLE_FLAGS: memcpy(&c->ft_flags, data, sizeof(c->ft_flags)); break; -- 1.8.3.1