GSocket: fix a couple of return values
authorClaudio Saavedra <csaavedra@igalia.com>
Tue, 3 Jan 2012 22:18:59 +0000 (00:18 +0200)
committerClaudio Saavedra <csaavedra@igalia.com>
Wed, 4 Jan 2012 16:14:00 +0000 (18:14 +0200)
g_socket_receive_with_blocking() and g_socket_send_with_blocking claim
to return -1 in error, their return type is gssize, and yet they
return FALSE if the initial g_return_val_if_fail() call fails.

https://bugzilla.gnome.org/show_bug.cgi?id=667226

gio/gsocket.c

index 6bf1213..1679280 100644 (file)
@@ -1837,7 +1837,7 @@ g_socket_receive_with_blocking (GSocket       *socket,
 {
   gssize ret;
 
-  g_return_val_if_fail (G_IS_SOCKET (socket) && buffer != NULL, FALSE);
+  g_return_val_if_fail (G_IS_SOCKET (socket) && buffer != NULL, -1);
 
   if (!check_socket (socket, error))
     return -1;
@@ -2009,7 +2009,7 @@ g_socket_send_with_blocking (GSocket       *socket,
 {
   gssize ret;
 
-  g_return_val_if_fail (G_IS_SOCKET (socket) && buffer != NULL, FALSE);
+  g_return_val_if_fail (G_IS_SOCKET (socket) && buffer != NULL, -1);
 
   if (!check_socket (socket, error))
     return -1;