[PATCH] wbemprox: Fix the build on FreeBSD by including sys/socket.h.

[Gerald Pfeifer]

Commit 88d52edcc4d5a81965627905278d1212250ae84d

    Author: Piotr Caban <piotr(a)codeweavers.com>
    Date:   Wed Aug 1 17:23:29 2018 +0200

    wbemprox: Add Win32_NetworkAdapterConfiguration::IPSubnet property.
    
    Signed-off-by: Piotr Caban <piotr(a)codeweavers.com>
    Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>
    Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>

broke the build on FreeBSD 10.x and likely other versions:

    builtin.c: In function 'get_ipsubnet':
    builtin.c:2499:55: error: 'AF_INET' undeclared (first use in this function)
    2499 |         if (address->Address.lpSockaddr->sa_family == AF_INET)
         |                                                       ^~~~~~~

This fixes it for me.

Gerald


Signed-off-by: Gerald Pfeifer <gerald(a)pfeifer.com>
---
 dlls/wbemprox/builtin.c | 3 +++
 1 file changed, 3 insertions(+)

diff --git a/dlls/wbemprox/builtin.c b/dlls/wbemprox/builtin.c
index 72c6b4e933..3b8b43e868 100644
--- a/dlls/wbemprox/builtin.c
+++ b/dlls/wbemprox/builtin.c
@@ -26,6 +26,9 @@
 #ifdef HAVE_UNISTD_H
 # include <unistd.h>
 #endif
+#ifdef HAVE_SYS_SOCKET_H
+# include <sys/socket.h>
+#endif
 #ifdef HAVE_ARPA_INET_H
 # include <arpa/inet.h>
 #endif
-- 
2.17.0

[Piotr Caban]

Hi Gerald,

I think we should use WS_AF_INET here (and define it in __MINGW32__ case).

Thanks,
Piotr

On 08/18/18 18:56, Gerald Pfeifer wrote:
> Commit 88d52edcc4d5a81965627905278d1212250ae84d
>
> Author: Piotr Caban <piotr(a)codeweavers.com>
>      Date:   Wed Aug 1 17:23:29 2018 +0200
>
> wbemprox: Add Win32_NetworkAdapterConfiguration::IPSubnet property.
>
> Signed-off-by: Piotr Caban <piotr(a)codeweavers.com>
>      Signed-off-by: Hans Leidekker <hans(a)codeweavers.com>
>      Signed-off-by: Alexandre Julliard <julliard(a)winehq.org>
>
> broke the build on FreeBSD 10.x and likely other versions:
>
> builtin.c: In function 'get_ipsubnet':
>      builtin.c:2499:55: error: 'AF_INET' undeclared (first use in this function)
>      2499 |         if (address->Address.lpSockaddr->sa_family == AF_INET)
>           |                                                       ^~~~~~~
>
> This fixes it for me.
>
> Gerald
>
> Signed-off-by: Gerald Pfeifer <gerald(a)pfeifer.com>
> ---
>   dlls/wbemprox/builtin.c | 3 +++
>   1 file changed, 3 insertions(+)
>
> diff --git a/dlls/wbemprox/builtin.c b/dlls/wbemprox/builtin.c
> index 72c6b4e933..3b8b43e868 100644
> --- a/dlls/wbemprox/builtin.c
> +++ b/dlls/wbemprox/builtin.c
> @@ -26,6 +26,9 @@
>   #ifdef HAVE_UNISTD_H
>   # include <unistd.h>
>   #endif
> +#ifdef HAVE_SYS_SOCKET_H
> +# include <sys/socket.h>
> +#endif
>   #ifdef HAVE_ARPA_INET_H
>   # include <arpa/inet.h>
>   #endif