Fix indentation and spacing. Use INADDR_LOOPBACK instead of

2003-06-08  Tor Lillqvist  <tml@iki.fi>

	* glib/giowin32.c: Fix indentation and spacing. Use
	INADDR_LOOPBACK instead of inet_addr("127.0.0.1") and
	gethostbyaddr().
This commit is contained in:
Tor Lillqvist 2003-06-08 01:38:13 +00:00 committed by Tor Lillqvist
parent be76dd63f9
commit 51704100d0
7 changed files with 151 additions and 120 deletions

View File

@ -1,3 +1,9 @@
2003-06-08 Tor Lillqvist <tml@iki.fi>
* glib/giowin32.c: Fix indentation and spacing. Use
INADDR_LOOPBACK instead of inet_addr("127.0.0.1") and
gethostbyaddr().
Sat Jun 6 16:18:10 2003 Andrew Lanoix <alanoix@umich.edu>
* glib/giowin32.c: Resolved thread deadlocks in socket

View File

@ -1,3 +1,9 @@
2003-06-08 Tor Lillqvist <tml@iki.fi>
* glib/giowin32.c: Fix indentation and spacing. Use
INADDR_LOOPBACK instead of inet_addr("127.0.0.1") and
gethostbyaddr().
Sat Jun 6 16:18:10 2003 Andrew Lanoix <alanoix@umich.edu>
* glib/giowin32.c: Resolved thread deadlocks in socket

View File

@ -1,3 +1,9 @@
2003-06-08 Tor Lillqvist <tml@iki.fi>
* glib/giowin32.c: Fix indentation and spacing. Use
INADDR_LOOPBACK instead of inet_addr("127.0.0.1") and
gethostbyaddr().
Sat Jun 6 16:18:10 2003 Andrew Lanoix <alanoix@umich.edu>
* glib/giowin32.c: Resolved thread deadlocks in socket

View File

@ -1,3 +1,9 @@
2003-06-08 Tor Lillqvist <tml@iki.fi>
* glib/giowin32.c: Fix indentation and spacing. Use
INADDR_LOOPBACK instead of inet_addr("127.0.0.1") and
gethostbyaddr().
Sat Jun 6 16:18:10 2003 Andrew Lanoix <alanoix@umich.edu>
* glib/giowin32.c: Resolved thread deadlocks in socket

View File

@ -1,3 +1,9 @@
2003-06-08 Tor Lillqvist <tml@iki.fi>
* glib/giowin32.c: Fix indentation and spacing. Use
INADDR_LOOPBACK instead of inet_addr("127.0.0.1") and
gethostbyaddr().
Sat Jun 6 16:18:10 2003 Andrew Lanoix <alanoix@umich.edu>
* glib/giowin32.c: Resolved thread deadlocks in socket

View File

@ -1,3 +1,9 @@
2003-06-08 Tor Lillqvist <tml@iki.fi>
* glib/giowin32.c: Fix indentation and spacing. Use
INADDR_LOOPBACK instead of inet_addr("127.0.0.1") and
gethostbyaddr().
Sat Jun 6 16:18:10 2003 Andrew Lanoix <alanoix@umich.edu>
* glib/giowin32.c: Resolved thread deadlocks in socket

View File

@ -313,7 +313,7 @@ read_thread (void *parameter)
SetEvent (channel->data_avail_event);
UNLOCK (channel->mutex);
g_io_channel_unref((GIOChannel *)channel);
g_io_channel_unref ((GIOChannel *)channel);
/* No need to call _endthreadex(), the actual thread starter routine
* in MSVCRT (see crt/src/threadex.c:_threadstartex) calls
@ -343,14 +343,12 @@ create_thread (GIOWin32Channel *channel,
}
static void
init_reset_sockets(GIOWin32Channel *channel)
init_reset_sockets (GIOWin32Channel *channel)
{
struct sockaddr_in local, local2, server;
unsigned int addr;
int len;
struct hostent *hp;
channel->reset_send = (gint) socket(AF_INET, SOCK_DGRAM, 0);
channel->reset_send = (gint) socket (AF_INET, SOCK_DGRAM, 0);
if (channel->reset_send == INVALID_SOCKET)
{
g_warning (G_STRLOC ": Error creating reset_send socket: %s\n",
@ -359,9 +357,9 @@ init_reset_sockets(GIOWin32Channel *channel)
local.sin_family = AF_INET;
local.sin_port = 0;
local.sin_addr.S_un.S_addr = htonl(INADDR_ANY);
local.sin_addr.s_addr = htonl (INADDR_ANY);
if (bind(channel->reset_send, (struct sockaddr *)&local, sizeof(local)) == SOCKET_ERROR)
if (bind (channel->reset_send, (struct sockaddr *)&local, sizeof (local)) == SOCKET_ERROR)
{
g_warning (G_STRLOC ": Error binding to reset_send socket: %s\n",
g_win32_error_message (WSAGetLastError ()));
@ -369,37 +367,34 @@ init_reset_sockets(GIOWin32Channel *channel)
local2.sin_family = AF_INET;
local2.sin_port = 0;
local2.sin_addr.S_un.S_addr = htonl(INADDR_ANY);
local2.sin_addr.s_addr = htonl (INADDR_ANY);
channel->reset_recv = (gint) socket(AF_INET, SOCK_DGRAM, 0);
channel->reset_recv = (gint) socket (AF_INET, SOCK_DGRAM, 0);
if (channel->reset_recv == INVALID_SOCKET)
{
g_warning (G_STRLOC ": Error creating reset_recv socket: %s\n",
g_win32_error_message (WSAGetLastError ()));
}
if (bind(channel->reset_recv, (struct sockaddr *)&local2, sizeof(local)) == SOCKET_ERROR)
if (bind (channel->reset_recv, (struct sockaddr *)&local2, sizeof (local)) == SOCKET_ERROR)
{
g_warning (G_STRLOC ": Error binding to reset_recv socket: %s\n",
g_win32_error_message (WSAGetLastError ()));
}
len = sizeof(local2);
if (getsockname(channel->reset_recv, (struct sockaddr *)&local2, &len) == SOCKET_ERROR)
len = sizeof (local2);
if (getsockname (channel->reset_recv, (struct sockaddr *)&local2, &len) == SOCKET_ERROR)
{
g_warning (G_STRLOC ": Error getsockname with reset_recv socket: %s\n",
g_win32_error_message (WSAGetLastError ()));
}
addr = inet_addr("127.0.0.1");
hp = gethostbyaddr((char *)&addr,4,AF_INET);
memset(&server,0,sizeof(server));
memcpy(&(server.sin_addr),hp->h_addr,hp->h_length);
server.sin_family = hp->h_addrtype;
memset (&server, 0, sizeof (server));
server.sin_addr.s_addr = htonl (INADDR_LOOPBACK);
server.sin_family = AF_INET;
server.sin_port = local2.sin_port;
if (connect(channel->reset_send, (struct sockaddr *)&server, sizeof(server)) == SOCKET_ERROR)
if (connect (channel->reset_send, (struct sockaddr *)&server, sizeof (server)) == SOCKET_ERROR)
{
g_warning (G_STRLOC ": connect to reset_recv socket: %s\n",
g_win32_error_message (WSAGetLastError ()));
@ -505,7 +500,7 @@ select_thread (void *parameter)
FD_ZERO (&read_fds);
FD_ZERO (&write_fds);
FD_ZERO (&except_fds);
FD_SET(channel->reset_recv, &read_fds);
FD_SET (channel->reset_recv, &read_fds);
LOCK (channel->mutex);
tmp = channel->watches;
@ -549,12 +544,12 @@ select_thread (void *parameter)
break;
}
if (FD_ISSET(channel->reset_recv, &read_fds))
if (FD_ISSET (channel->reset_recv, &read_fds))
{
if (channel->debug)
g_print ("select_thread %#x: re-looping\n",
channel->thread_id);
recv(channel->reset_recv, (char *)&buffer, (int) sizeof(buffer), 0);
recv (channel->reset_recv, (char *)&buffer, (int) sizeof (buffer), 0);
continue;
}
@ -602,7 +597,7 @@ select_thread (void *parameter)
g_print ("select_thread %#x: got error, setting data_avail\n",
channel->thread_id);
SetEvent (channel->data_avail_event);
g_io_channel_unref((GIOChannel *)channel);
g_io_channel_unref ((GIOChannel *)channel);
UNLOCK (channel->mutex);
/* No need to call _endthreadex(), the actual thread starter routine
@ -734,7 +729,7 @@ g_io_win32_finalize (GSource *source)
SetEvent (channel->data_avail_noticed_event);
if (channel->type == G_IO_WIN32_SOCKET)
send(channel->reset_send, send_buffer, sizeof(send_buffer), 0);
send (channel->reset_send, send_buffer, sizeof (send_buffer), 0);
g_io_channel_unref (watch->channel);
UNLOCK (channel->mutex);
@ -784,7 +779,7 @@ g_io_win32_create_watch (GIOChannel *channel,
if (win32_channel->thread_id == 0)
create_thread (win32_channel, condition, thread);
else
send(win32_channel->reset_send, send_buffer, sizeof(send_buffer), 0);
send (win32_channel->reset_send, send_buffer, sizeof (send_buffer), 0);
g_source_add_poll (source, &watch->pollfd);
UNLOCK (win32_channel->mutex);
@ -804,7 +799,7 @@ g_io_win32_msg_read (GIOChannel *channel,
if (count < sizeof (MSG))
{
g_set_error(err, G_IO_CHANNEL_ERROR, G_IO_CHANNEL_ERROR_INVAL,
g_set_error (err, G_IO_CHANNEL_ERROR, G_IO_CHANNEL_ERROR_INVAL,
"Incorrect message size"); /* Informative enough error message? */
return G_IO_STATUS_ERROR;
}
@ -833,7 +828,7 @@ g_io_win32_msg_write (GIOChannel *channel,
if (count != sizeof (MSG))
{
g_set_error(err, G_IO_CHANNEL_ERROR, G_IO_CHANNEL_ERROR_INVAL,
g_set_error (err, G_IO_CHANNEL_ERROR, G_IO_CHANNEL_ERROR_INVAL,
"Incorrect message size"); /* Informative enough error message? */
return G_IO_STATUS_ERROR;
}
@ -843,7 +838,7 @@ g_io_win32_msg_write (GIOChannel *channel,
if (!PostMessage (win32_channel->hwnd, msg.message, msg.wParam, msg.lParam))
{
gchar *emsg = g_win32_error_message (GetLastError ());
g_set_error(err, G_IO_CHANNEL_ERROR, G_IO_CHANNEL_ERROR_FAILED, emsg);
g_set_error (err, G_IO_CHANNEL_ERROR, G_IO_CHANNEL_ERROR_FAILED, emsg);
g_free (emsg);
return G_IO_STATUS_ERROR;
}
@ -873,9 +868,9 @@ g_io_win32_free (GIOChannel *channel)
win32_channel->fd);
if (win32_channel->reset_send)
closesocket(win32_channel->reset_send);
closesocket (win32_channel->reset_send);
if (win32_channel->reset_recv)
closesocket(win32_channel->reset_recv);
closesocket (win32_channel->reset_recv);
if (win32_channel->data_avail_event)
CloseHandle (win32_channel->data_avail_event);
if (win32_channel->space_avail_event)
@ -940,7 +935,7 @@ g_io_win32_fd_read (GIOChannel *channel,
{
*bytes_read = 0;
switch(errno)
switch (errno)
{
#ifdef EAGAIN
case EAGAIN:
@ -978,7 +973,7 @@ g_io_win32_fd_write (GIOChannel *channel,
{
*bytes_written = 0;
switch(errno)
switch (errno)
{
#ifdef EAGAIN
case EAGAIN:
@ -1021,7 +1016,7 @@ g_io_win32_fd_seek (GIOChannel *channel,
break;
default:
whence = -1; /* Keep the compiler quiet */
g_assert_not_reached();
g_assert_not_reached ();
}
tmp_offset = offset;
@ -1133,7 +1128,7 @@ repeat:
error = G_IO_CHANNEL_ERROR_FAILED;
break;
}
g_set_error(err, G_IO_CHANNEL_ERROR, error, "Socket read error");
g_set_error (err, G_IO_CHANNEL_ERROR, error, "Socket read error");
internal_status = G_IO_STATUS_ERROR;
/* FIXME get all errors, better error messages */
}
@ -1148,9 +1143,9 @@ repeat:
(internal_status == G_IO_STATUS_ERROR))
{
LOCK (win32_channel->mutex);
SetEvent(win32_channel->data_avail_noticed_event);
SetEvent (win32_channel->data_avail_noticed_event);
win32_channel->needs_close = 1;
send(win32_channel->reset_send, send_buffer, sizeof(send_buffer), 0);
send (win32_channel->reset_send, send_buffer, sizeof (send_buffer), 0);
UNLOCK (win32_channel->mutex);
}
return internal_status;
@ -1198,11 +1193,11 @@ repeat:
error = G_IO_CHANNEL_ERROR_FAILED;
break;
}
g_set_error(err, G_IO_CHANNEL_ERROR, error, "Socket write error");
g_set_error (err, G_IO_CHANNEL_ERROR, error, "Socket write error");
LOCK (win32_channel->mutex);
SetEvent(win32_channel->data_avail_noticed_event);
SetEvent (win32_channel->data_avail_noticed_event);
win32_channel->needs_close = 1;
send(win32_channel->reset_send, send_buffer, sizeof(send_buffer), 0);
send (win32_channel->reset_send, send_buffer, sizeof (send_buffer), 0);
UNLOCK (win32_channel->mutex);
return G_IO_STATUS_ERROR;
/* FIXME get all errors, better error messages */
@ -1221,7 +1216,7 @@ g_io_win32_sock_close (GIOChannel *channel,
{
GIOWin32Channel *win32_channel = (GIOWin32Channel *)channel;
LOCK(win32_channel->mutex);
LOCK (win32_channel->mutex);
if (win32_channel->running)
{
if (win32_channel->debug)
@ -1241,7 +1236,7 @@ g_io_win32_sock_close (GIOChannel *channel,
closesocket (win32_channel->fd);
win32_channel->fd = -1;
}
UNLOCK(win32_channel->mutex);
UNLOCK (win32_channel->mutex);
/* FIXME error detection? */
@ -1590,7 +1585,7 @@ g_io_channel_win32_new_socket (int socket)
g_io_channel_init (channel);
g_io_channel_win32_init (win32_channel);
init_reset_sockets(channel);
init_reset_sockets (win32_channel);
if (win32_channel->debug)
g_print ("g_io_channel_win32_new_socket: sockfd:%d\n", socket);
channel->funcs = &win32_channel_sock_funcs;
@ -1615,7 +1610,7 @@ g_io_channel_unix_new (gint fd)
return g_io_channel_win32_new_fd_internal (fd, &st);
if (getsockopt (fd, SOL_SOCKET, SO_TYPE, NULL, NULL) != SOCKET_ERROR)
return g_io_channel_win32_new_socket(fd);
return g_io_channel_win32_new_socket (fd);
g_warning (G_STRLOC ": %d is neither a file descriptor or a socket", fd);
return NULL;