Attention is currently required from: flichtenheld, plaisthos.
Hello cron2, flichtenheld,
I'd like you to reexamine a change. Please visit
http://gerrit.openvpn.net/c/openvpn/+/749?usp=email
to look at the new patch set (#11).
The change is no longer submittable: checks~ChecksSubmitRule is unsatisfied now.
Change subject: Use print_tun_backend_driver instead of custom code to print
type
......................................................................
Use print_tun_backend_driver instead of custom code to print type
Also show the device type that we opened always instead of certain
conditions only.
Change-Id: Ib8f12516dbe294e21d3fed77478fb7660d4600c1
Signed-off-by: Arne Schwabe <[email protected]>
---
M src/openvpn/init.c
M src/openvpn/tun.c
M src/openvpn/tun.h
3 files changed, 8 insertions(+), 5 deletions(-)
git pull ssh://gerrit.openvpn.net:29418/openvpn refs/changes/49/749/11
diff --git a/src/openvpn/init.c b/src/openvpn/init.c
index fbf2c5b..cd9203a 100644
--- a/src/openvpn/init.c
+++ b/src/openvpn/init.c
@@ -1876,6 +1876,7 @@
open_tun(c->options.dev, c->options.dev_type, c->options.dev_node,
tt, &c->net_ctx);
}
+ msg(M_INFO, "%s device [%s] opened",
print_tun_backend_driver(tt->backend_driver), tt->actual_name);
}
@@ -2056,7 +2057,7 @@
do_close_tun_simple(struct context *c)
{
msg(D_CLOSE, "Closing %s interface",
- dco_enabled(&c->options) ? "DCO" : "TUN/TAP");
+ print_tun_backend_driver(c->c1.tuntap->backend_driver));
if (c->c1.tuntap)
{
diff --git a/src/openvpn/tun.c b/src/openvpn/tun.c
index 770e806..206ddc0 100644
--- a/src/openvpn/tun.c
+++ b/src/openvpn/tun.c
@@ -6678,7 +6678,6 @@
* GUID using the registry */
tt->actual_name = string_alloc((const char *)actual_buffer, NULL);
- msg(M_INFO, "%s device [%s] opened",
print_tun_backend_driver(tt->backend_driver), tt->actual_name);
tt->adapter_index = get_adapter_index(*device_guid);
}
diff --git a/src/openvpn/tun.h b/src/openvpn/tun.h
index b2c1b01..329cd10 100644
--- a/src/openvpn/tun.h
+++ b/src/openvpn/tun.h
@@ -361,6 +361,12 @@
void warn_on_use_of_common_subnets(openvpn_net_ctx_t *ctx);
+/**
+ * Return a string representation of the tun backed driver type
+ */
+const char *
+print_tun_backend_driver(enum tun_driver_type driver);
+
/*
* Should ifconfig be called before or after
* tun dev open?
@@ -689,9 +695,6 @@
return tuntap_is_dco_win(tt) && (status < 0) && (openvpn_errno() ==
ERROR_NETNAME_DELETED);
}
-const char *
-print_tun_backend_driver(enum tun_driver_type driver);
-
#else /* ifdef _WIN32 */
static inline bool
--
To view, visit http://gerrit.openvpn.net/c/openvpn/+/749?usp=email
To unsubscribe, or for help writing mail filters, visit
http://gerrit.openvpn.net/settings
Gerrit-Project: openvpn
Gerrit-Branch: master
Gerrit-Change-Id: Ib8f12516dbe294e21d3fed77478fb7660d4600c1
Gerrit-Change-Number: 749
Gerrit-PatchSet: 11
Gerrit-Owner: plaisthos <[email protected]>
Gerrit-Reviewer: cron2 <[email protected]>
Gerrit-Reviewer: flichtenheld <[email protected]>
Gerrit-CC: openvpn-devel <[email protected]>
Gerrit-Attention: plaisthos <[email protected]>
Gerrit-Attention: flichtenheld <[email protected]>
Gerrit-MessageType: newpatchset
_______________________________________________
Openvpn-devel mailing list
[email protected]
https://lists.sourceforge.net/lists/listinfo/openvpn-devel
