diff options
| author | Arne Schwabe <arne@rfc2549.org> | 2013-11-25 13:31:13 +0100 |
|---|---|---|
| committer | Gert Doering <gert@greenie.muc.de> | 2013-11-26 15:34:59 +0100 |
| commit | 34136dd8533510f68a012ba9e6bcd8cf5d1ce80e (patch) | |
| tree | e70ad2c3e9238cc4021f6475ded4316d0eabbdf4 /src/openvpn/socket.c | |
| parent | 48f1d41a80b1f2aa9e47b38f645dc880bd7090a4 (diff) | |
| download | openvpn-34136dd8533510f68a012ba9e6bcd8cf5d1ce80e.tar.gz openvpn-34136dd8533510f68a012ba9e6bcd8cf5d1ce80e.tar.xz openvpn-34136dd8533510f68a012ba9e6bcd8cf5d1ce80e.zip | |
Change proto_remote() function to return a constant string
Instead of using the implicit protocol string that is returned by the
proto/af to names function return a constant string. The strings have
become part of the wire protocl and we do not want them to change if
the printing of proto/af changes.
Acked-by: Gert Doering <gert@greenie.muc.de>
Message-Id: <1385382680-5912-2-git-send-email-arne@rfc2549.org>
URL: http://article.gmane.org/gmane.network.openvpn.devel/8055
Signed-off-by: Gert Doering <gert@greenie.muc.de>
Diffstat (limited to 'src/openvpn/socket.c')
| -rw-r--r-- | src/openvpn/socket.c | 22 |
1 files changed, 12 insertions, 10 deletions
diff --git a/src/openvpn/socket.c b/src/openvpn/socket.c index f0e0914..1117e30 100644 --- a/src/openvpn/socket.c +++ b/src/openvpn/socket.c @@ -2599,19 +2599,21 @@ addr_family_name (int af) * has always sent UDPv4, TCPv4 over the wire. Keep these * strings for backward compatbility */ -int +const char* proto_remote (int proto, bool remote) { ASSERT (proto >= 0 && proto < PROTO_N); - if (remote) - { - switch (proto) - { - case PROTO_TCP_SERVER: return PROTO_TCP_CLIENT; - case PROTO_TCP_CLIENT: return PROTO_TCP_SERVER; - } - } - return proto; + if (proto == PROTO_UDP) + return "UDPv4"; + + if ( (remote && proto == PROTO_TCP_CLIENT) || + (!remote && proto == PROTO_TCP_SERVER)) + return "TCPv4_SERVER"; + if ( (remote && proto == PROTO_TCP_SERVER) || + (!remote && proto == PROTO_TCP_CLIENT)) + return "TCPv4_CLIENT"; + + ASSERT (0); } /* |