Repository : ssh://git@diktynna/batctl
On branch : master
>---------------------------------------------------------------
commit df8bf5164b6904f61ae0b0db090fb5bb41b4f06d
Author: Leonardo Mörlein <me(a)irrelefant.net>
Date: Wed Apr 8 23:49:03 2020 +0200
batctl: Return EXIT_FAILURE when throughputmeter failed
The command returned a success even an error was shown during the
execution.
$ (sudo batctl tp 77:77:77:77:77:77 && echo true) || echo false
Destination unreachable
true
Instead it should indicate a failure when the kernel replied with a
non-success return_value:
$ (sudo ./batctl tp 77:77:77:77:77:77 && echo true) || echo false
Destination unreachable
false
Fixes: f109b3473f86 ("batctl: introduce throughput meter support")
Signed-off-by: Leonardo Mörlein <me(a)irrelefant.net>
[sven(a)narfation.org: adjusted commit message]
Signed-off-by: Sven Eckelmann <sven(a)narfation.org>
>---------------------------------------------------------------
df8bf5164b6904f61ae0b0db090fb5bb41b4f06d
throughputmeter.c | 1 +
1 file changed, 1 insertion(+)
diff --git a/throughputmeter.c b/throughputmeter.c
index 27f1346..0ec5c31 100644
--- a/throughputmeter.c
+++ b/throughputmeter.c
@@ -465,6 +465,7 @@ static int throughputmeter(struct state *state, int argc, char **argv)
goto out;
}
+ ret = EXIT_FAILURE;
switch (result.return_value) {
case BATADV_TP_REASON_DST_UNREACHABLE:
fprintf(stderr, "Destination unreachable\n");