[Openvpn-devel] [PATCH] make sure sa_family_t is defined

[Openvpn-devel] [PATCH] make sure sa_family_t is defined

[Heiko Hund]

On Windows there's no sa_family_t. This patch defines it in
syshead.h is configure did not find it in the system headers.

Signed-off-by: Heiko Hund <heiko.hund@...1489...>
---
 configure.ac          |    6 ++++++
 src/openvpn/syshead.h |    7 +++++++
 2 files changed, 13 insertions(+), 0 deletions(-)

diff --git a/configure.ac b/configure.ac
index 636d4e2..65dbc25 100644
--- a/configure.ac
+++ b/configure.ac
@@ -501,6 +501,12 @@ AC_CHECK_TYPE(
 	[[${SOCKET_INCLUDES}]]
 )
 AC_CHECK_TYPE(
+        [sa_family_t],
+        [AC_DEFINE([HAVE_SA_FAMILY_T], [1], [struct in_pktinfo needed for IP_PKTINFO support])],
+        ,
+        [[${SOCKET_INCLUDES}]]
+)
+AC_CHECK_TYPE(
 	[struct sockaddr_in6],
 	,
 	[AC_MSG_ERROR([struct sockaddr_in6 not found, needed for ipv6 transport support.])],
diff --git a/src/openvpn/syshead.h b/src/openvpn/syshead.h
index ab6fa01..4050d54 100644
--- a/src/openvpn/syshead.h
+++ b/src/openvpn/syshead.h
@@ -426,6 +426,13 @@
 #endif
 
 /*
+ * Define type sa_family_t if it isn't defined in the socket headers
+ */
+#ifndef HAVE_SA_FAMILY_T
+typedef unsigned short sa_family_t;
+#endif
+
+/*
  * Disable ESEC
  */
 #if 0
-- 
1.7.9

[Openvpn-devel] [PATCH] convert struct signal_info element

[Heiko Hund]

This one place was forgotten when the elements type was changed
from bool to int in the dual stack patch 23d61c56.

Signed-off-by: Heiko Hund <heiko.hund@...1489...>
---
 src/openvpn/win32.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)

diff --git a/src/openvpn/win32.c b/src/openvpn/win32.c
index f35c96b..7c89a5a 100644
--- a/src/openvpn/win32.c
+++ b/src/openvpn/win32.c
@@ -517,7 +517,7 @@ win32_signal_get (struct win32_signal *ws)
       if (ret)
 	{
 	  siginfo_static.signal_received = ret;
-	  siginfo_static.hard = true;
+	  siginfo_static.source = SIG_SOURCE_HARD;
 	}
     }
   return ret;
-- 
1.7.9

Re: [Openvpn-devel] [PATCH] make sure sa_family_t is defined

[Arne Schwabe]

[-- Attachment #1: Type: text/plain, Size: 186 bytes --]

Am 17.01.14 16:30, schrieb Heiko Hund:
> On Windows there's no sa_family_t. This patch defines it in
> syshead.h is configure did not find it in the system headers.

ACK.

Arne


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

Re: [Openvpn-devel] [PATCH] convert struct signal_info element

[Arne Schwabe]

[-- Attachment #1: Type: text/plain, Size: 179 bytes --]

Am 17.01.14 16:30, schrieb Heiko Hund:
> This one place was forgotten when the elements type was changed
> from bool to int in the dual stack patch 23d61c56.
>
ACK

Arne


[-- Attachment #2: OpenPGP digital signature --]
[-- Type: application/pgp-signature, Size: 260 bytes --]

[Openvpn-devel] [PATCH applied] Re: make sure sa_family_t is defined

[Gert Doering]

Your patch has been applied to the master branch.

I took the liberty to change the descriptive comment to something actually
related to sa_family_t :-)

commit 87b468d42811f0ca6f046b3a0335811370193514
Author: Heiko Hund
Date:   Fri Jan 17 16:30:37 2014 +0100

     make sure sa_family_t is defined

     Signed-off-by: Heiko Hund <heiko.hund@...1489...>
     Acked-by: Arne Schwabe <arne@...1227...>
     Message-Id: <1389972638-8006-1-git-send-email-heiko.hund@...1489...>
     URL: http://article.gmane.org/gmane.network.openvpn.devel/8242
     Signed-off-by: Gert Doering <gert@...1296...>


--
kind regards,

Gert Doering

[Openvpn-devel] [PATCH applied] Re: convert struct signal_info element

[Gert Doering]

Your patch has been applied to the master branch.

commit 9853b14f0934f6e37600b186831fae3d5bad213f
Author: Heiko Hund
Date:   Fri Jan 17 16:30:38 2014 +0100

     convert struct signal_info element

     Signed-off-by: Heiko Hund <heiko.hund@...1489...>
     Acked-by: Arne Schwabe <arne@...1227...>
     Message-Id: <1389972638-8006-2-git-send-email-heiko.hund@...1489...>
     URL: http://article.gmane.org/gmane.network.openvpn.devel/8243
     Signed-off-by: Gert Doering <gert@...1296...>


--
kind regards,

Gert Doering