Message ID | 20220407201411.22486-1-a@unstable.cc |
---|---|
State | Accepted |
Headers | show |
Series | [Openvpn-devel] networking_iproute2: don't pass M_WARN to openvpn_execve_check() | expand |
Acked-by: Gert Doering <gert@greenie.muc.de> Good catch :-) Your patch introduced a whitespace change on the second line. I asked uncrustify, uncrustify said "previous indent is correct", so only changing the first line. Your patch has been applied to the master and release/2.5 branch. commit 545f6bdea30e04a09b118c4e1faf6523544fd935 (master) commit 113b6c4192ecb927997b187de023aec71712f312 (release/2.5) Author: Antonio Quartulli Date: Thu Apr 7 22:14:11 2022 +0200 networking_iproute2: don't pass M_WARN to openvpn_execve_check() Signed-off-by: Antonio Quartulli <a@unstable.cc> Acked-by: Gert Doering <gert@greenie.muc.de> Message-Id: <20220407201411.22486-1-a@unstable.cc> URL: https://www.mail-archive.com/openvpn-devel@lists.sourceforge.net/msg24090.html Signed-off-by: Gert Doering <gert@greenie.muc.de> -- kind regards, Gert Doering
diff --git a/src/openvpn/networking_iproute2.c b/src/openvpn/networking_iproute2.c index 3ca2bb35..02fd3cb6 100644 --- a/src/openvpn/networking_iproute2.c +++ b/src/openvpn/networking_iproute2.c @@ -105,8 +105,8 @@ net_addr_ll_set(openvpn_net_ctx_t *ctx, const openvpn_net_iface_t *iface, iproute_path, MAC_PRINT_ARG(addr), iface); argv_msg(M_INFO, &argv); - if (!openvpn_execve_check(&argv, ctx->es, M_WARN, - "Linux ip link set addr failed")) + if (!openvpn_execve_check(&argv, ctx->es, 0, + "Linux ip link set addr failed")) { ret = -1; }
openvpn_execve_check() expects a set of flags as third argument and not a loglevel. For this reason, if no FATAL behaviour is expected, we should simply pass 0. openvpn_execve_check() will then pick the appropriate loglevel on its own. Signed-off-by: Antonio Quartulli <a@unstable.cc> --- src/openvpn/networking_iproute2.c | 4 ++-- 1 file changed, 2 insertions(+), 2 deletions(-)