Changes in uspace/lib/c/generic/net/socket_client.c [16ac756:e721462] in mainline

File:

• uspace/lib/c/generic/net/socket_client.c (modified) (14 diffs)

uspace/lib/c/generic/net/socket_client.c

@@ -43 +43 @@
 #include <stdlib.h>
 #include <errno.h>
+#include <err.h>
 
 #include <ipc/services.h>
@@ -211 +212 @@
 static void socket_connection(ipc_callid_t iid, ipc_call_t * icall)
 {
+        ERROR_DECLARE;
+
         ipc_callid_t callid;
         ipc_call_t call;
         socket_ref socket;
-        int rc;
 
 loop:
@@ -229 +231 @@
                     SOCKET_GET_SOCKET_ID(call));
                 if (!socket) {
-                        rc = ENOTSOCK;
+                        ERROR_CODE = ENOTSOCK;
                         fibril_rwlock_read_unlock(&socket_globals.lock);
                         break;
@@ -238 +240 @@
                         fibril_mutex_lock(&socket->receive_lock);
                         // push the number of received packet fragments
-                        rc = dyn_fifo_push(&socket->received,
+                        if (!ERROR_OCCURRED(dyn_fifo_push(&socket->received,
                             SOCKET_GET_DATA_FRAGMENTS(call),
-                            SOCKET_MAX_RECEIVED_SIZE);
-                        if (rc == EOK) {
+                            SOCKET_MAX_RECEIVED_SIZE))) {
                                 // signal the received packet
                                 fibril_condvar_signal(&socket->receive_signal);
@@ -251 +252 @@
                         // push the new socket identifier
                         fibril_mutex_lock(&socket->accept_lock);
-                        rc = dyn_fifo_push(&socket->accepted, 1,
-                            SOCKET_MAX_ACCEPTED_SIZE);
-                        if (rc != EOK) {
+                        if (!ERROR_OCCURRED(dyn_fifo_push(&socket->accepted,
+                            1, SOCKET_MAX_ACCEPTED_SIZE))) {
                                 // signal the accepted socket
                                 fibril_condvar_signal(&socket->accept_signal);
@@ -261 +261 @@
 
                 default:
-                        rc = ENOTSUP;
+                        ERROR_CODE = ENOTSUP;
                 }
 
@@ -280 +280 @@
 
         default:
-                rc = ENOTSUP;
-        }
-
-        ipc_answer_0(callid, (ipcarg_t) rc);
+                ERROR_CODE = ENOTSUP;
+        }
+
+        ipc_answer_0(callid, (ipcarg_t) ERROR_CODE);
         goto loop;
 }
@@ -405 +405 @@
 int socket(int domain, int type, int protocol)
 {
+        ERROR_DECLARE;
+
         socket_ref socket;
         int phone;
@@ -411 +413 @@
         ipcarg_t fragment_size;
         ipcarg_t header_size;
-        int rc;
 
         // find the appropriate service
@@ -478 +479 @@
         }
 
-        rc = (int) async_req_3_3(phone, NET_SOCKET, socket_id, 0, service, NULL,
-            &fragment_size, &header_size);
-        if (rc != EOK) {
+        if (ERROR_OCCURRED((int) async_req_3_3(phone, NET_SOCKET, socket_id, 0,
+            service, NULL, &fragment_size, &header_size))) {
                 fibril_rwlock_write_unlock(&socket_globals.lock);
                 free(socket);
-                return rc;
+                return ERROR_CODE;
         }
 
@@ -492 +492 @@
         socket_initialize(socket, socket_id, phone, service);
         // store the new socket
-        rc = sockets_add(socket_get_sockets(), socket_id, socket);
+        ERROR_CODE = sockets_add(socket_get_sockets(), socket_id, socket);
 
         fibril_rwlock_write_unlock(&socket_globals.lock);
-        if (rc < 0) {
+        if (ERROR_CODE < 0) {
                 dyn_fifo_destroy(&socket->received);
                 dyn_fifo_destroy(&socket->accepted);
@@ -501 +501 @@
                 async_msg_3(phone, NET_SOCKET_CLOSE, (ipcarg_t) socket_id, 0,
                     service);
-                return rc;
+                return ERROR_CODE;
         }
 
@@ -770 +770 @@
 int closesocket(int socket_id)
 {
+        ERROR_DECLARE;
+
         socket_ref socket;
-        int rc;
 
         fibril_rwlock_write_lock(&socket_globals.lock);
@@ -786 +787 @@
 
         // request close
-        rc = (int) async_req_3_0(socket->phone, NET_SOCKET_CLOSE,
-            (ipcarg_t) socket->socket_id, 0, socket->service);
-        if (rc != EOK) {
-                fibril_rwlock_write_unlock(&socket_globals.lock);
-                return rc;
-        }
+        ERROR_PROPAGATE((int) async_req_3_0(socket->phone, NET_SOCKET_CLOSE,
+            (ipcarg_t) socket->socket_id, 0, socket->service));
         // free the socket structure
         socket_destroy(socket);