[Openvpn-devel] close_tun: print interface type consistently in message

Message ID 20220818001358.175361-1-a@unstable.cc
State Changes Requested
Headers show
Series [Openvpn-devel] close_tun: print interface type consistently in message | expand

Commit Message

Antonio Quartulli Aug. 17, 2022, 2:13 p.m. UTC
When closing the tunnel interface we know if we were using DCO or not.
for this reason we can customize the closing message and make it
consistent with the opening one.

Signed-off-by: Antonio Quartulli <a@unstable.cc>
---
 src/openvpn/init.c | 10 +++++++++-
 1 file changed, 9 insertions(+), 1 deletion(-)

Comments

Gert Doering Aug. 17, 2022, 7:52 p.m. UTC | #1
Hi,

On Thu, Aug 18, 2022 at 02:13:58AM +0200, Antonio Quartulli wrote:
> When closing the tunnel interface we know if we were using DCO or not.
> for this reason we can customize the closing message and make it
> consistent with the opening one.
> 
> Signed-off-by: Antonio Quartulli <a@unstable.cc>
> ---
>  src/openvpn/init.c | 10 +++++++++-
>  1 file changed, 9 insertions(+), 1 deletion(-)
> 
> diff --git a/src/openvpn/init.c b/src/openvpn/init.c
> index e8335c53..92c4cab9 100644
> --- a/src/openvpn/init.c
> +++ b/src/openvpn/init.c
> @@ -1921,7 +1921,15 @@ do_open_tun(struct context *c)
>  static void
>  do_close_tun_simple(struct context *c)
>  {
> -    msg(D_CLOSE, "Closing TUN/TAP interface");
> +    if (dco_enabled(&c->options))
> +    {
> +        msg(D_CLOSE, "Closing DCO interface");
> +    }
> +    else
> +    {
> +        msg(D_CLOSE, "Closing TUN/TAP interface");
> +    }

I find this is a bit excessive in lines of code...  

If we really want this, I think we should do

 +    msg(D_CLOSE, "Closing %s interface",
          dco_enabled(&c->options)? "DCO": "TUN/TAP");

gert

Patch

diff --git a/src/openvpn/init.c b/src/openvpn/init.c
index e8335c53..92c4cab9 100644
--- a/src/openvpn/init.c
+++ b/src/openvpn/init.c
@@ -1921,7 +1921,15 @@  do_open_tun(struct context *c)
 static void
 do_close_tun_simple(struct context *c)
 {
-    msg(D_CLOSE, "Closing TUN/TAP interface");
+    if (dco_enabled(&c->options))
+    {
+        msg(D_CLOSE, "Closing DCO interface");
+    }
+    else
+    {
+        msg(D_CLOSE, "Closing TUN/TAP interface");
+    }
+
     if (c->c1.tuntap)
     {
         close_tun(c->c1.tuntap, &c->net_ctx);