add keywords from metadata for files
[oweals/gnunet.git] / src / util / server.c
index d518cbeec2e6f60a2c488b58ee654e2b212eefac..f586e42043618ab0122292f1f69a74093ec40122 100644 (file)
  * @file util/server.c
  * @brief library for building GNUnet network servers
  * @author Christian Grothoff
- *
- * TODO:
- * - fix inefficient memmove in message processing
  */
 
 #include "platform.h"
 #include "gnunet_common.h"
-#include "gnunet_network_lib.h"
+#include "gnunet_connection_lib.h"
 #include "gnunet_scheduler_lib.h"
 #include "gnunet_server_lib.h"
 #include "gnunet_time_lib.h"
+#include "gnunet_disk_lib.h"
+#include "gnunet_protocols.h"
 
 #define DEBUG_SERVER GNUNET_NO
 
@@ -80,11 +79,6 @@ struct NotifyList
  */
 struct GNUNET_SERVER_Handle
 {
-  /**
-   * My scheduler.
-   */
-  struct GNUNET_SCHEDULER_Handle *sched;
-
   /**
    * List of handlers for incoming messages.
    */
@@ -103,7 +97,7 @@ struct GNUNET_SERVER_Handle
   /**
    * Function to call for access control.
    */
-  GNUNET_NETWORK_AccessCheck access;
+  GNUNET_CONNECTION_AccessCheck access;
 
   /**
    * Closure for access.
@@ -111,38 +105,34 @@ struct GNUNET_SERVER_Handle
   void *access_cls;
 
   /**
-   * After how long should an idle connection time
-   * out (on write).
+   * NULL-terminated array of sockets used to listen for new
+   * connections.
    */
-  struct GNUNET_TIME_Relative idle_timeout;
+  struct GNUNET_NETWORK_Handle **listen_sockets;
 
   /**
-   * maximum write buffer size for accepted sockets
+   * After how long should an idle connection time
+   * out (on write).
    */
-  size_t maxbuf;
+  struct GNUNET_TIME_Relative idle_timeout;
 
   /**
-   * Pipe used to signal shutdown of the server.
+   * Task scheduled to do the listening.
    */
-  int shutpipe[2];
+  GNUNET_SCHEDULER_TaskIdentifier listen_task;
 
   /**
-   * Socket used to listen for new connections.  Set to
-   * "-1" by GNUNET_SERVER_destroy to initiate shutdown.
+   * Do we ignore messages of types that we do not understand or do we
+   * require that a handler is found (and if not kill the connection)?
    */
-  int listen_socket;
+  int require_found;
 
   /**
-   * Set to GNUNET_YES if we are shutting down.
+   * Should all of the clients of this server continue to process
+   * connections as usual even if we get a shutdown request? (the
+   * listen socket always ignores shutdown).
    */
-  int do_shutdown;
-
-  /**
-   * Do we ignore messages of types that we do not
-   * understand or do we require that a handler
-   * is found (and if not kill the connection)?
-   */
-  int require_found;
+  int clients_ignore_shutdown;
 
 };
 
@@ -154,15 +144,14 @@ struct GNUNET_SERVER_Client
 {
 
   /**
-   * Size of the buffer for incoming data.  Should be
-   * first so we get nice alignment.
+   * This is a linked list.
    */
-  char incoming_buffer[GNUNET_SERVER_MAX_MESSAGE_SIZE];
+  struct GNUNET_SERVER_Client *next;
 
   /**
-   * This is a linked list.
+   * Processing of incoming data.
    */
-  struct GNUNET_SERVER_Client *next;
+  struct GNUNET_SERVER_MessageStreamTokenizer *mst;
 
   /**
    * Server that this client belongs to.
@@ -172,48 +161,12 @@ struct GNUNET_SERVER_Client
   /**
    * Client closure for callbacks.
    */
-  void *client_closure;
-
-  /**
-   * Callback to receive from client.
-   */
-  GNUNET_SERVER_ReceiveCallback receive;
-
-  /**
-   * Callback to cancel receive from client.
-   */
-  GNUNET_SERVER_ReceiveCancelCallback receive_cancel;
-
-  /**
-   * Callback to ask about transmit-ready notification.
-   */
-  GNUNET_SERVER_TransmitReadyCallback notify_transmit_ready;
-
-   /**
-   * Callback to ask about transmit-ready notification.
-   */
-  GNUNET_SERVER_TransmitReadyCancelCallback notify_transmit_ready_cancel;
-
-  /**
-   * Callback to check if client is still valid.
-   */
-  GNUNET_SERVER_CheckCallback check;
-
-  /**
-   * Callback to destroy client.
-   */
-  GNUNET_SERVER_DestroyCallback destroy;
+  struct GNUNET_CONNECTION_Handle *connection;
 
   /**
-   * Side-buffer for incoming data used when processing
-   * is suspended.
+   * ID of task used to restart processing.
    */
-  char *side_buf;
-
-  /**
-   * Number of bytes in the side buffer.
-   */
-  size_t side_buf_size;
+  GNUNET_SCHEDULER_TaskIdentifier restart_task;
 
   /**
    * Last activity on this socket (used to time it out
@@ -221,18 +174,6 @@ struct GNUNET_SERVER_Client
    */
   struct GNUNET_TIME_Absolute last_activity;
 
-  /**
-   * Current task identifier for the receive call
-   * (or GNUNET_SCHEDULER_NO_TASK for none).
-   */
-  GNUNET_SCHEDULER_TaskIdentifier my_receive;
-
-  /**
-   * How many bytes in the "incoming_buffer" are currently
-   * valid? (starting at offset 0).
-   */
-  size_t receive_pos;
-
   /**
    * Number of external entities with a reference to
    * this client object.
@@ -263,113 +204,105 @@ struct GNUNET_SERVER_Client
    */
   int shutdown_now;
 
-};
-
-
-/**
- * Server has been asked to shutdown, free resources.
- */
-static void
-destroy_server (struct GNUNET_SERVER_Handle *server)
-{
-  struct GNUNET_SERVER_Client *pos;
-  struct HandlerList *hpos;
-  struct NotifyList *npos;
+  /**
+   * Are we currently trying to receive? (YES if we are, NO if we are not,
+   * SYSERR if data is already available in MST).
+   */
+  int receive_pending;
 
-#if DEBUG_SERVER
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-             "Server shutting down.\n");
-#endif
-  GNUNET_assert (server->listen_socket == -1);
-  GNUNET_break (0 == CLOSE (server->shutpipe[0]));
-  GNUNET_break (0 == CLOSE (server->shutpipe[1]));
-  while (server->clients != NULL)
-    {
-      pos = server->clients;
-      server->clients = pos->next;
-      pos->server = NULL;
-    }
-  while (NULL != (hpos = server->handlers))
-    {
-      server->handlers = hpos->next;
-      GNUNET_free (hpos);
-    }
-  while (NULL != (npos = server->disconnect_notify_list))
-    {
-      server->disconnect_notify_list = npos->next;
-      GNUNET_free (npos);
-    }
-  GNUNET_free (server);
-}
+  /**
+   * Persist the file handle for this client no matter what happens,
+   * force the OS to close once the process actually dies.  Should only
+   * be used in special cases!
+   */
+  int persist;
+};
 
 
 /**
- * Scheduler says our listen socket is ready.
- * Process it!
+ * Scheduler says our listen socket is ready.  Process it!
+ *
+ * @param cls handle to our server for which we are processing the listen
+ *        socket
+ * @param tc reason why we are running right now
  */
 static void
 process_listen_socket (void *cls,
                        const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   struct GNUNET_SERVER_Handle *server = cls;
-  struct GNUNET_NETWORK_ConnectionHandle *sock;
+  struct GNUNET_CONNECTION_Handle *sock;
   struct GNUNET_SERVER_Client *client;
-  fd_set r;
+  struct GNUNET_NETWORK_FDSet *r;
+  unsigned int i;
 
-  if ((server->do_shutdown) ||
-      ((tc->reason & GNUNET_SCHEDULER_REASON_SHUTDOWN) != 0))
+  server->listen_task = GNUNET_SCHEDULER_NO_TASK;
+  r = GNUNET_NETWORK_fdset_create ();
+  i = 0;
+  while (NULL != server->listen_sockets[i])
+    GNUNET_NETWORK_fdset_set (r, server->listen_sockets[i++]);
+  if (0 != (tc->reason & GNUNET_SCHEDULER_REASON_SHUTDOWN))
     {
-      /* shutdown was initiated */
-      GNUNET_assert (server->listen_socket != -1);
-      GNUNET_break (0 == CLOSE (server->listen_socket));
-      server->listen_socket = -1;
-      if (server->do_shutdown)
-        destroy_server (server);
+      /* ignore shutdown, someone else will take care of it! */
+      server->listen_task = GNUNET_SCHEDULER_add_select (GNUNET_SCHEDULER_PRIORITY_HIGH,
+                                                         GNUNET_SCHEDULER_NO_TASK,
+                                                         GNUNET_TIME_UNIT_FOREVER_REL,
+                                                         r, NULL,
+                                                         &process_listen_socket,
+                                                         server);
+      GNUNET_NETWORK_fdset_destroy (r);
       return;
     }
-  GNUNET_assert (FD_ISSET (server->listen_socket, tc->read_ready));
-  GNUNET_assert (!FD_ISSET (server->shutpipe[0], tc->read_ready));
-  sock = GNUNET_NETWORK_connection_create_from_accept (tc->sched,
-                                                   server->access,
-                                                   server->access_cls,
-                                                   server->listen_socket,
-                                                   server->maxbuf);
-  if (sock != NULL)
+  i = 0;
+  while (NULL != server->listen_sockets[i])
     {
+      if (GNUNET_NETWORK_fdset_isset
+          (tc->read_ready, server->listen_sockets[i]))
+        {
+          sock =
+            GNUNET_CONNECTION_create_from_accept (server->access,
+                                                  server->access_cls,
+                                                  server->listen_sockets[i]);
+          if (sock != NULL)
+            {
 #if DEBUG_SERVER
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                 "Server accepted incoming connection.\n");
+              GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                          "Server accepted incoming connection.\n");
 #endif
-      client = GNUNET_SERVER_connect_socket (server, sock);
-      /* decrement reference count, we don't keep "client" alive */
-      GNUNET_SERVER_client_drop (client);
+              client = GNUNET_SERVER_connect_socket (server, sock);
+              GNUNET_CONNECTION_ignore_shutdown (sock,
+                                                 server->clients_ignore_shutdown);
+              /* decrement reference count, we don't keep "client" alive */
+              GNUNET_SERVER_client_drop (client);
+            }
+        }
+      i++;
     }
   /* listen for more! */
-  FD_ZERO (&r);
-  FD_SET (server->listen_socket, &r);
-  FD_SET (server->shutpipe[0], &r);
-  GNUNET_SCHEDULER_add_select (server->sched,
-                               GNUNET_YES,
-                               GNUNET_SCHEDULER_PRIORITY_HIGH,
-                               GNUNET_SCHEDULER_NO_TASK,
-                               GNUNET_TIME_UNIT_FOREVER_REL,
-                               GNUNET_MAX (server->listen_socket,
-                                           server->shutpipe[0]) + 1, &r, NULL,
-                               &process_listen_socket, server);
+  server->listen_task = GNUNET_SCHEDULER_add_select (GNUNET_SCHEDULER_PRIORITY_HIGH,
+                                                     GNUNET_SCHEDULER_NO_TASK,
+                                                     GNUNET_TIME_UNIT_FOREVER_REL,
+                                                     r, NULL,
+                                                     &process_listen_socket,
+                                                     server);
+  GNUNET_NETWORK_fdset_destroy (r);
 }
 
 
 /**
  * Create and initialize a listen socket for the server.
  *
- * @return -1 on error, otherwise the listen socket
+ * @param serverAddr address to listen on
+ * @param socklen length of address
+ * @return NULL on error, otherwise the listen socket
  */
-static int
+static struct GNUNET_NETWORK_Handle *
 open_listen_socket (const struct sockaddr *serverAddr, socklen_t socklen)
 {
   const static int on = 1;
-  int fd;
+  struct GNUNET_NETWORK_Handle *sock;
   uint16_t port;
+  int eno;
 
   switch (serverAddr->sa_family)
     {
@@ -380,60 +313,75 @@ open_listen_socket (const struct sockaddr *serverAddr, socklen_t socklen)
       port = ntohs (((const struct sockaddr_in6 *) serverAddr)->sin6_port);
       break;
     default:
-      GNUNET_break (0);
-      return -1;
+      port = 0;
+      break;
     }
-  fd = SOCKET (serverAddr->sa_family, SOCK_STREAM, 0);
-  if (fd < 0)
+  sock = GNUNET_NETWORK_socket_create (serverAddr->sa_family, SOCK_STREAM, 0);
+  if (NULL == sock)
     {
       GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "socket");
-      return -1;
+      errno = 0;
+      return NULL;
     }
-#ifndef MINGW
-  // FIXME NILS
-  if (0 != fcntl (fd, F_SETFD, fcntl (fd, F_GETFD) | FD_CLOEXEC))
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR | GNUNET_ERROR_TYPE_BULK,
-                         "fcntl");
+  if (port != 0) 
+    {
+      if (GNUNET_NETWORK_socket_setsockopt
+         (sock, SOL_SOCKET, SO_REUSEADDR, &on, sizeof (on)) != GNUNET_OK)
+       GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR | GNUNET_ERROR_TYPE_BULK,
+                            "setsockopt");
+#ifdef IPV6_V6ONLY
+      if ( (serverAddr->sa_family == AF_INET6) &&
+          (GNUNET_NETWORK_socket_setsockopt
+           (sock, IPPROTO_IPV6, IPV6_V6ONLY, &on, sizeof (on)) != GNUNET_OK) )
+       GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR | GNUNET_ERROR_TYPE_BULK,
+                            "setsockopt");
 #endif
-  if (SETSOCKOPT (fd, SOL_SOCKET, SO_REUSEADDR, &on, sizeof (on)) < 0)
-    GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR | GNUNET_ERROR_TYPE_BULK,
-                         "setsockopt");
+    }
   /* bind the socket */
-  if (BIND (fd, serverAddr, socklen) < 0)
+  if (GNUNET_NETWORK_socket_bind (sock, serverAddr, socklen) != GNUNET_OK)
     {
-      GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "bind");
-      GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
-                  _
-                  ("`%s' failed for port %d. Is the service already running?\n"),
-                  "bind", port);
-      GNUNET_break (0 == CLOSE (fd));
-      return -1;
+      eno = errno;
+      if (errno != EADDRINUSE)
+        {
+          /* we don't log 'EADDRINUSE' here since an IPv4 bind may
+             fail if we already took the port on IPv6; if both IPv4 and
+             IPv6 binds fail, then our caller will log using the
+             errno preserved in 'eno' */
+          GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "bind");
+          if (port != 0)
+            GNUNET_log (GNUNET_ERROR_TYPE_ERROR,
+                        _
+                        ("`%s' failed for port %d (%s).\n"),
+                        "bind", port,
+                        (serverAddr->sa_family == AF_INET) ? "IPv4" : "IPv6");
+          eno = 0;
+        }
+      GNUNET_break (GNUNET_OK == GNUNET_NETWORK_socket_close (sock));
+      errno = eno;
+      return NULL;
     }
-  if (0 != LISTEN (fd, 5))
+  if (GNUNET_OK != GNUNET_NETWORK_socket_listen (sock, 5))
     {
       GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "listen");
-      GNUNET_break (0 == CLOSE (fd));
-      return -1;
+      GNUNET_break (GNUNET_OK == GNUNET_NETWORK_socket_close (sock));
+      errno = 0;
+      return NULL;
     }
 #if DEBUG_SERVER
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                 "Server starts to listen on port %u.\n",
-                 port);
+  if (port != 0)
+    GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                "Server starts to listen on port %u.\n", port);
 #endif
-  return fd;
+  return sock;
 }
 
 
 /**
  * Create a new server.
  *
- * @param sched scheduler to use
  * @param access function for access control
  * @param access_cls closure for access
- * @param serverAddr address to listen on (including port), use NULL
- *        for internal server (no listening)
- * @param socklen length of serverAddr
- * @param maxbuf maximum write buffer size for accepted sockets
+ * @param lsocks NULL-terminated array of listen sockets
  * @param idle_timeout after how long should we timeout idle connections?
  * @param require_found if YES, connections sending messages of unknown type
  *        will be closed
@@ -441,72 +389,143 @@ open_listen_socket (const struct sockaddr *serverAddr, socklen_t socklen)
  *         (typically, "port" already in use)
  */
 struct GNUNET_SERVER_Handle *
-GNUNET_SERVER_create (struct GNUNET_SCHEDULER_Handle *sched,
-                      GNUNET_NETWORK_AccessCheck access,
-                      void *access_cls,
-                      const struct sockaddr *serverAddr,
-                      socklen_t socklen,
-                      size_t maxbuf,
-                      struct GNUNET_TIME_Relative
-                      idle_timeout, int require_found)
+GNUNET_SERVER_create_with_sockets (GNUNET_CONNECTION_AccessCheck access, void *access_cls,
+                                  struct GNUNET_NETWORK_Handle **lsocks,
+                                  struct GNUNET_TIME_Relative
+                                  idle_timeout,
+                                  int require_found)
 {
   struct GNUNET_SERVER_Handle *ret;
-  int lsock;
-  fd_set r;
+  struct GNUNET_NETWORK_FDSet *r;
+  int i;
 
-  lsock = -2;
-  if (serverAddr != NULL)
-    {
-      lsock = open_listen_socket (serverAddr, socklen);
-      if (lsock == -1)
-        return NULL;
-    }
   ret = GNUNET_malloc (sizeof (struct GNUNET_SERVER_Handle));
-  if (0 != PIPE (ret->shutpipe))
-    {
-      GNUNET_break (0 == CLOSE (lsock));
-      GNUNET_free (ret);
-      return NULL;
-    }
-  ret->sched = sched;
-  ret->maxbuf = maxbuf;
   ret->idle_timeout = idle_timeout;
-  ret->listen_socket = lsock;
+  ret->listen_sockets = lsocks;
   ret->access = access;
   ret->access_cls = access_cls;
   ret->require_found = require_found;
-  if (lsock >= 0)
+  if (lsocks != NULL)
     {
-      FD_ZERO (&r);
-      FD_SET (ret->listen_socket, &r);
-      FD_SET (ret->shutpipe[0], &r);
-      GNUNET_SCHEDULER_add_select (sched,
-                                   GNUNET_YES,
-                                   GNUNET_SCHEDULER_PRIORITY_HIGH,
-                                   GNUNET_SCHEDULER_NO_TASK,
-                                   GNUNET_TIME_UNIT_FOREVER_REL,
-                                   GNUNET_MAX (ret->listen_socket,
-                                               ret->shutpipe[0]) + 1, &r,
-                                   NULL, &process_listen_socket, ret);
+      r = GNUNET_NETWORK_fdset_create ();
+      i = 0;
+      while (NULL != ret->listen_sockets[i])
+        GNUNET_NETWORK_fdset_set (r, ret->listen_sockets[i++]);
+      ret->listen_task = GNUNET_SCHEDULER_add_select (GNUNET_SCHEDULER_PRIORITY_HIGH,
+                                                      GNUNET_SCHEDULER_NO_TASK,
+                                                      GNUNET_TIME_UNIT_FOREVER_REL,
+                                                      r, NULL,
+                                                      &process_listen_socket,
+                                                      ret);
+      GNUNET_NETWORK_fdset_destroy (r);
     }
   return ret;
 }
 
 
+/**
+ * Create a new server.
+ *
+ * @param access function for access control
+ * @param access_cls closure for access
+ * @param serverAddr address to listen on (including port), NULL terminated array
+ * @param socklen length of serverAddr
+ * @param idle_timeout after how long should we timeout idle connections?
+ * @param require_found if YES, connections sending messages of unknown type
+ *        will be closed
+ * @return handle for the new server, NULL on error
+ *         (typically, "port" already in use)
+ */
+struct GNUNET_SERVER_Handle *
+GNUNET_SERVER_create (GNUNET_CONNECTION_AccessCheck access,
+                      void *access_cls,
+                      struct sockaddr *const *serverAddr,
+                      const socklen_t * socklen,
+                      struct GNUNET_TIME_Relative
+                      idle_timeout, int require_found)
+{
+  struct GNUNET_NETWORK_Handle **lsocks;
+  unsigned int i;
+  unsigned int j;
+
+  i = 0;
+  while (serverAddr[i] != NULL)
+    i++;
+  if (i > 0)
+    {
+      lsocks =
+        GNUNET_malloc (sizeof (struct GNUNET_NETWORK_Handle *) * (i + 1));
+      i = 0;
+      j = 0;
+      while (serverAddr[i] != NULL)
+        {
+          lsocks[j] = open_listen_socket (serverAddr[i], socklen[i]);
+          if (lsocks[j] != NULL)
+            j++;
+          i++;
+        }
+      if (j == 0)
+        {
+          if (errno != 0)
+           GNUNET_log_strerror (GNUNET_ERROR_TYPE_ERROR, "bind");          
+          GNUNET_free (lsocks);
+          lsocks = NULL;
+        }
+    }
+  else
+    {
+      lsocks = NULL;
+    }
+  return GNUNET_SERVER_create_with_sockets (access, access_cls,
+                                           lsocks,
+                                           idle_timeout,
+                                           require_found);
+}
+
+
 /**
  * Free resources held by this server.
+ *
+ * @param s server to destroy
  */
 void
 GNUNET_SERVER_destroy (struct GNUNET_SERVER_Handle *s)
 {
-  static char c;
+  struct HandlerList *hpos;
+  struct NotifyList *npos;
+  unsigned int i;
 
-  GNUNET_assert (s->do_shutdown == GNUNET_NO);
-  s->do_shutdown = GNUNET_YES;
-  if (s->listen_socket == -1)
-    destroy_server (s);
-  else
-    GNUNET_break (1 == WRITE (s->shutpipe[1], &c, 1));
+#if DEBUG_SERVER
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Server shutting down.\n");
+#endif
+  if (GNUNET_SCHEDULER_NO_TASK != s->listen_task)
+    {
+      GNUNET_SCHEDULER_cancel (s->listen_task);
+      s->listen_task = GNUNET_SCHEDULER_NO_TASK;
+    }
+  if (s->listen_sockets != NULL)
+    {
+      i = 0;
+      while (s->listen_sockets[i] != NULL)
+        GNUNET_break (GNUNET_OK ==
+                      GNUNET_NETWORK_socket_close (s->listen_sockets[i++]));
+      GNUNET_free (s->listen_sockets);
+      s->listen_sockets = NULL;
+    }
+  while (s->clients != NULL)
+    GNUNET_SERVER_client_disconnect (s->clients);
+  while (NULL != (hpos = s->handlers))
+    {
+      s->handlers = hpos->next;
+      GNUNET_free (hpos);
+    }
+  while (NULL != (npos = s->disconnect_notify_list))
+    {
+      npos->callback (npos->callback_cls, NULL);
+      s->disconnect_notify_list = npos->next;
+      GNUNET_free (npos);
+    }
+  GNUNET_free (s);
 }
 
 
@@ -568,9 +587,8 @@ GNUNET_SERVER_inject (struct GNUNET_SERVER_Handle *server,
   size = ntohs (message->size);
 #if DEBUG_SERVER
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-             "Server schedules transmission of %u-byte message of type %u to client.\n",
-             size,
-             type);
+              "Server schedules transmission of %u-byte message of type %u to client.\n",
+              size, type);
 #endif
   pos = server->handlers;
   found = GNUNET_NO;
@@ -580,11 +598,19 @@ GNUNET_SERVER_inject (struct GNUNET_SERVER_Handle *server,
       while (pos->handlers[i].callback != NULL)
         {
           mh = &pos->handlers[i];
-          if (mh->type == type)
+          if ( (mh->type == type) ||
+               (mh->type == GNUNET_MESSAGE_TYPE_ALL) )
             {
               if ((mh->expected_size != 0) && (mh->expected_size != size))
                 {
+#if GNUNET8_NETWORK_IS_DEAD
+                 GNUNET_log (GNUNET_ERROR_TYPE_WARNING,
+                             "Expected %u bytes for message of type %u, got %u\n",
+                             mh->expected_size,
+                             mh->type,
+                             size);
                   GNUNET_break_op (0);
+#endif
                   return GNUNET_SYSERR;
                 }
               if (sender != NULL)
@@ -599,7 +625,8 @@ GNUNET_SERVER_inject (struct GNUNET_SERVER_Handle *server,
   if (found == GNUNET_NO)
     {
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG | GNUNET_ERROR_TYPE_BULK,
-                  _("Received message of unknown type %d\n"), type);
+                  "Received message of unknown type %d\n",
+                 type);
       if (server->require_found == GNUNET_YES)
         return GNUNET_SYSERR;
     }
@@ -608,112 +635,84 @@ GNUNET_SERVER_inject (struct GNUNET_SERVER_Handle *server,
 
 
 /**
- * We're finished with this client and especially its input
- * processing.  If the RC is zero, free all resources otherwise wait
- * until RC hits zero to do so.
+ * We are receiving an incoming message.  Process it.
+ *
+ * @param cls our closure (handle for the client)
+ * @param buf buffer with data received from network
+ * @param available number of bytes available in buf
+ * @param addr address of the sender
+ * @param addrlen length of addr
+ * @param errCode code indicating errors receiving, 0 for success
  */
 static void
-shutdown_incoming_processing (struct GNUNET_SERVER_Client *client)
-{
-  struct GNUNET_SERVER_Client *prev;
-  struct GNUNET_SERVER_Client *pos;
-  struct GNUNET_SERVER_Handle *server;
-  struct NotifyList *n;
-  unsigned int rc;
-
-  GNUNET_assert (client->my_receive == GNUNET_SCHEDULER_NO_TASK);
-  rc = client->reference_count;
-  if (client->server != NULL)
-    {
-      server = client->server;
-      client->server = NULL;
-      prev = NULL;
-      pos = server->clients;
-      while ((pos != NULL) && (pos != client))
-        {
-          prev = pos;
-          pos = pos->next;
-        }
-      GNUNET_assert (pos != NULL);
-      if (prev == NULL)
-        server->clients = pos->next;
-      else
-        prev->next = pos->next;
-      n = server->disconnect_notify_list;
-      while (n != NULL)
-        {
-          n->callback (n->callback_cls, client);
-          n = n->next;
-        }
-    }
-  /* wait for RC to hit zero, then free */
-  if (rc > 0)
-    return;
-  client->destroy (client->client_closure);
-  GNUNET_free (client);
-}
+process_incoming (void *cls,
+                  const void *buf,
+                  size_t available,
+                  const struct sockaddr *addr, 
+                 socklen_t addrlen, int errCode);
 
 
 /**
- * Go over the contents of the client buffer; as long as full messages
- * are available, pass them on for processing.  Update the buffer
- * accordingly.  Handles fatal errors by shutting down the connection.
+ * Process messages from the client's message tokenizer until either
+ * the tokenizer is empty (and then schedule receiving more), or 
+ * until some handler is not immediately done (then wait for restart_processing)
+ * or shutdown.
  *
- * @param client identifies which client receive buffer to process
+ * @param client the client to process, RC must have already been increased
+ *        using GNUNET_SERVER_client_keep and will be decreased by one in this
+ *        function
+ * @param ret GNUNET_NO to start processing from the buffer,
+ *            GNUNET_OK if the mst buffer is drained and we should instantly go back to receiving
+ *            GNUNET_SYSERR if we should instantly abort due to error in a previous step
  */
 static void
-process_client_buffer (struct GNUNET_SERVER_Client *client)
+process_mst (struct GNUNET_SERVER_Client *client,
+            int ret)
 {
-  struct GNUNET_SERVER_Handle *server;
-  const struct GNUNET_MessageHeader *hdr;
-  size_t msize;
-
-  client->in_process_client_buffer = GNUNET_YES;
-  server = client->server;
-#if DEBUG_SERVER
-  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-             "Private buffer contains %u bytes; client is %s and we are %s\n",
-             client->receive_pos,
-             client->suspended ? "suspended" : "up",
-             client->shutdown_now ? "in shutdown" : "running");
-#endif
-  while ((client->receive_pos >= sizeof (struct GNUNET_MessageHeader)) &&
-         (0 == client->suspended) && (GNUNET_YES != client->shutdown_now))
+  while ( (ret != GNUNET_SYSERR) && 
+         (client->server != NULL) &&
+         (GNUNET_YES != client->shutdown_now) &&
+         (0 == client->suspended) )
     {
-      hdr = (const struct GNUNET_MessageHeader *) &client->incoming_buffer;
-      msize = ntohs (hdr->size);
-      if (msize > client->receive_pos)
+      if (ret == GNUNET_OK)
        {
+         client->receive_pending = GNUNET_YES;
 #if DEBUG_SERVER
          GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                     "Total message size is %u, we only have %u bytes; need more data\n",
-                     msize,
-                     client->receive_pos);
+                     "Server re-enters receive loop.\n");
 #endif
+         GNUNET_CONNECTION_receive (client->connection,
+                                    GNUNET_SERVER_MAX_MESSAGE_SIZE - 1,
+                                    client->server->idle_timeout, 
+                                    &process_incoming, client);
          break;
        }
+#if DEBUG_SERVER
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG, "Server processes additional messages instantly.\n");
+#endif
+      ret = GNUNET_SERVER_mst_receive (client->mst, client, NULL, 0, GNUNET_NO, GNUNET_YES);
+    }
+#if DEBUG_SERVER
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Server leaves instant processing loop: ret = %d, server = %p, shutdown = %d, suspended = %u\n",
+             ret,
+             client->server,
+             client->shutdown_now,
+             client->suspended);
+#endif
+
+  if (ret == GNUNET_NO)
+    {
 #if DEBUG_SERVER
       GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                 "Passing %u bytes to callback for processing\n",
-                 msize);
+                 "Server has more data pending but is suspended.\n");
 #endif
-      if ((msize < sizeof (struct GNUNET_MessageHeader)) ||
-          (GNUNET_OK != GNUNET_SERVER_inject (server, client, hdr)))
-        {
-          client->in_process_client_buffer = GNUNET_NO;
-          shutdown_incoming_processing (client);
-          return;
-        }
-      /* FIXME: this is highly inefficient; we should
-         try to avoid this if the new base address is
-         already nicely aligned.  See old handler code... */
-      memmove (client->incoming_buffer,
-               &client->incoming_buffer[msize], client->receive_pos - msize);
-      client->receive_pos -= msize;
+      client->receive_pending = GNUNET_SYSERR; /* data pending */
     }
-  client->in_process_client_buffer = GNUNET_NO;
-  if (GNUNET_YES == client->shutdown_now)
-    shutdown_incoming_processing (client);
+  if ( (ret == GNUNET_SYSERR) ||
+       (GNUNET_YES == client->shutdown_now) )
+    GNUNET_SERVER_client_disconnect (client);
+  GNUNET_SERVER_client_drop (client);
 }
 
 
@@ -732,205 +731,108 @@ process_incoming (void *cls,
                   const void *buf,
                   size_t available,
                   const struct sockaddr *addr, 
-                 socklen_t addrlen,
-                 int errCode)
+                 socklen_t addrlen, int errCode)
 {
   struct GNUNET_SERVER_Client *client = cls;
   struct GNUNET_SERVER_Handle *server = client->server;
-  const char *cbuf = buf;
-  size_t maxcpy;
+  int ret;
 
-  client->my_receive = GNUNET_SCHEDULER_NO_TASK;
+  GNUNET_assert (client->receive_pending == GNUNET_YES);
+  client->receive_pending = GNUNET_NO;
   if ((buf == NULL) ||
       (available == 0) ||
       (errCode != 0) ||
       (server == NULL) ||
       (client->shutdown_now == GNUNET_YES) ||
-      (GNUNET_YES != client->check (client->client_closure)))
+      (GNUNET_YES != GNUNET_CONNECTION_check (client->connection)))
     {
       /* other side closed connection, error connecting, etc. */
-      shutdown_incoming_processing (client);
+      GNUNET_SERVER_client_disconnect (client);
       return;
     }
 #if DEBUG_SERVER
   GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-             "Server receives %u bytes from `%s'.\n",
-             available,
-             GNUNET_a2s(addr, addrlen));
+              "Server receives %u bytes from `%s'.\n",
+              (unsigned int) available, 
+             GNUNET_a2s (addr, addrlen));
 #endif
   GNUNET_SERVER_client_keep (client);
   client->last_activity = GNUNET_TIME_absolute_get ();
-  /* process data (if available) */
-  while (available > 0)
-    {
-      maxcpy = available;
-      if (maxcpy > sizeof (client->incoming_buffer) - client->receive_pos)
-        maxcpy = sizeof (client->incoming_buffer) - client->receive_pos;
-#if DEBUG_SERVER
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                 "Can copy %u bytes to private buffer\n",
-                 maxcpy);
-#endif
-      memcpy (&client->incoming_buffer[client->receive_pos], cbuf, maxcpy);
-      client->receive_pos += maxcpy;
-      cbuf += maxcpy;
-      available -= maxcpy;
-      if (0 < client->suspended)
-        {
-          if (available > 0)
-            {
-#if DEBUG_SERVER
-             GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                         "Client has suspended processing; copying %u bytes to side buffer to be used later.\n",
-                         available);
-#endif
-             GNUNET_assert (client->side_buf_size == 0);
-             GNUNET_assert (client->side_buf == NULL);
-              client->side_buf_size = available;
-              client->side_buf = GNUNET_malloc (available);
-              memcpy (client->side_buf, cbuf, available);
-              available = 0;
-            }
-          break;                /* do not run next client iteration! */
-        }
-#if DEBUG_SERVER
-      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
-                 "Now processing messages in private buffer\n");
-#endif
-      process_client_buffer (client);
-    }
-  GNUNET_assert (available == 0);
-  if ((client->suspended == 0) &&
-      (GNUNET_YES != client->shutdown_now) && (client->server != NULL))
-    {
-      /* Finally, keep receiving! */
-      client->my_receive = client->receive (client->client_closure,
-                                            GNUNET_SERVER_MAX_MESSAGE_SIZE,
-                                            server->idle_timeout,
-                                            &process_incoming, client);
-    }
-  if (GNUNET_YES == client->shutdown_now)
-    shutdown_incoming_processing (client);
-  GNUNET_SERVER_client_drop (client);
+  ret = GNUNET_SERVER_mst_receive (client->mst, client, buf, available, GNUNET_NO, GNUNET_YES);
+  process_mst (client, ret);
 }
 
 
 /**
- * FIXME: document.
+ * Task run to start again receiving from the network
+ * and process requests.
+ *
+ * @param cls our 'struct GNUNET_SERVER_Client*' to process more requests from
+ * @param tc scheduler context (unused)
  */
 static void
-restart_processing (void *cls, const struct GNUNET_SCHEDULER_TaskContext *tc)
+restart_processing (void *cls, 
+                   const struct GNUNET_SCHEDULER_TaskContext *tc)
 {
   struct GNUNET_SERVER_Client *client = cls;
+  struct GNUNET_SERVER_Handle *server = client->server;
 
-  process_client_buffer (client);
-  if (0 == client->suspended)
-    client->my_receive = client->receive (client->client_closure,
-                                          GNUNET_SERVER_MAX_MESSAGE_SIZE,
-                                          client->server->idle_timeout,
-                                          &process_incoming, client);
-}
-
-
-/**
- * Add a client to the set of our clients and
- * start receiving.
- */
-static void
-add_client (struct GNUNET_SERVER_Handle *server,
-            struct GNUNET_SERVER_Client *client)
-{
-  client->server = server;
-  client->last_activity = GNUNET_TIME_absolute_get ();
-  client->next = server->clients;
-  server->clients = client;
-  client->my_receive = client->receive (client->client_closure,
-                                        GNUNET_SERVER_MAX_MESSAGE_SIZE,
-                                        server->idle_timeout,
-                                        &process_incoming, client);
+  client->restart_task = GNUNET_SCHEDULER_NO_TASK;
+  if ( (0 != (tc->reason & GNUNET_SCHEDULER_REASON_SHUTDOWN)) &&
+       (GNUNET_NO == server->clients_ignore_shutdown) )
+    {
+      GNUNET_SERVER_client_disconnect (client);
+      return;
+    }  
+  if (client->receive_pending == GNUNET_NO)
+    {
+#if DEBUG_SERVER
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                 "Server begins to read again from client.\n");
+#endif
+      client->receive_pending = GNUNET_YES;
+      GNUNET_CONNECTION_receive (client->connection,
+                                GNUNET_SERVER_MAX_MESSAGE_SIZE - 1,
+                                client->server->idle_timeout, &process_incoming, client);
+      return;
+    }
+#if DEBUG_SERVER
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Server continues processing messages still in the buffer.\n");
+#endif
+  GNUNET_SERVER_client_keep (client);
+  client->receive_pending = GNUNET_NO;
+  process_mst (client, GNUNET_NO);
 }
 
 
 /**
- * Create a request for receiving data from a socket.
+ * This function is called whenever our inbound message tokenizer has
+ * received a complete message.
  *
- * @param cls identifies the socket to receive from
- * @param max how much data to read at most
- * @param timeout when should this operation time out
- * @param receiver function to call for processing
- * @param receiver_cls closure for receiver
- * @return task identifier that can be used to cancel the operation
- */
-static GNUNET_SCHEDULER_TaskIdentifier
-sock_receive (void *cls,
-              size_t max,
-              struct GNUNET_TIME_Relative timeout,
-              GNUNET_NETWORK_Receiver receiver, void *receiver_cls)
-{
-  return GNUNET_NETWORK_connection_receive (cls, max, timeout, receiver, receiver_cls);
-}
-
-
-/**
- * Wrapper to cancel receiving from a socket.
- * 
- * @param cls handle to the GNUNET_NETWORK_ConnectionHandle to cancel
- * @param tc task ID that was returned by GNUNET_NETWORK_connection_receive
- */
-static void
-sock_receive_cancel (void *cls, GNUNET_SCHEDULER_TaskIdentifier ti)
-{
-  GNUNET_NETWORK_connection_receive_cancel (cls, ti);
-}
-
-
-/**
- * FIXME: document.
- */
-static void *
-sock_notify_transmit_ready (void *cls,
-                            size_t size,
-                            struct GNUNET_TIME_Relative timeout,
-                            GNUNET_NETWORK_TransmitReadyNotify notify,
-                            void *notify_cls)
-{
-  return GNUNET_NETWORK_connection_notify_transmit_ready (cls, size, timeout, notify,
-                                               notify_cls);
-}
-
-
-/**
- * FIXME: document.
+ * @param cls closure (struct GNUNET_SERVER_Handle)
+ * @param client identification of the client (struct GNUNET_SERVER_Client*)
+ * @param message the actual message
  */
 static void
-sock_notify_transmit_ready_cancel (void *cls, void *h)
+client_message_tokenizer_callback (void *cls,
+                                  void *client,
+                                  const struct GNUNET_MessageHeader *message)
 {
-  GNUNET_NETWORK_connection_notify_transmit_ready_cancel (h);
-}
-
-
-/**
- * Check if socket is still valid (no fatal errors have happened so far).
- *
- * @param cls the socket
- * @return GNUNET_YES if valid, GNUNET_NO otherwise
- */
-static int
-sock_check (void *cls)
-{
-  return GNUNET_NETWORK_connection_check (cls);
-}
-
+  struct GNUNET_SERVER_Handle *server = cls;
+  struct GNUNET_SERVER_Client *sender = client;
+  int ret;
 
-/**
- * Destroy this socket (free resources).
- *
- * @param cls the socket
- */
-static void
-sock_destroy (void *cls)
-{
-  GNUNET_NETWORK_connection_destroy (cls);
+#if DEBUG_SERVER
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Tokenizer gives server message of type %u from client\n",
+             ntohs (message->type));
+#endif
+  sender->in_process_client_buffer = GNUNET_YES;
+  ret = GNUNET_SERVER_inject (server, sender, message);
+  sender->in_process_client_buffer = GNUNET_NO;
+  if (GNUNET_OK != ret)
+    GNUNET_SERVER_client_disconnect (sender); 
 }
 
 
@@ -950,70 +852,23 @@ struct GNUNET_SERVER_Client *
 GNUNET_SERVER_connect_socket (struct
                               GNUNET_SERVER_Handle
                               *server,
-                              struct GNUNET_NETWORK_ConnectionHandle *connection)
+                              struct GNUNET_CONNECTION_Handle *connection)
 {
   struct GNUNET_SERVER_Client *client;
 
   client = GNUNET_malloc (sizeof (struct GNUNET_SERVER_Client));
-  client->client_closure = connection;
-  client->receive = &sock_receive;
-  client->receive_cancel = &sock_receive_cancel;
-  client->notify_transmit_ready = &sock_notify_transmit_ready;
-  client->notify_transmit_ready_cancel = &sock_notify_transmit_ready_cancel;
-  client->check = &sock_check;
-  client->destroy = &sock_destroy;
+  client->connection = connection;
+  client->mst = GNUNET_SERVER_mst_create (&client_message_tokenizer_callback,
+                                         server);
   client->reference_count = 1;
-  add_client (server, client);
-  return client;
-}
-
-
-/**
- * Add an arbitrary connection to the set of handles managed by this
- * server.  This can be used if a sending and receiving does not
- * really go over the network (internal transmission) or for servers
- * using UDP.
- *
- * @param server the server to use
- * @param chandle opaque handle for the connection
- * @param creceive receive function for the connection
- * @param ccancel cancel receive function for the connection
- * @param cnotify transmit notification function for the connection
- * @param cnotify_cancel transmit notification cancellation function for the connection
- * @param ccheck function to test if the connection is still up
- * @param cdestroy function to close and free the connection
- * @return the client handle (client should call
- *         "client_drop" on the return value eventually)
- */
-struct GNUNET_SERVER_Client *
-GNUNET_SERVER_connect_callback (struct
-                                GNUNET_SERVER_Handle
-                                *server,
-                                void *chandle,
-                                GNUNET_SERVER_ReceiveCallback
-                                creceive,
-                                GNUNET_SERVER_ReceiveCancelCallback
-                                ccancel,
-                                GNUNET_SERVER_TransmitReadyCallback
-                                cnotify,
-                                GNUNET_SERVER_TransmitReadyCancelCallback
-                                cnotify_cancel,
-                                GNUNET_SERVER_CheckCallback
-                                ccheck,
-                                GNUNET_SERVER_DestroyCallback cdestroy)
-{
-  struct GNUNET_SERVER_Client *client;
-
-  client = GNUNET_malloc (sizeof (struct GNUNET_SERVER_Client));
-  client->client_closure = chandle;
-  client->receive = creceive;
-  client->receive_cancel = ccancel;
-  client->notify_transmit_ready = cnotify;
-  client->notify_transmit_ready_cancel = cnotify_cancel;
-  client->check = ccheck;
-  client->destroy = cdestroy;
-  client->reference_count = 1;
-  add_client (server, client);
+  client->server = server;
+  client->last_activity = GNUNET_TIME_absolute_get ();
+  client->next = server->clients;
+  server->clients = client;
+  client->receive_pending = GNUNET_YES;
+  GNUNET_CONNECTION_receive (client->connection,
+                            GNUNET_SERVER_MAX_MESSAGE_SIZE - 1,
+                            server->idle_timeout, &process_incoming, client);
   return client;
 }
 
@@ -1045,8 +900,9 @@ GNUNET_SERVER_client_drop (struct GNUNET_SERVER_Client *client)
 {
   GNUNET_assert (client->reference_count > 0);
   client->reference_count--;
-  if ((client->server == NULL) && (client->reference_count == 0))
-    shutdown_incoming_processing (client);
+  if ( (client->shutdown_now == GNUNET_YES) && 
+       (client->reference_count == 0) )
+    GNUNET_SERVER_client_disconnect (client);
 }
 
 
@@ -1062,10 +918,8 @@ int
 GNUNET_SERVER_client_get_address (struct GNUNET_SERVER_Client *client,
                                   void **addr, size_t * addrlen)
 {
-  if (client->receive != &sock_receive)
-    return GNUNET_SYSERR;       /* not a network client */
-  return GNUNET_NETWORK_connection_get_address (client->client_closure,
-                                            addr, addrlen);
+  return GNUNET_CONNECTION_get_address (client->connection,
+                                        addr, addrlen);
 }
 
 
@@ -1094,6 +948,44 @@ GNUNET_SERVER_disconnect_notify (struct GNUNET_SERVER_Handle *server,
 }
 
 
+/**
+ * Ask the server to stop notifying us whenever a client disconnects.
+ *
+ * @param server the server manageing the clients
+ * @param callback function to call on disconnect
+ * @param callback_cls closure for callback
+ */
+void
+GNUNET_SERVER_disconnect_notify_cancel (struct GNUNET_SERVER_Handle *server,
+                                       GNUNET_SERVER_DisconnectCallback callback,
+                                       void *callback_cls)
+{
+  struct NotifyList *pos;
+  struct NotifyList *prev;
+
+  prev = NULL;
+  pos = server->disconnect_notify_list;
+  while (pos != NULL)
+    {
+      if ( (pos->callback == callback) &&
+          (pos->callback_cls == callback_cls ) )
+       break;
+      prev = pos;
+      pos = pos->next;
+    }
+  if (pos == NULL)
+    {
+      GNUNET_break (0);
+      return;
+    }
+  if (prev == NULL)
+    server->disconnect_notify_list = pos->next;
+  else
+    prev->next = pos->next;
+  GNUNET_free (pos);
+}
+
+
 /**
  * Ask the server to disconnect from the given client.
  * This is the same as returning GNUNET_SYSERR from a message
@@ -1105,12 +997,79 @@ GNUNET_SERVER_disconnect_notify (struct GNUNET_SERVER_Handle *server,
 void
 GNUNET_SERVER_client_disconnect (struct GNUNET_SERVER_Client *client)
 {
-  if (client->server == NULL)
-    return;                     /* already disconnected */
-  GNUNET_assert (client->my_receive != GNUNET_SCHEDULER_NO_TASK);
-  client->receive_cancel (client->client_closure, client->my_receive);
-  client->my_receive = GNUNET_SCHEDULER_NO_TASK;
-  shutdown_incoming_processing (client);
+  struct GNUNET_SERVER_Client *prev;
+  struct GNUNET_SERVER_Client *pos;
+  struct GNUNET_SERVER_Handle *server;
+  struct NotifyList *n;
+  unsigned int rc;
+
+#if DEBUG_SERVER
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "Client is being disconnected from the server.\n");
+#endif
+  if (client->restart_task != GNUNET_SCHEDULER_NO_TASK)
+    {
+      GNUNET_SCHEDULER_cancel (client->restart_task);
+      client->restart_task = GNUNET_SCHEDULER_NO_TASK;
+    }
+  if (GNUNET_YES == client->receive_pending)
+    {
+      GNUNET_CONNECTION_receive_cancel (client->connection);
+      client->receive_pending = GNUNET_NO;
+    }
+
+  rc = client->reference_count;  
+  if (client->server != NULL)
+    {
+      server = client->server;
+      client->server = NULL;
+      client->shutdown_now = GNUNET_YES;
+      prev = NULL;
+      pos = server->clients;
+      while ((pos != NULL) && (pos != client))
+        {
+          prev = pos;
+          pos = pos->next;
+        }
+      GNUNET_assert (pos != NULL);
+      if (prev == NULL)
+        server->clients = pos->next;
+      else
+        prev->next = pos->next;
+      if (client->restart_task != GNUNET_SCHEDULER_NO_TASK)
+       {
+         GNUNET_SCHEDULER_cancel (client->restart_task);
+         client->restart_task = GNUNET_SCHEDULER_NO_TASK;
+       }
+      n = server->disconnect_notify_list;
+      while (n != NULL)
+        {
+          n->callback (n->callback_cls, client);
+          n = n->next;
+        }
+    }
+  if (rc > 0)
+    {
+#if DEBUG_SERVER
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                 "RC still positive, not destroying everything.\n");
+#endif
+      return;
+    }
+  if (client->in_process_client_buffer == GNUNET_YES)
+    {
+#if DEBUG_SERVER
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                 "Still processing inputs, not destroying everything.\n");
+#endif
+      return;
+    }
+
+  if (client->persist == GNUNET_YES)
+    GNUNET_CONNECTION_persist_ (client->connection);  
+  GNUNET_CONNECTION_destroy (client->connection, GNUNET_NO);
+  GNUNET_SERVER_mst_destroy (client->mst);
+  GNUNET_free (client);  
 }
 
 
@@ -1118,7 +1077,6 @@ GNUNET_SERVER_client_disconnect (struct GNUNET_SERVER_Client *client)
  * Notify us when the server has enough space to transmit
  * a message of the given size to the given client.
  *
- * @param server the server to use
  * @param client client to transmit message to
  * @param size requested amount of buffer space
  * @param timeout after how long should we give up (and call
@@ -1127,21 +1085,32 @@ GNUNET_SERVER_client_disconnect (struct GNUNET_SERVER_Client *client)
  * @param callback_cls closure for callback
  * @return non-NULL if the notify callback was queued; can be used
  *           to cancel the request using
- *           GNUNET_NETWORK_connection_notify_transmit_ready_cancel.
+ *           GNUNET_CONNECTION_notify_transmit_ready_cancel.
  *         NULL if we are already going to notify someone else (busy)
  */
-struct GNUNET_NETWORK_TransmitHandle *
+struct GNUNET_CONNECTION_TransmitHandle *
 GNUNET_SERVER_notify_transmit_ready (struct GNUNET_SERVER_Client *client,
                                      size_t size,
                                      struct GNUNET_TIME_Relative timeout,
-                                     GNUNET_NETWORK_TransmitReadyNotify
+                                     GNUNET_CONNECTION_TransmitReadyNotify
                                      callback, void *callback_cls)
 {
-  return client->notify_transmit_ready (client->client_closure,
-                                        size,
-                                        timeout, callback, callback_cls);
+  return GNUNET_CONNECTION_notify_transmit_ready (client->connection,
+                                                 size,
+                                                 timeout, callback, callback_cls);
 }
 
+/**
+ * Set the persistent flag on this client, used to setup client connection
+ * to only be killed when the service it's connected to is actually dead.
+ *
+ * @param client the client to set the persistent flag on
+ */
+void
+GNUNET_SERVER_client_persist_ (struct GNUNET_SERVER_Client *client)
+{
+  client->persist = GNUNET_YES;
+}
 
 /**
  * Resume receiving from this client, we are done processing the
@@ -1151,50 +1120,72 @@ GNUNET_SERVER_notify_transmit_ready (struct GNUNET_SERVER_Client *client,
  * @param client client we were processing a message of
  * @param success GNUNET_OK to keep the connection open and
  *                          continue to receive
+ *                GNUNET_NO to close the connection (normal behavior)
  *                GNUNET_SYSERR to close the connection (signal
  *                          serious error)
  */
 void
 GNUNET_SERVER_receive_done (struct GNUNET_SERVER_Client *client, int success)
 {
-  char *sb;
-
   if (client == NULL)
     return;
   GNUNET_assert (client->suspended > 0);
   client->suspended--;
   if (success != GNUNET_OK)
-    client->shutdown_now = GNUNET_YES;
+    {
+#if DEBUG_SERVER
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                 "GNUNET_SERVER_receive_done called with failure indication\n");
+#endif
+      GNUNET_SERVER_client_disconnect (client);
+      return;
+    }
   if (client->suspended > 0)
-    return;
+    {
+#if DEBUG_SERVER
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                 "GNUNET_SERVER_receive_done called, but more clients pending\n");
+#endif
+      return;
+    }
   if (client->in_process_client_buffer == GNUNET_YES)
-    return;
-  if (client->side_buf_size > 0)
     {
-      /* resume processing from side-buf */
-      sb = client->side_buf;
-      client->side_buf = NULL;
-      /* this will also resume the receive job */
-      if (GNUNET_YES != client->shutdown_now)
-        process_incoming (client, sb, client->side_buf_size, NULL, 0, 0);
-      else
-        shutdown_incoming_processing (client);
-      /* finally, free the side-buf */
-      GNUNET_free (sb);
+#if DEBUG_SERVER
+      GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+                 "GNUNET_SERVER_receive_done called while still in processing loop\n");
+#endif
       return;
     }
-  /* resume receive job */
-  if (GNUNET_YES != client->shutdown_now)
+  if (client->server == NULL)
     {
-      GNUNET_SCHEDULER_add_continuation (client->server->sched,
-                                         GNUNET_NO,
-                                         &restart_processing,
-                                         client,
-                                         GNUNET_SCHEDULER_REASON_PREREQ_DONE);
+      GNUNET_SERVER_client_disconnect (client);
       return;
     }
-  shutdown_incoming_processing (client);
+#if DEBUG_SERVER
+  GNUNET_log (GNUNET_ERROR_TYPE_DEBUG,
+             "GNUNET_SERVER_receive_done causes restart in reading from the socket\n");
+#endif
+  GNUNET_assert (GNUNET_SCHEDULER_NO_TASK == client->restart_task);
+  client->restart_task = GNUNET_SCHEDULER_add_now (&restart_processing,
+                                                  client);
 }
 
 
+/**
+ * Configure this server's connections to continue handling client
+ * requests as usual even after we get a shutdown signal.  The change
+ * only applies to clients that connect to the server from the outside
+ * using TCP after this call.  Clients managed previously or those
+ * added using GNUNET_SERVER_connect_socket and
+ * GNUNET_SERVER_connect_callback are not affected by this option.
+ *
+ * @param h server handle
+ * @param do_ignore GNUNET_YES to ignore, GNUNET_NO to restore default
+ */
+void
+GNUNET_SERVER_ignore_shutdown (struct GNUNET_SERVER_Handle *h, int do_ignore)
+{
+  h->clients_ignore_shutdown = do_ignore;
+}
+
 /* end of server.c */