From db1869971dde5b42884f128ca6ce0a2a751ff854 Mon Sep 17 00:00:00 2001 From: Sven Eckelmann Date: Thu, 6 Dec 2018 09:21:15 +0100 Subject: [PATCH] batctl: Fix parsing of optional debug table command parameters The commands which should have no support for -t/-u/-m/-i were allowed to accept these parameters but commands which should have accepted them were denying them. Fixes: 8936141bf038 ("batctl: upgrade package to latest release 2018.4") Signed-off-by: Sven Eckelmann --- batctl/Makefile | 2 +- ...ng-of-optional-debug-table-command-p.patch | 57 +++++++++++++++++++ 2 files changed, 58 insertions(+), 1 deletion(-) create mode 100644 batctl/patches/0001-batctl-Fix-parsing-of-optional-debug-table-command-p.patch diff --git a/batctl/Makefile b/batctl/Makefile index 045ecde..d38874e 100644 --- a/batctl/Makefile +++ b/batctl/Makefile @@ -10,7 +10,7 @@ include $(TOPDIR)/rules.mk PKG_NAME:=batctl PKG_VERSION:=2018.4 -PKG_RELEASE:=0 +PKG_RELEASE:=1 PKG_HASH:=e43827a5e868b4e134e77ca04da989fde1981463166bf1b6f2053acc3edd6257 PKG_SOURCE:=$(PKG_NAME)-$(PKG_VERSION).tar.gz diff --git a/batctl/patches/0001-batctl-Fix-parsing-of-optional-debug-table-command-p.patch b/batctl/patches/0001-batctl-Fix-parsing-of-optional-debug-table-command-p.patch new file mode 100644 index 0000000..559d2ed --- /dev/null +++ b/batctl/patches/0001-batctl-Fix-parsing-of-optional-debug-table-command-p.patch @@ -0,0 +1,57 @@ +From: Sven Eckelmann +Date: Thu, 6 Dec 2018 07:31:59 +0100 +Subject: [PATCH] batctl: Fix parsing of optional debug table command parameters + +The commands which should have no support for -t/-u/-m/-i were allowed to +accept these parameters but commands which should have accepted them were +denying them. + +Fixes: cd2b4047ead9 ("batctl: Convert debug table to command infrastructure") +Reported-by: Andreas Ziegler +Signed-off-by: Sven Eckelmann + +Forwarded: https://patchwork.open-mesh.org/patch/17667/ +--- + debug.c | 8 ++++---- + 1 file changed, 4 insertions(+), 4 deletions(-) + +diff --git a/debug.c b/debug.c +index 2979dff99af0b1c98f56931e8650e298dd9eac9b..316312f7162dbe0dc38ced2df1444887482e7bad 100644 +--- a/debug.c ++++ b/debug.c +@@ -89,7 +89,7 @@ int handle_debug_table(struct state *state, int argc, char **argv) + } + break; + case 't': +- if (debug_table->option_watch_interval) { ++ if (!debug_table->option_watch_interval) { + fprintf(stderr, "Error - unrecognised option '-%c'\n", optchar); + debug_table_usage(state); + return EXIT_FAILURE; +@@ -105,7 +105,7 @@ int handle_debug_table(struct state *state, int argc, char **argv) + read_opt |= SKIP_HEADER; + break; + case 'u': +- if (debug_table->option_unicast_only) { ++ if (!debug_table->option_unicast_only) { + fprintf(stderr, "Error - unrecognised option '-%c'\n", optchar); + debug_table_usage(state); + return EXIT_FAILURE; +@@ -114,7 +114,7 @@ int handle_debug_table(struct state *state, int argc, char **argv) + read_opt |= UNICAST_ONLY; + break; + case 'm': +- if (debug_table->option_multicast_only) { ++ if (!debug_table->option_multicast_only) { + fprintf(stderr, "Error - unrecognised option '-%c'\n", optchar); + debug_table_usage(state); + return EXIT_FAILURE; +@@ -123,7 +123,7 @@ int handle_debug_table(struct state *state, int argc, char **argv) + read_opt |= MULTICAST_ONLY; + break; + case 'i': +- if (debug_table->option_orig_iface) { ++ if (!debug_table->option_orig_iface) { + fprintf(stderr, "Error - unrecognised option '-%c'\n", optchar); + debug_table_usage(state); + return EXIT_FAILURE; -- 2.30.2