diff options
author | Phil Sutter <phil@nwl.cc> | 2020-11-17 11:38:27 +0100 |
---|---|---|
committer | Phil Sutter <phil@nwl.cc> | 2020-11-17 12:52:09 +0100 |
commit | 55b7c71dce7144f4dc0297c17abf0f04879ee247 (patch) | |
tree | 2f49bbbe63bb05598e9e7f396f4c98075a185c19 /iptables/xtables-eb.c | |
parent | 83ee6e179829461572be6583ce6e83f68e636eb6 (diff) |
ebtables: Fix for broken chain renaming
Loading extensions pollutes 'errno' value, hence before using it to
indicate failure it should be sanitized. This was done by the called
function before the parsing/netlink split and not migrated by accident.
Move it into calling code to clarify the connection.
Fixes: a7f1e208cdf9c ("nft: split parsing from netlink commands")
Signed-off-by: Phil Sutter <phil@nwl.cc>
Diffstat (limited to 'iptables/xtables-eb.c')
-rw-r--r-- | iptables/xtables-eb.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/iptables/xtables-eb.c b/iptables/xtables-eb.c index 6641a21a..5e4184b8 100644 --- a/iptables/xtables-eb.c +++ b/iptables/xtables-eb.c @@ -853,6 +853,7 @@ int do_commandeb(struct nft_handle *h, int argc, char *argv[], char **table, else if (strchr(argv[optind], ' ') != NULL) xtables_error(PARAMETER_PROBLEM, "Use of ' ' not allowed in chain names"); + errno = 0; ret = nft_cmd_chain_user_rename(h, chain, *table, argv[optind]); if (ret != 0 && errno == ENOENT) |