diff options
Diffstat (limited to '')
-rw-r--r-- | src/client_new.c | 12 | ||||
-rw-r--r-- | src/event_pipe.c | 7 | ||||
-rw-r--r-- | src/fd_util.c | 10 | ||||
-rw-r--r-- | src/fd_util.h | 12 | ||||
-rw-r--r-- | src/glib_socket.h | 40 | ||||
-rw-r--r-- | src/output/httpd_client.c | 7 | ||||
-rw-r--r-- | src/output/httpd_output_plugin.c | 4 | ||||
-rw-r--r-- | src/output/solaris_output_plugin.c | 20 | ||||
-rw-r--r-- | src/server_socket.c | 5 | ||||
-rw-r--r-- | src/socket_util.c | 6 |
10 files changed, 98 insertions, 25 deletions
diff --git a/src/client_new.c b/src/client_new.c index 5b2dfde65..1145c0236 100644 --- a/src/client_new.c +++ b/src/client_new.c @@ -19,9 +19,11 @@ #include "config.h" #include "client_internal.h" +#include "fd_util.h" #include "fifo_buffer.h" #include "socket_util.h" #include "permission.h" +#include "glib_socket.h" #include <assert.h> #include <sys/types.h> @@ -69,7 +71,7 @@ client_new(struct player_control *player_control, progname, hostaddr); g_free(hostaddr); - close(fd); + close_socket(fd); return; } @@ -79,18 +81,14 @@ client_new(struct player_control *player_control, if (client_list_is_full()) { g_warning("Max Connections Reached!"); - close(fd); + close_socket(fd); return; } client = g_new0(struct client, 1); client->player_control = player_control; -#ifndef G_OS_WIN32 - client->channel = g_io_channel_unix_new(fd); -#else - client->channel = g_io_channel_win32_new_socket(fd); -#endif + client->channel = g_io_channel_new_socket(fd); /* GLib is responsible for closing the file descriptor */ g_io_channel_set_close_on_unref(client->channel, true); /* NULL encoding means the stream is binary safe; the MPD diff --git a/src/event_pipe.c b/src/event_pipe.c index 85fd6d917..890d89f24 100644 --- a/src/event_pipe.c +++ b/src/event_pipe.c @@ -21,6 +21,7 @@ #include "event_pipe.h" #include "fd_util.h" #include "mpd_error.h" +#include "glib_socket.h" #include <stdbool.h> #include <assert.h> @@ -94,11 +95,7 @@ void event_pipe_init(void) if (ret < 0) MPD_ERROR("Couldn't open pipe: %s", strerror(errno)); -#ifndef G_OS_WIN32 - channel = g_io_channel_unix_new(event_pipe[0]); -#else - channel = g_io_channel_win32_new_fd(event_pipe[0]); -#endif + channel = g_io_channel_new_socket(event_pipe[0]); g_io_channel_set_encoding(channel, NULL, NULL); g_io_channel_set_buffered(channel, false); diff --git a/src/fd_util.c b/src/fd_util.c index 36f6e2fc9..882b4c7d5 100644 --- a/src/fd_util.c +++ b/src/fd_util.c @@ -327,3 +327,13 @@ inotify_init_cloexec(void) } #endif + +int +close_socket(int fd) +{ +#ifdef WIN32 + return closesocket(fd); +#else + return close(fd); +#endif +} diff --git a/src/fd_util.h b/src/fd_util.h index 96ad784d4..dd4df7a13 100644 --- a/src/fd_util.h +++ b/src/fd_util.h @@ -36,6 +36,8 @@ #ifndef FD_UTIL_H #define FD_UTIL_H +#include "check.h" + #include <stdbool.h> #include <stddef.h> @@ -127,6 +129,8 @@ recvmsg_cloexec(int sockfd, struct msghdr *msg, int flags); #endif +#ifdef HAVE_INOTIFY_INIT + /** * Wrapper for inotify_init(), which sets the CLOEXEC flag (atomically * if supported by the OS). @@ -135,3 +139,11 @@ int inotify_init_cloexec(void); #endif + +/** + * Portable wrapper for close(); use closesocket() on WIN32/WinSock. + */ +int +close_socket(int fd); + +#endif diff --git a/src/glib_socket.h b/src/glib_socket.h new file mode 100644 index 000000000..46fab37dd --- /dev/null +++ b/src/glib_socket.h @@ -0,0 +1,40 @@ +/* + * Copyright (C) 2003-2011 The Music Player Daemon Project + * http://www.musicpd.org + * + * This program is free software; you can redistribute it and/or modify + * it under the terms of the GNU General Public License as published by + * the Free Software Foundation; either version 2 of the License, or + * (at your option) any later version. + * + * This program is distributed in the hope that it will be useful, + * but WITHOUT ANY WARRANTY; without even the implied warranty of + * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE. See the + * GNU General Public License for more details. + * + * You should have received a copy of the GNU General Public License along + * with this program; if not, write to the Free Software Foundation, Inc., + * 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA. + */ + +#ifndef MPD_GLIB_SOCKET_H +#define MPD_GLIB_SOCKET_H + +#include <glib.h> + +/** + * Portable wrapper for g_io_channel_unix_new() or + * g_io_channel_win32_new_socket(). + */ +G_GNUC_MALLOC +static inline GIOChannel * +g_io_channel_new_socket(int fd) +{ +#ifdef G_OS_WIN32 + return g_io_channel_win32_new_socket(fd); +#else + return g_io_channel_unix_new(fd); +#endif +} + +#endif diff --git a/src/output/httpd_client.c b/src/output/httpd_client.c index e2c49c6c8..7a50f6fe6 100644 --- a/src/output/httpd_client.c +++ b/src/output/httpd_client.c @@ -24,6 +24,7 @@ #include "page.h" #include "icy_server.h" #include "glib_compat.h" +#include "glib_socket.h" #include <stdbool.h> #include <assert.h> @@ -459,11 +460,7 @@ httpd_client_new(struct httpd_output *httpd, int fd, bool metadata_supported) client->httpd = httpd; -#ifndef G_OS_WIN32 - client->channel = g_io_channel_unix_new(fd); -#else - client->channel = g_io_channel_win32_new_socket(fd); -#endif + client->channel = g_io_channel_new_socket(fd); /* GLib is responsible for closing the file descriptor */ g_io_channel_set_close_on_unref(client->channel, true); diff --git a/src/output/httpd_output_plugin.c b/src/output/httpd_output_plugin.c index ab663badf..8ef0c11a6 100644 --- a/src/output/httpd_output_plugin.c +++ b/src/output/httpd_output_plugin.c @@ -214,7 +214,7 @@ httpd_listen_in_event(int fd, const struct sockaddr *address, g_warning("libwrap refused connection (libwrap=%s) from %s", progname, hostaddr); g_free(hostaddr); - close(fd); + close_socket(fd); g_mutex_unlock(httpd->mutex); return; } @@ -235,7 +235,7 @@ httpd_listen_in_event(int fd, const struct sockaddr *address, httpd->clients_cnt < httpd->clients_max)) httpd_client_add(httpd, fd); else - close(fd); + close_socket(fd); } else if (fd < 0 && errno != EINTR) { g_warning("accept() failed: %s", g_strerror(errno)); } diff --git a/src/output/solaris_output_plugin.c b/src/output/solaris_output_plugin.c index 796c7806d..ce726009a 100644 --- a/src/output/solaris_output_plugin.c +++ b/src/output/solaris_output_plugin.c @@ -24,7 +24,6 @@ #include <glib.h> -#include <sys/audio.h> #include <sys/stropts.h> #include <sys/types.h> #include <sys/stat.h> @@ -32,6 +31,25 @@ #include <fcntl.h> #include <errno.h> +#ifdef __sun +#include <sys/audio.h> +#else + +/* some fake declarations that allow build this plugin on systems + other than Solaris, just to see if it compiles */ + +#define AUDIO_GETINFO 0 +#define AUDIO_SETINFO 0 +#define AUDIO_ENCODING_LINEAR 0 + +struct audio_info { + struct { + unsigned sample_rate, channels, precision, encoding; + } play; +}; + +#endif + #undef G_LOG_DOMAIN #define G_LOG_DOMAIN "solaris_output" diff --git a/src/server_socket.c b/src/server_socket.c index fc8dbd17a..4affeb91b 100644 --- a/src/server_socket.c +++ b/src/server_socket.c @@ -27,6 +27,7 @@ #include "socket_util.h" #include "fd_util.h" #include "glib_compat.h" +#include "glib_socket.h" #include <sys/types.h> #include <sys/stat.h> @@ -227,7 +228,7 @@ server_socket_open(struct server_socket *ss, GError **error_r) /* register in the GLib main loop */ - GIOChannel *channel = g_io_channel_unix_new(s->fd); + GIOChannel *channel = g_io_channel_new_socket(s->fd); s->source_id = g_io_add_watch(channel, G_IO_IN, server_socket_in_event, s); g_io_channel_unref(channel); @@ -261,7 +262,7 @@ server_socket_close(struct server_socket *ss) continue; g_source_remove(s->source_id); - close(s->fd); + close_socket(s->fd); s->fd = -1; } } diff --git a/src/socket_util.c b/src/socket_util.c index aa0a44e44..f673d73e9 100644 --- a/src/socket_util.c +++ b/src/socket_util.c @@ -122,7 +122,7 @@ socket_bind_listen(int domain, int type, int protocol, if (ret < 0) { g_set_error(error, listen_quark(), errno, "setsockopt() failed: %s", g_strerror(errno)); - close(fd); + close_socket(fd); return -1; } @@ -130,7 +130,7 @@ socket_bind_listen(int domain, int type, int protocol, if (ret < 0) { g_set_error(error, listen_quark(), errno, "%s", g_strerror(errno)); - close(fd); + close_socket(fd); return -1; } @@ -138,7 +138,7 @@ socket_bind_listen(int domain, int type, int protocol, if (ret < 0) { g_set_error(error, listen_quark(), errno, "listen() failed: %s", g_strerror(errno)); - close(fd); + close_socket(fd); return -1; } |