Changes in uspace/srv/net/tcp/sock.c [db81577:32d19f7] in mainline

File:

• uspace/srv/net/tcp/sock.c (modified) (7 diffs)

uspace/srv/net/tcp/sock.c

@@ -114 +114 @@
         *rsock = NULL;
 
-        sock = calloc(1, sizeof(tcp_sockdata_t));
+        sock = calloc(sizeof(tcp_sockdata_t), 1);
         if (sock == NULL)
                 return ENOMEM;
@@ -277 +277 @@
 
         socket->backlog = backlog;
-        socket->lconn = calloc(backlog, sizeof(tcp_conn_t *));
+        socket->lconn = calloc(sizeof(tcp_conn_t *), backlog);
         if (socket->lconn == NULL) {
                 fibril_mutex_unlock(&socket->lock);
@@ -293 +293 @@
         for (i = 0; i < backlog; i++) {
 
-                lconn = calloc(1, sizeof(tcp_sock_lconn_t));
+                lconn = calloc(sizeof(tcp_sock_lconn_t), 1);
                 if (lconn == NULL) {
                         /* XXX Clean up */
@@ -779 +779 @@
         }
 
-        /* Grab recv_buffer_lock because of CV wait in tcp_sock_recv_fibril() */
-        fibril_mutex_lock(&socket->recv_buffer_lock);
-        socket->sock_core = NULL;
-        fibril_mutex_unlock(&socket->recv_buffer_lock);
-
         rc = socket_destroy(NULL, socket_id, &client->sockets, &gsock,
             tcp_free_sock_data);
@@ -844 +839 @@
         log_msg(LVL_DEBUG, "tcp_sock_recv_fibril()");
 
-        fibril_mutex_lock(&sock->recv_buffer_lock);
-
         while (true) {
                 log_msg(LVL_DEBUG, "call tcp_uc_receive()");
-                while (sock->recv_buffer_used != 0 && sock->sock_core != NULL)
+                fibril_mutex_lock(&sock->recv_buffer_lock);
+                while (sock->recv_buffer_used != 0)
                         fibril_condvar_wait(&sock->recv_buffer_cv,
                             &sock->recv_buffer_lock);
@@ -858 +852 @@
                         sock->recv_error = trc;
                         fibril_condvar_broadcast(&sock->recv_buffer_cv);
-                        if (sock->sock_core != NULL)
-                                tcp_sock_notify_data(sock->sock_core);
+                        fibril_mutex_unlock(&sock->recv_buffer_lock);
+                        tcp_sock_notify_data(sock->sock_core);
                         break;
                 }
@@ -867 +861 @@
                 sock->recv_buffer_used = data_len;
                 fibril_condvar_broadcast(&sock->recv_buffer_cv);
-                if (sock->sock_core != NULL)
-                        tcp_sock_notify_data(sock->sock_core);
-        }
-
-        fibril_mutex_unlock(&sock->recv_buffer_lock);
+                fibril_mutex_unlock(&sock->recv_buffer_lock);
+                tcp_sock_notify_data(sock->sock_core);
+        }
 
         tcp_uc_delete(sock->conn);