[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [PATCH 1/4] util: move socket_init() to osdep.c
From: |
Paolo Bonzini |
Subject: |
Re: [Qemu-devel] [PATCH 1/4] util: move socket_init() to osdep.c |
Date: |
Mon, 25 Feb 2013 19:13:40 +0100 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:17.0) Gecko/20130110 Thunderbird/17.0.2 |
Il 25/02/2013 18:56, Marc-André Lureau ha scritto:
> vscclient needs to call socket_init() for portability.
> Moving to osdep.c since it has no internal dependency.
>
> Signed-off-by: Marc-André Lureau <address@hidden>
> ---
> util/osdep.c | 23 +++++++++++++++++++++++
> util/qemu-sockets.c | 24 ------------------------
> 2 files changed, 23 insertions(+), 24 deletions(-)
>
> diff --git a/util/osdep.c b/util/osdep.c
> index 5b51a03..3c1a0a3 100644
> --- a/util/osdep.c
> +++ b/util/osdep.c
> @@ -400,3 +400,26 @@ bool fips_get_state(void)
> return fips_enabled;
> }
>
> +#ifdef _WIN32
> +static void socket_cleanup(void)
> +{
> + WSACleanup();
> +}
> +#endif
> +
> +int socket_init(void)
> +{
> +#ifdef _WIN32
> + WSADATA Data;
> + int ret, err;
> +
> + ret = WSAStartup(MAKEWORD(2, 2), &Data);
> + if (ret != 0) {
> + err = WSAGetLastError();
> + fprintf(stderr, "WSAStartup: %d\n", err);
> + return -1;
> + }
> + atexit(socket_cleanup);
> +#endif
> + return 0;
> +}
> diff --git a/util/qemu-sockets.c b/util/qemu-sockets.c
> index 1350ccc..d8994c1 100644
> --- a/util/qemu-sockets.c
> +++ b/util/qemu-sockets.c
> @@ -944,27 +944,3 @@ int socket_listen(SocketAddress *addr, Error **errp)
> qemu_opts_del(opts);
> return fd;
> }
> -
> -#ifdef _WIN32
> -static void socket_cleanup(void)
> -{
> - WSACleanup();
> -}
> -#endif
> -
> -int socket_init(void)
> -{
> -#ifdef _WIN32
> - WSADATA Data;
> - int ret, err;
> -
> - ret = WSAStartup(MAKEWORD(2,2), &Data);
> - if (ret != 0) {
> - err = WSAGetLastError();
> - fprintf(stderr, "WSAStartup: %d\n", err);
> - return -1;
> - }
> - atexit(socket_cleanup);
> -#endif
> - return 0;
> -}
>
Reviewed-by: Paolo Bonzini <address@hidden>