gst/tcp: Factor out common symbols; fix tests.
authorThomas Vander Stichele <thomas (at) apestaart (dot) org>
Thu, 26 Jan 2012 09:41:22 +0000 (10:41 +0100)
committerThomas Vander Stichele <thomas (at) apestaart (dot) org>
Sun, 12 Feb 2012 21:07:29 +0000 (22:07 +0100)
gst/tcp/gstmultifdsink.c
gst/tcp/gstmultifdsink.h
gst/tcp/gstmultihandlesink.c [new file with mode: 0644]
gst/tcp/gstmultihandlesink.h [new file with mode: 0644]
gst/tcp/gstmultisocketsink.h
gst/tcp/gsttcp-marshal.list
gst/tcp/gsttcpplugin.c
tests/check/elements/multifdsink.c

index 061fd68..c53df68 100644 (file)
 #endif
 
 #include "gstmultifdsink.h"
-#include "gstmultifdsink-marshal.h"
+#include "gsttcp-marshal.h"
 
 #define NOT_IMPLEMENTED 0
 
@@ -533,7 +533,7 @@ gst_multi_fd_sink_class_init (GstMultiFdSinkClass * klass)
       g_signal_new ("add-full", G_TYPE_FROM_CLASS (klass),
       G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
           add_full), NULL, NULL,
-      gst_multi_fd_sink_marshal_VOID__INT_ENUM_INT_UINT64_INT_UINT64,
+      gst_tcp_marshal_VOID__INT_ENUM_INT_UINT64_INT_UINT64,
       G_TYPE_NONE, 6, G_TYPE_INT, GST_TYPE_SYNC_METHOD, GST_TYPE_UNIT_TYPE,
       G_TYPE_UINT64, GST_TYPE_UNIT_TYPE, G_TYPE_UINT64);
   /**
@@ -546,7 +546,7 @@ gst_multi_fd_sink_class_init (GstMultiFdSinkClass * klass)
   gst_multi_fd_sink_signals[SIGNAL_REMOVE] =
       g_signal_new ("remove", G_TYPE_FROM_CLASS (klass),
       G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
-          remove), NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT, G_TYPE_NONE,
+          remove), NULL, NULL, gst_tcp_marshal_VOID__INT, G_TYPE_NONE,
       1, G_TYPE_INT);
   /**
    * GstMultiFdSink::remove-flush:
@@ -559,7 +559,7 @@ gst_multi_fd_sink_class_init (GstMultiFdSinkClass * klass)
   gst_multi_fd_sink_signals[SIGNAL_REMOVE_FLUSH] =
       g_signal_new ("remove-flush", G_TYPE_FROM_CLASS (klass),
       G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
-          remove_flush), NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT,
+          remove_flush), NULL, NULL, gst_tcp_marshal_VOID__INT,
       G_TYPE_NONE, 1, G_TYPE_INT);
   /**
    * GstMultiFdSink::clear:
@@ -593,7 +593,7 @@ gst_multi_fd_sink_class_init (GstMultiFdSinkClass * klass)
   gst_multi_fd_sink_signals[SIGNAL_GET_STATS] =
       g_signal_new ("get-stats", G_TYPE_FROM_CLASS (klass),
       G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION, G_STRUCT_OFFSET (GstMultiFdSinkClass,
-          get_stats), NULL, NULL, gst_multi_fd_sink_marshal_BOXED__INT,
+          get_stats), NULL, NULL, gst_tcp_marshal_BOXED__INT,
       G_TYPE_VALUE_ARRAY, 1, G_TYPE_INT);
 
   /**
@@ -608,8 +608,7 @@ gst_multi_fd_sink_class_init (GstMultiFdSinkClass * klass)
   gst_multi_fd_sink_signals[SIGNAL_CLIENT_ADDED] =
       g_signal_new ("client-added", G_TYPE_FROM_CLASS (klass),
       G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiFdSinkClass, client_added),
-      NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT, G_TYPE_NONE, 1,
-      G_TYPE_INT);
+      NULL, NULL, gst_tcp_marshal_VOID__INT, G_TYPE_NONE, 1, G_TYPE_INT);
   /**
    * GstMultiFdSink::client-removed:
    * @gstmultifdsink: the multifdsink element that emitted this signal
@@ -627,7 +626,7 @@ gst_multi_fd_sink_class_init (GstMultiFdSinkClass * klass)
   gst_multi_fd_sink_signals[SIGNAL_CLIENT_REMOVED] =
       g_signal_new ("client-removed", G_TYPE_FROM_CLASS (klass),
       G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiFdSinkClass,
-          client_removed), NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT_ENUM,
+          client_removed), NULL, NULL, gst_tcp_marshal_VOID__INT_ENUM,
       G_TYPE_NONE, 2, G_TYPE_INT, GST_TYPE_CLIENT_STATUS);
   /**
    * GstMultiFdSink::client-fd-removed:
@@ -647,7 +646,7 @@ gst_multi_fd_sink_class_init (GstMultiFdSinkClass * klass)
   gst_multi_fd_sink_signals[SIGNAL_CLIENT_FD_REMOVED] =
       g_signal_new ("client-fd-removed", G_TYPE_FROM_CLASS (klass),
       G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiFdSinkClass,
-          client_fd_removed), NULL, NULL, gst_multi_fd_sink_marshal_VOID__INT,
+          client_fd_removed), NULL, NULL, gst_tcp_marshal_VOID__INT,
       G_TYPE_NONE, 1, G_TYPE_INT);
 
   gst_element_class_add_pad_template (gstelement_class,
@@ -1974,14 +1973,15 @@ gst_multi_fd_sink_handle_client_write (GstMultiFdSink * sink,
     if (client->sending) {
       ssize_t wrote;
       GstBuffer *head;
+      GstMapInfo info;
       guint8 *data;
-      gsize size;
 
       /* pick first buffer from list */
       head = GST_BUFFER (client->sending->data);
 
-      data = gst_buffer_map (head, &size, NULL, GST_MAP_READ);
-      maxsize = size - client->bufoffset;
+      g_assert (gst_buffer_map (head, &info, GST_MAP_READ));
+      data = info.data;
+      maxsize = info.size - client->bufoffset;
 
       /* try to write the complete buffer */
 #ifdef MSG_NOSIGNAL
@@ -1994,7 +1994,7 @@ gst_multi_fd_sink_handle_client_write (GstMultiFdSink * sink,
       } else {
         wrote = write (fd, data + client->bufoffset, maxsize);
       }
-      gst_buffer_unmap (head, data, size);
+      gst_buffer_unmap (head, &info);
 
       if (wrote < 0) {
         /* hmm error.. */
@@ -2523,6 +2523,7 @@ gst_multi_fd_sink_render (GstBaseSink * bsink, GstBuffer * buf)
     gst_buffer_ref (buf);
   }
 #endif
+  gst_buffer_ref (buf);
 
   GST_LOG_OBJECT (sink, "received buffer %p, in_caps: %s, offset %"
       G_GINT64_FORMAT ", offset_end %" G_GINT64_FORMAT
@@ -2893,19 +2894,3 @@ start_failed:
     return GST_STATE_CHANGE_FAILURE;
   }
 }
-
-static gboolean
-plugin_init (GstPlugin * plugin)
-{
-  if (!gst_element_register (plugin, "multifdsink", GST_RANK_NONE,
-          GST_TYPE_MULTI_FD_SINK))
-    return FALSE;
-
-  return TRUE;
-}
-
-GST_PLUGIN_DEFINE (GST_VERSION_MAJOR,
-    GST_VERSION_MINOR,
-    "multifdsink",
-    "transfer data to multiple fds",
-    plugin_init, VERSION, GST_LICENSE, GST_PACKAGE_NAME, GST_PACKAGE_ORIGIN)
index 5ac0df7..8cc541b 100644 (file)
@@ -25,6 +25,8 @@
 #include <gst/gst.h>
 #include <gst/base/gstbasesink.h>
 
+#include "gstmultihandlesink.h"
+
 G_BEGIN_DECLS
 
 #define GST_TYPE_MULTI_FD_SINK \
@@ -50,48 +52,6 @@ typedef enum {
   GST_MULTI_FD_SINK_FLAG_LAST        = (GST_ELEMENT_FLAG_LAST << 2)
 } GstMultiFdSinkFlags;
 
-/**
- * GstRecoverPolicy:
- * @GST_RECOVER_POLICY_NONE             : no recovering is done
- * @GST_RECOVER_POLICY_RESYNC_LATEST    : client is moved to last buffer
- * @GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT: client is moved to the soft limit
- * @GST_RECOVER_POLICY_RESYNC_KEYFRAME  : client is moved to latest keyframe
- *
- * Possible values for the recovery procedure to use when a client consumes
- * data too slow and has a backlag of more that soft-limit buffers.
- */
-typedef enum
-{
-  GST_RECOVER_POLICY_NONE,
-  GST_RECOVER_POLICY_RESYNC_LATEST,
-  GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT,
-  GST_RECOVER_POLICY_RESYNC_KEYFRAME
-} GstRecoverPolicy;
-
-/**
- * GstSyncMethod:
- * @GST_SYNC_METHOD_LATEST              : client receives most recent buffer
- * @GST_SYNC_METHOD_NEXT_KEYFRAME       : client receives next keyframe
- * @GST_SYNC_METHOD_LATEST_KEYFRAME     : client receives latest keyframe (burst)
- * @GST_SYNC_METHOD_BURST               : client receives specific amount of data
- * @GST_SYNC_METHOD_BURST_KEYFRAME      : client receives specific amount of data 
- *                                        starting from latest keyframe
- * @GST_SYNC_METHOD_BURST_WITH_KEYFRAME : client receives specific amount of data from
- *                                        a keyframe, or if there is not enough data after
- *                                        the keyframe, starting before the keyframe
- *
- * This enum defines the selection of the first buffer that is sent
- * to a new client.
- */
-typedef enum
-{
-  GST_SYNC_METHOD_LATEST,
-  GST_SYNC_METHOD_NEXT_KEYFRAME,
-  GST_SYNC_METHOD_LATEST_KEYFRAME,
-  GST_SYNC_METHOD_BURST,
-  GST_SYNC_METHOD_BURST_KEYFRAME,
-  GST_SYNC_METHOD_BURST_WITH_KEYFRAME
-} GstSyncMethod;
 
 /**
  * GstTCPUnitType:
@@ -110,30 +70,6 @@ typedef enum
   GST_TCP_UNIT_TYPE_BYTES
 } GstTCPUnitType;
 
-/**
- * GstClientStatus:
- * @GST_CLIENT_STATUS_OK       : client is ok
- * @GST_CLIENT_STATUS_CLOSED   : client closed the socket
- * @GST_CLIENT_STATUS_REMOVED  : client is removed
- * @GST_CLIENT_STATUS_SLOW     : client is too slow
- * @GST_CLIENT_STATUS_ERROR    : client is in error
- * @GST_CLIENT_STATUS_DUPLICATE: same client added twice
- * @GST_CLIENT_STATUS_FLUSHING : client is flushing out the remaining buffers.
- *
- * This specifies the reason why a client was removed from
- * multifdsink and is received in the "client-removed" signal.
- */
-typedef enum
-{
-  GST_CLIENT_STATUS_OK          = 0,
-  GST_CLIENT_STATUS_CLOSED      = 1,
-  GST_CLIENT_STATUS_REMOVED     = 2,
-  GST_CLIENT_STATUS_SLOW        = 3,
-  GST_CLIENT_STATUS_ERROR       = 4,
-  GST_CLIENT_STATUS_DUPLICATE   = 5,
-  GST_CLIENT_STATUS_FLUSHING    = 6
-} GstClientStatus;
-
 /* structure for a client
  */
 typedef struct {
@@ -176,11 +112,6 @@ typedef struct {
   guint64 last_buffer_ts;
 } GstTCPClient;
 
-#define CLIENTS_LOCK_INIT(fdsink)       (g_rec_mutex_init(&fdsink->clientslock))
-#define CLIENTS_LOCK_CLEAR(fdsink)      (g_rec_mutex_clear(&fdsink->clientslock))
-#define CLIENTS_LOCK(fdsink)            (g_rec_mutex_lock(&fdsink->clientslock))
-#define CLIENTS_UNLOCK(fdsink)          (g_rec_mutex_unlock(&fdsink->clientslock))
-
 /**
  * GstMultiFdSink:
  *
diff --git a/gst/tcp/gstmultihandlesink.c b/gst/tcp/gstmultihandlesink.c
new file mode 100644 (file)
index 0000000..d858746
--- /dev/null
@@ -0,0 +1,2788 @@
+/* GStreamer
+ * Copyright (C) <1999> Erik Walthinsen <omega@cse.ogi.edu>
+ * Copyright (C) <2004> Thomas Vander Stichele <thomas at apestaart dot org>
+ * Copyright (C) 2006 Wim Taymans <wim at fluendo dot com>
+ * Copyright (C) <2011> Collabora Ltd.
+ *     Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ * Boston, MA 02111-1307, USA.
+ */
+
+/**
+ * SECTION:element-multisocketsink
+ * @see_also: tcpserversink
+ *
+ * This plugin writes incoming data to a set of file descriptors. The
+ * file descriptors can be added to multisocketsink by emitting the #GstMultiHandleSink::add signal. 
+ * For each descriptor added, the #GstMultiHandleSink::client-added signal will be called.
+ *
+ * As of version 0.10.8, a client can also be added with the #GstMultiHandleSink::add-full signal
+ * that allows for more control over what and how much data a client 
+ * initially receives.
+ *
+ * Clients can be removed from multisocketsink by emitting the #GstMultiHandleSink::remove signal. For
+ * each descriptor removed, the #GstMultiHandleSink::client-removed signal will be called. The
+ * #GstMultiHandleSink::client-removed signal can also be fired when multisocketsink decides that a
+ * client is not active anymore or, depending on the value of the
+ * #GstMultiHandleSink:recover-policy property, if the client is reading too slowly.
+ * In all cases, multisocketsink will never close a file descriptor itself.
+ * The user of multisocketsink is responsible for closing all file descriptors.
+ * This can for example be done in response to the #GstMultiHandleSink::client-fd-removed signal.
+ * Note that multisocketsink still has a reference to the file descriptor when the
+ * #GstMultiHandleSink::client-removed signal is emitted, so that "get-stats" can be performed on
+ * the descriptor; it is therefore not safe to close the file descriptor in
+ * the #GstMultiHandleSink::client-removed signal handler, and you should use the 
+ * #GstMultiHandleSink::client-fd-removed signal to safely close the fd.
+ *
+ * Multisocketsink internally keeps a queue of the incoming buffers and uses a
+ * separate thread to send the buffers to the clients. This ensures that no
+ * client write can block the pipeline and that clients can read with different
+ * speeds.
+ *
+ * When adding a client to multisocketsink, the #GstMultiHandleSink:sync-method property will define
+ * which buffer in the queued buffers will be sent first to the client. Clients 
+ * can be sent the most recent buffer (which might not be decodable by the 
+ * client if it is not a keyframe), the next keyframe received in 
+ * multisocketsink (which can take some time depending on the keyframe rate), or the
+ * last received keyframe (which will cause a simple burst-on-connect). 
+ * Multisocketsink will always keep at least one keyframe in its internal buffers
+ * when the sync-mode is set to latest-keyframe.
+ *
+ * As of version 0.10.8, there are additional values for the #GstMultiHandleSink:sync-method 
+ * property to allow finer control over burst-on-connect behaviour. By selecting
+ * the 'burst' method a minimum burst size can be chosen, 'burst-keyframe'
+ * additionally requires that the burst begin with a keyframe, and 
+ * 'burst-with-keyframe' attempts to burst beginning with a keyframe, but will
+ * prefer a minimum burst size even if it requires not starting with a keyframe.
+ *
+ * Multisocketsink can be instructed to keep at least a minimum amount of data
+ * expressed in time or byte units in its internal queues with the 
+ * #GstMultiHandleSink:time-min and #GstMultiHandleSink:bytes-min properties respectively.
+ * These properties are useful if the application adds clients with the 
+ * #GstMultiHandleSink::add-full signal to make sure that a burst connect can
+ * actually be honored. 
+ *
+ * When streaming data, clients are allowed to read at a different rate than
+ * the rate at which multisocketsink receives data. If the client is reading too
+ * fast, no data will be send to the client until multisocketsink receives more
+ * data. If the client, however, reads too slowly, data for that client will be 
+ * queued up in multisocketsink. Two properties control the amount of data 
+ * (buffers) that is queued in multisocketsink: #GstMultiHandleSink:buffers-max and 
+ * #GstMultiHandleSink:buffers-soft-max. A client that falls behind by
+ * #GstMultiHandleSink:buffers-max is removed from multisocketsink forcibly.
+ *
+ * A client with a lag of at least #GstMultiHandleSink:buffers-soft-max enters the recovery
+ * procedure which is controlled with the #GstMultiHandleSink:recover-policy property.
+ * A recover policy of NONE will do nothing, RESYNC_LATEST will send the most recently
+ * received buffer as the next buffer for the client, RESYNC_SOFT_LIMIT
+ * positions the client to the soft limit in the buffer queue and
+ * RESYNC_KEYFRAME positions the client at the most recent keyframe in the
+ * buffer queue.
+ *
+ * multisocketsink will by default synchronize on the clock before serving the 
+ * buffers to the clients. This behaviour can be disabled by setting the sync 
+ * property to FALSE. Multisocketsink will by default not do QoS and will never
+ * drop late buffers.
+ *
+ * Last reviewed on 2006-09-12 (0.10.10)
+ */
+
+#ifdef HAVE_CONFIG_H
+#include "config.h"
+#endif
+
+#include <gst/gst-i18n-plugin.h>
+
+#include "gstmultisocketsink.h"
+#include "gsttcp-marshal.h"
+
+#ifndef G_OS_WIN32
+#include <netinet/in.h>
+#endif
+
+#define NOT_IMPLEMENTED 0
+
+static GstStaticPadTemplate sinktemplate = GST_STATIC_PAD_TEMPLATE ("sink",
+    GST_PAD_SINK,
+    GST_PAD_ALWAYS,
+    GST_STATIC_CAPS_ANY);
+
+GST_DEBUG_CATEGORY_STATIC (multisocketsink_debug);
+#define GST_CAT_DEFAULT (multisocketsink_debug)
+
+/* MultiHandleSink signals and args */
+enum
+{
+  /* methods */
+  SIGNAL_ADD,
+  SIGNAL_ADD_BURST,
+  SIGNAL_REMOVE,
+  SIGNAL_REMOVE_FLUSH,
+  SIGNAL_CLEAR,
+  SIGNAL_GET_STATS,
+
+  /* signals */
+  SIGNAL_CLIENT_ADDED,
+  SIGNAL_CLIENT_REMOVED,
+  SIGNAL_CLIENT_SOCKET_REMOVED,
+
+  LAST_SIGNAL
+};
+
+
+/* this is really arbitrarily chosen */
+#define DEFAULT_MODE                    1
+#define DEFAULT_BUFFERS_MAX             -1
+#define DEFAULT_BUFFERS_SOFT_MAX        -1
+#define DEFAULT_TIME_MIN                -1
+#define DEFAULT_BYTES_MIN               -1
+#define DEFAULT_BUFFERS_MIN             -1
+#define DEFAULT_UNIT_TYPE               GST_FORMAT_BUFFERS
+#define DEFAULT_UNITS_MAX               -1
+#define DEFAULT_UNITS_SOFT_MAX          -1
+#define DEFAULT_RECOVER_POLICY          GST_RECOVER_POLICY_NONE
+#define DEFAULT_TIMEOUT                 0
+#define DEFAULT_SYNC_METHOD             GST_SYNC_METHOD_LATEST
+
+#define DEFAULT_BURST_FORMAT            GST_FORMAT_UNDEFINED
+#define DEFAULT_BURST_VALUE             0
+
+#define DEFAULT_QOS_DSCP                -1
+#define DEFAULT_HANDLE_READ             TRUE
+
+#define DEFAULT_RESEND_STREAMHEADER      TRUE
+
+enum
+{
+  PROP_0,
+  PROP_MODE,
+  PROP_BUFFERS_QUEUED,
+  PROP_BYTES_QUEUED,
+  PROP_TIME_QUEUED,
+
+  PROP_UNIT_TYPE,
+  PROP_UNITS_MAX,
+  PROP_UNITS_SOFT_MAX,
+
+  PROP_BUFFERS_MAX,
+  PROP_BUFFERS_SOFT_MAX,
+
+  PROP_TIME_MIN,
+  PROP_BYTES_MIN,
+  PROP_BUFFERS_MIN,
+
+  PROP_RECOVER_POLICY,
+  PROP_TIMEOUT,
+  PROP_SYNC_METHOD,
+  PROP_BYTES_TO_SERVE,
+  PROP_BYTES_SERVED,
+
+  PROP_BURST_FORMAT,
+  PROP_BURST_VALUE,
+
+  PROP_QOS_DSCP,
+
+  PROP_HANDLE_READ,
+
+  PROP_RESEND_STREAMHEADER,
+
+  PROP_NUM_SOCKETS,
+
+  PROP_LAST
+};
+
+#define GST_TYPE_RECOVER_POLICY (gst_multi_handle_sink_recover_policy_get_type())
+static GType
+gst_multi_handle_sink_recover_policy_get_type (void)
+{
+  static GType recover_policy_type = 0;
+  static const GEnumValue recover_policy[] = {
+    {GST_RECOVER_POLICY_NONE,
+        "Do not try to recover", "none"},
+    {GST_RECOVER_POLICY_RESYNC_LATEST,
+        "Resync client to latest buffer", "latest"},
+    {GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT,
+        "Resync client to soft limit", "soft-limit"},
+    {GST_RECOVER_POLICY_RESYNC_KEYFRAME,
+        "Resync client to most recent keyframe", "keyframe"},
+    {0, NULL, NULL},
+  };
+
+  if (!recover_policy_type) {
+    recover_policy_type =
+        g_enum_register_static ("GstMultiHandleSinkRecoverPolicy",
+        recover_policy);
+  }
+  return recover_policy_type;
+}
+
+#define GST_TYPE_SYNC_METHOD (gst_multi_handle_sink_sync_method_get_type())
+static GType
+gst_multi_handle_sink_sync_method_get_type (void)
+{
+  static GType sync_method_type = 0;
+  static const GEnumValue sync_method[] = {
+    {GST_SYNC_METHOD_LATEST,
+        "Serve starting from the latest buffer", "latest"},
+    {GST_SYNC_METHOD_NEXT_KEYFRAME,
+        "Serve starting from the next keyframe", "next-keyframe"},
+    {GST_SYNC_METHOD_LATEST_KEYFRAME,
+          "Serve everything since the latest keyframe (burst)",
+        "latest-keyframe"},
+    {GST_SYNC_METHOD_BURST, "Serve burst-value data to client", "burst"},
+    {GST_SYNC_METHOD_BURST_KEYFRAME,
+          "Serve burst-value data starting on a keyframe",
+        "burst-keyframe"},
+    {GST_SYNC_METHOD_BURST_WITH_KEYFRAME,
+          "Serve burst-value data preferably starting on a keyframe",
+        "burst-with-keyframe"},
+    {0, NULL, NULL},
+  };
+
+  if (!sync_method_type) {
+    sync_method_type =
+        g_enum_register_static ("GstMultiHandleSinkSyncMethod", sync_method);
+  }
+  return sync_method_type;
+}
+
+#define GST_TYPE_CLIENT_STATUS (gst_multi_handle_sink_client_status_get_type())
+static GType
+gst_multi_handle_sink_client_status_get_type (void)
+{
+  static GType client_status_type = 0;
+  static const GEnumValue client_status[] = {
+    {GST_CLIENT_STATUS_OK, "ok", "ok"},
+    {GST_CLIENT_STATUS_CLOSED, "Closed", "closed"},
+    {GST_CLIENT_STATUS_REMOVED, "Removed", "removed"},
+    {GST_CLIENT_STATUS_SLOW, "Too slow", "slow"},
+    {GST_CLIENT_STATUS_ERROR, "Error", "error"},
+    {GST_CLIENT_STATUS_DUPLICATE, "Duplicate", "duplicate"},
+    {GST_CLIENT_STATUS_FLUSHING, "Flushing", "flushing"},
+    {0, NULL, NULL},
+  };
+
+  if (!client_status_type) {
+    client_status_type =
+        g_enum_register_static ("GstMultiHandleSinkClientStatus",
+        client_status);
+  }
+  return client_status_type;
+}
+
+static void gst_multi_handle_sink_finalize (GObject * object);
+
+static void gst_multi_handle_sink_remove_client_link (GstMultiHandleSink * sink,
+    GList * link);
+static gboolean gst_multi_handle_sink_socket_condition (GSocket * socket,
+    GIOCondition condition, GstMultiHandleSink * sink);
+
+static GstFlowReturn gst_multi_handle_sink_render (GstBaseSink * bsink,
+    GstBuffer * buf);
+static gboolean gst_multi_handle_sink_unlock (GstBaseSink * bsink);
+static gboolean gst_multi_handle_sink_unlock_stop (GstBaseSink * bsink);
+static GstStateChangeReturn gst_multi_handle_sink_change_state (GstElement *
+    element, GstStateChange transition);
+
+static void gst_multi_handle_sink_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec);
+static void gst_multi_handle_sink_get_property (GObject * object, guint prop_id,
+    GValue * value, GParamSpec * pspec);
+
+#define gst_multi_handle_sink_parent_class parent_class
+G_DEFINE_TYPE (GstMultiHandleSink, gst_multi_handle_sink, GST_TYPE_BASE_SINK);
+
+static guint gst_multi_handle_sink_signals[LAST_SIGNAL] = { 0 };
+
+static void
+gst_multi_handle_sink_class_init (GstMultiHandleSinkClass * klass)
+{
+  GObjectClass *gobject_class;
+  GstElementClass *gstelement_class;
+  GstBaseSinkClass *gstbasesink_class;
+
+  gobject_class = (GObjectClass *) klass;
+  gstelement_class = (GstElementClass *) klass;
+  gstbasesink_class = (GstBaseSinkClass *) klass;
+
+  gobject_class->set_property = gst_multi_handle_sink_set_property;
+  gobject_class->get_property = gst_multi_handle_sink_get_property;
+  gobject_class->finalize = gst_multi_handle_sink_finalize;
+
+  g_object_class_install_property (gobject_class, PROP_BUFFERS_MAX,
+      g_param_spec_int ("buffers-max", "Buffers max",
+          "max number of buffers to queue for a client (-1 = no limit)", -1,
+          G_MAXINT, DEFAULT_BUFFERS_MAX,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BUFFERS_SOFT_MAX,
+      g_param_spec_int ("buffers-soft-max", "Buffers soft max",
+          "Recover client when going over this limit (-1 = no limit)", -1,
+          G_MAXINT, DEFAULT_BUFFERS_SOFT_MAX,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_BYTES_MIN,
+      g_param_spec_int ("bytes-min", "Bytes min",
+          "min number of bytes to queue (-1 = as little as possible)", -1,
+          G_MAXINT, DEFAULT_BYTES_MIN,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_TIME_MIN,
+      g_param_spec_int64 ("time-min", "Time min",
+          "min number of time to queue (-1 = as little as possible)", -1,
+          G_MAXINT64, DEFAULT_TIME_MIN,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BUFFERS_MIN,
+      g_param_spec_int ("buffers-min", "Buffers min",
+          "min number of buffers to queue (-1 = as few as possible)", -1,
+          G_MAXINT, DEFAULT_BUFFERS_MIN,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_UNIT_TYPE,
+      g_param_spec_enum ("unit-type", "Units type",
+          "The unit to measure the max/soft-max/queued properties",
+          GST_TYPE_FORMAT, DEFAULT_UNIT_TYPE,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_UNITS_MAX,
+      g_param_spec_int64 ("units-max", "Units max",
+          "max number of units to queue (-1 = no limit)", -1, G_MAXINT64,
+          DEFAULT_UNITS_MAX, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_UNITS_SOFT_MAX,
+      g_param_spec_int64 ("units-soft-max", "Units soft max",
+          "Recover client when going over this limit (-1 = no limit)", -1,
+          G_MAXINT64, DEFAULT_UNITS_SOFT_MAX,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_BUFFERS_QUEUED,
+      g_param_spec_uint ("buffers-queued", "Buffers queued",
+          "Number of buffers currently queued", 0, G_MAXUINT, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+#if NOT_IMPLEMENTED
+  g_object_class_install_property (gobject_class, PROP_BYTES_QUEUED,
+      g_param_spec_uint ("bytes-queued", "Bytes queued",
+          "Number of bytes currently queued", 0, G_MAXUINT, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_TIME_QUEUED,
+      g_param_spec_uint64 ("time-queued", "Time queued",
+          "Number of time currently queued", 0, G_MAXUINT64, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+#endif
+
+  g_object_class_install_property (gobject_class, PROP_RECOVER_POLICY,
+      g_param_spec_enum ("recover-policy", "Recover Policy",
+          "How to recover when client reaches the soft max",
+          GST_TYPE_RECOVER_POLICY, DEFAULT_RECOVER_POLICY,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_TIMEOUT,
+      g_param_spec_uint64 ("timeout", "Timeout",
+          "Maximum inactivity timeout in nanoseconds for a client (0 = no limit)",
+          0, G_MAXUINT64, DEFAULT_TIMEOUT,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_SYNC_METHOD,
+      g_param_spec_enum ("sync-method", "Sync Method",
+          "How to sync new clients to the stream", GST_TYPE_SYNC_METHOD,
+          DEFAULT_SYNC_METHOD, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BYTES_TO_SERVE,
+      g_param_spec_uint64 ("bytes-to-serve", "Bytes to serve",
+          "Number of bytes received to serve to clients", 0, G_MAXUINT64, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BYTES_SERVED,
+      g_param_spec_uint64 ("bytes-served", "Bytes served",
+          "Total number of bytes send to all clients", 0, G_MAXUINT64, 0,
+          G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_BURST_FORMAT,
+      g_param_spec_enum ("burst-format", "Burst format",
+          "The format of the burst units (when sync-method is burst[[-with]-keyframe])",
+          GST_TYPE_FORMAT, DEFAULT_BURST_FORMAT,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  g_object_class_install_property (gobject_class, PROP_BURST_VALUE,
+      g_param_spec_uint64 ("burst-value", "Burst value",
+          "The amount of burst expressed in burst-unit", 0, G_MAXUINT64,
+          DEFAULT_BURST_VALUE, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_QOS_DSCP,
+      g_param_spec_int ("qos-dscp", "QoS diff srv code point",
+          "Quality of Service, differentiated services code point (-1 default)",
+          -1, 63, DEFAULT_QOS_DSCP,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  /**
+   * GstMultiHandleSink::handle-read
+   *
+   * Handle read requests from clients and discard the data.
+   *
+   * Since: 0.10.23
+   */
+  g_object_class_install_property (gobject_class, PROP_HANDLE_READ,
+      g_param_spec_boolean ("handle-read", "Handle Read",
+          "Handle client reads and discard the data",
+          DEFAULT_HANDLE_READ, G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+  /**
+   * GstMultiHandleSink::resend-streamheader
+   *
+   * Resend the streamheaders to existing clients when they change.
+   *
+   * Since: 0.10.23
+   */
+  g_object_class_install_property (gobject_class, PROP_RESEND_STREAMHEADER,
+      g_param_spec_boolean ("resend-streamheader", "Resend streamheader",
+          "Resend the streamheader if it changes in the caps",
+          DEFAULT_RESEND_STREAMHEADER,
+          G_PARAM_READWRITE | G_PARAM_STATIC_STRINGS));
+
+  g_object_class_install_property (gobject_class, PROP_NUM_SOCKETS,
+      g_param_spec_uint ("num-sockets", "Number of sockets",
+          "The current number of client sockets",
+          0, G_MAXUINT, 0, G_PARAM_READABLE | G_PARAM_STATIC_STRINGS));
+
+  /**
+   * GstMultiHandleSink::add:
+   * @gstmultisocketsink: the multisocketsink element to emit this signal on
+   * @socket:             the socket to add to multisocketsink
+   *
+   * Hand the given open socket to multisocketsink to write to.
+   */
+  gst_multi_handle_sink_signals[SIGNAL_ADD] =
+      g_signal_new ("add", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+      G_STRUCT_OFFSET (GstMultiHandleSinkClass, add), NULL, NULL,
+      g_cclosure_marshal_VOID__OBJECT, G_TYPE_NONE, 1, G_TYPE_SOCKET);
+  /**
+   * GstMultiHandleSink::add-full:
+   * @gstmultisocketsink: the multisocketsink element to emit this signal on
+   * @socket:         the socket to add to multisocketsink
+   * @sync:           the sync method to use
+   * @format_min:     the format of @value_min
+   * @value_min:      the minimum amount of data to burst expressed in
+   *                  @format_min units.
+   * @format_max:     the format of @value_max
+   * @value_max:      the maximum amount of data to burst expressed in
+   *                  @format_max units.
+   *
+   * Hand the given open socket to multisocketsink to write to and
+   * specify the burst parameters for the new connection.
+   */
+  gst_multi_handle_sink_signals[SIGNAL_ADD_BURST] =
+      g_signal_new ("add-full", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+      G_STRUCT_OFFSET (GstMultiHandleSinkClass, add_full), NULL, NULL,
+      gst_tcp_marshal_VOID__OBJECT_ENUM_ENUM_UINT64_ENUM_UINT64, G_TYPE_NONE, 6,
+      G_TYPE_SOCKET, GST_TYPE_SYNC_METHOD, GST_TYPE_FORMAT, G_TYPE_UINT64,
+      GST_TYPE_FORMAT, G_TYPE_UINT64);
+  /**
+   * GstMultiHandleSink::remove:
+   * @gstmultisocketsink: the multisocketsink element to emit this signal on
+   * @socket:             the socket to remove from multisocketsink
+   *
+   * Remove the given open socket from multisocketsink.
+   */
+  gst_multi_handle_sink_signals[SIGNAL_REMOVE] =
+      g_signal_new ("remove", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+      G_STRUCT_OFFSET (GstMultiHandleSinkClass, remove), NULL, NULL,
+      g_cclosure_marshal_VOID__OBJECT, G_TYPE_NONE, 1, G_TYPE_SOCKET);
+  /**
+   * GstMultiHandleSink::remove-flush:
+   * @gstmultisocketsink: the multisocketsink element to emit this signal on
+   * @socket:             the socket to remove from multisocketsink
+   *
+   * Remove the given open socket from multisocketsink after flushing all
+   * the pending data to the socket.
+   */
+  gst_multi_handle_sink_signals[SIGNAL_REMOVE_FLUSH] =
+      g_signal_new ("remove-flush", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+      G_STRUCT_OFFSET (GstMultiHandleSinkClass, remove_flush), NULL, NULL,
+      g_cclosure_marshal_VOID__OBJECT, G_TYPE_NONE, 1, G_TYPE_SOCKET);
+  /**
+   * GstMultiHandleSink::clear:
+   * @gstmultisocketsink: the multisocketsink element to emit this signal on
+   *
+   * Remove all sockets from multisocketsink.  Since multisocketsink did not
+   * open sockets itself, it does not explicitly close the sockets. The application
+   * should do so by connecting to the client-socket-removed callback.
+   */
+  gst_multi_handle_sink_signals[SIGNAL_CLEAR] =
+      g_signal_new ("clear", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+      G_STRUCT_OFFSET (GstMultiHandleSinkClass, clear), NULL, NULL,
+      g_cclosure_marshal_VOID__VOID, G_TYPE_NONE, 0);
+
+  /**
+   * GstMultiHandleSink::get-stats:
+   * @gstmultisocketsink: the multisocketsink element to emit this signal on
+   * @socket:             the socket to get stats of from multisocketsink
+   *
+   * Get statistics about @socket. This function returns a GstStructure.
+   *
+   * Returns: a GstStructure with the statistics. The structure contains
+   *     values that represent: total number of bytes sent, time
+   *     when the client was added, time when the client was
+   *     disconnected/removed, time the client is/was active, last activity
+   *     time (in epoch seconds), number of buffers dropped.
+   *     All times are expressed in nanoseconds (GstClockTime).
+   */
+  gst_multi_handle_sink_signals[SIGNAL_GET_STATS] =
+      g_signal_new ("get-stats", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST | G_SIGNAL_ACTION,
+      G_STRUCT_OFFSET (GstMultiHandleSinkClass, get_stats), NULL, NULL,
+      gst_tcp_marshal_BOXED__OBJECT, GST_TYPE_STRUCTURE, 1, G_TYPE_SOCKET);
+
+  /**
+   * GstMultiHandleSink::client-added:
+   * @gstmultisocketsink: the multisocketsink element that emitted this signal
+   * @socket:             the socket that was added to multisocketsink
+   *
+   * The given socket was added to multisocketsink. This signal will
+   * be emitted from the streaming thread so application should be prepared
+   * for that.
+   */
+  gst_multi_handle_sink_signals[SIGNAL_CLIENT_ADDED] =
+      g_signal_new ("client-added", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiHandleSinkClass,
+          client_added), NULL, NULL, g_cclosure_marshal_VOID__OBJECT,
+      G_TYPE_NONE, 1, G_TYPE_OBJECT);
+  /**
+   * GstMultiHandleSink::client-removed:
+   * @gstmultisocketsink: the multisocketsink element that emitted this signal
+   * @socket:             the socket that is to be removed from multisocketsink
+   * @status:             the reason why the client was removed
+   *
+   * The given socket is about to be removed from multisocketsink. This
+   * signal will be emitted from the streaming thread so applications should
+   * be prepared for that.
+   *
+   * @gstmultisocketsink still holds a handle to @socket so it is possible to call
+   * the get-stats signal from this callback. For the same reason it is
+   * not safe to close() and reuse @socket in this callback.
+   */
+  gst_multi_handle_sink_signals[SIGNAL_CLIENT_REMOVED] =
+      g_signal_new ("client-removed", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiHandleSinkClass,
+          client_removed), NULL, NULL, gst_tcp_marshal_VOID__OBJECT_ENUM,
+      G_TYPE_NONE, 2, G_TYPE_INT, GST_TYPE_CLIENT_STATUS);
+  /**
+   * GstMultiHandleSink::client-socket-removed:
+   * @gstmultisocketsink: the multisocketsink element that emitted this signal
+   * @socket:             the socket that was removed from multisocketsink
+   *
+   * The given socket was removed from multisocketsink. This signal will
+   * be emitted from the streaming thread so applications should be prepared
+   * for that.
+   *
+   * In this callback, @gstmultisocketsink has removed all the information
+   * associated with @socket and it is therefore not possible to call get-stats
+   * with @socket. It is however safe to close() and reuse @fd in the callback.
+   *
+   * Since: 0.10.7
+   */
+  gst_multi_handle_sink_signals[SIGNAL_CLIENT_SOCKET_REMOVED] =
+      g_signal_new ("client-socket-removed", G_TYPE_FROM_CLASS (klass),
+      G_SIGNAL_RUN_LAST, G_STRUCT_OFFSET (GstMultiHandleSinkClass,
+          client_socket_removed), NULL, NULL, g_cclosure_marshal_VOID__OBJECT,
+      G_TYPE_NONE, 1, G_TYPE_SOCKET);
+
+  gst_element_class_add_pad_template (gstelement_class,
+      gst_static_pad_template_get (&sinktemplate));
+
+  gst_element_class_set_details_simple (gstelement_class,
+      "Multi socket sink", "Sink/Network",
+      "Send data to multiple sockets",
+      "Thomas Vander Stichele <thomas at apestaart dot org>, "
+      "Wim Taymans <wim@fluendo.com>, "
+      "Sebastian Dröge <sebastian.droege@collabora.co.uk>");
+
+  gstelement_class->change_state =
+      GST_DEBUG_FUNCPTR (gst_multi_handle_sink_change_state);
+
+  gstbasesink_class->render = GST_DEBUG_FUNCPTR (gst_multi_handle_sink_render);
+  gstbasesink_class->unlock = GST_DEBUG_FUNCPTR (gst_multi_handle_sink_unlock);
+  gstbasesink_class->unlock_stop =
+      GST_DEBUG_FUNCPTR (gst_multi_handle_sink_unlock_stop);
+
+  klass->add = GST_DEBUG_FUNCPTR (gst_multi_handle_sink_add);
+  klass->add_full = GST_DEBUG_FUNCPTR (gst_multi_handle_sink_add_full);
+  klass->remove = GST_DEBUG_FUNCPTR (gst_multi_handle_sink_remove);
+  klass->remove_flush = GST_DEBUG_FUNCPTR (gst_multi_handle_sink_remove_flush);
+  klass->clear = GST_DEBUG_FUNCPTR (gst_multi_handle_sink_clear);
+  klass->get_stats = GST_DEBUG_FUNCPTR (gst_multi_handle_sink_get_stats);
+
+  GST_DEBUG_CATEGORY_INIT (multisocketsink_debug, "multisocketsink", 0,
+      "Multi socket sink");
+}
+
+static void
+gst_multi_handle_sink_init (GstMultiHandleSink * this)
+{
+  GST_OBJECT_FLAG_UNSET (this, GST_MULTI_HANDLE_SINK_OPEN);
+
+  CLIENTS_LOCK_INIT (this);
+  this->clients = NULL;
+  this->socket_hash = g_hash_table_new (g_direct_hash, g_int_equal);
+
+  this->bufqueue = g_array_new (FALSE, TRUE, sizeof (GstBuffer *));
+  this->unit_type = DEFAULT_UNIT_TYPE;
+  this->units_max = DEFAULT_UNITS_MAX;
+  this->units_soft_max = DEFAULT_UNITS_SOFT_MAX;
+  this->time_min = DEFAULT_TIME_MIN;
+  this->bytes_min = DEFAULT_BYTES_MIN;
+  this->buffers_min = DEFAULT_BUFFERS_MIN;
+  this->recover_policy = DEFAULT_RECOVER_POLICY;
+
+  this->timeout = DEFAULT_TIMEOUT;
+  this->def_sync_method = DEFAULT_SYNC_METHOD;
+  this->def_burst_format = DEFAULT_BURST_FORMAT;
+  this->def_burst_value = DEFAULT_BURST_VALUE;
+
+  this->qos_dscp = DEFAULT_QOS_DSCP;
+  this->handle_read = DEFAULT_HANDLE_READ;
+
+  this->resend_streamheader = DEFAULT_RESEND_STREAMHEADER;
+
+  this->header_flags = 0;
+  this->cancellable = g_cancellable_new ();
+}
+
+static void
+gst_multi_handle_sink_finalize (GObject * object)
+{
+  GstMultiHandleSink *this;
+
+  this = GST_MULTI_HANDLE_SINK (object);
+
+  CLIENTS_LOCK_CLEAR (this);
+  g_hash_table_destroy (this->socket_hash);
+  g_array_free (this->bufqueue, TRUE);
+
+  if (this->cancellable) {
+    g_object_unref (this->cancellable);
+    this->cancellable = NULL;
+  }
+
+  G_OBJECT_CLASS (parent_class)->finalize (object);
+}
+
+static gint
+setup_dscp_client (GstMultiHandleSink * sink, GstSocketClient * client)
+{
+#ifndef IP_TOS
+  return 0;
+#else
+  gint tos;
+  gint ret;
+  int fd;
+  union gst_sockaddr
+  {
+    struct sockaddr sa;
+    struct sockaddr_in6 sa_in6;
+    struct sockaddr_storage sa_stor;
+  } sa;
+  socklen_t slen = sizeof (sa);
+  gint af;
+
+  /* don't touch */
+  if (sink->qos_dscp < 0)
+    return 0;
+
+  fd = g_socket_get_fd (client->socket);
+
+  if ((ret = getsockname (fd, &sa.sa, &slen)) < 0) {
+    GST_DEBUG_OBJECT (sink, "could not get sockname: %s", g_strerror (errno));
+    return ret;
+  }
+
+  af = sa.sa.sa_family;
+
+  /* if this is an IPv4-mapped address then do IPv4 QoS */
+  if (af == AF_INET6) {
+
+    GST_DEBUG_OBJECT (sink, "check IP6 socket");
+    if (IN6_IS_ADDR_V4MAPPED (&(sa.sa_in6.sin6_addr))) {
+      GST_DEBUG_OBJECT (sink, "mapped to IPV4");
+      af = AF_INET;
+    }
+  }
+
+  /* extract and shift 6 bits of the DSCP */
+  tos = (sink->qos_dscp & 0x3f) << 2;
+
+  switch (af) {
+    case AF_INET:
+      ret = setsockopt (fd, IPPROTO_IP, IP_TOS, &tos, sizeof (tos));
+      break;
+    case AF_INET6:
+#ifdef IPV6_TCLASS
+      ret = setsockopt (fd, IPPROTO_IPV6, IPV6_TCLASS, &tos, sizeof (tos));
+      break;
+#endif
+    default:
+      ret = 0;
+      GST_ERROR_OBJECT (sink, "unsupported AF");
+      break;
+  }
+  if (ret)
+    GST_DEBUG_OBJECT (sink, "could not set DSCP: %s", g_strerror (errno));
+
+  return ret;
+#endif
+}
+
+static void
+setup_dscp (GstMultiHandleSink * sink)
+{
+  GList *clients;
+
+  CLIENTS_LOCK (sink);
+  for (clients = sink->clients; clients; clients = clients->next) {
+    GstSocketClient *client;
+
+    client = clients->data;
+
+    setup_dscp_client (sink, client);
+  }
+  CLIENTS_UNLOCK (sink);
+}
+
+/* "add-full" signal implementation */
+void
+gst_multi_handle_sink_add_full (GstMultiHandleSink * sink, GSocket * socket,
+    GstSyncMethod sync_method, GstFormat min_format, guint64 min_value,
+    GstFormat max_format, guint64 max_value)
+{
+  GstSocketClient *client;
+  GList *clink;
+  GTimeVal now;
+
+  GST_DEBUG_OBJECT (sink, "[socket %p] adding client, sync_method %d, "
+      "min_format %d, min_value %" G_GUINT64_FORMAT
+      ", max_format %d, max_value %" G_GUINT64_FORMAT, socket,
+      sync_method, min_format, min_value, max_format, max_value);
+
+  /* do limits check if we can */
+  if (min_format == max_format) {
+    if (max_value != -1 && min_value != -1 && max_value < min_value)
+      goto wrong_limits;
+  }
+
+  /* create client datastructure */
+  client = g_new0 (GstSocketClient, 1);
+  client->socket = G_SOCKET (g_object_ref (socket));
+  client->status = GST_CLIENT_STATUS_OK;
+  client->bufpos = -1;
+  client->flushcount = -1;
+  client->bufoffset = 0;
+  client->sending = NULL;
+  client->bytes_sent = 0;
+  client->dropped_buffers = 0;
+  client->avg_queue_size = 0;
+  client->first_buffer_ts = GST_CLOCK_TIME_NONE;
+  client->last_buffer_ts = GST_CLOCK_TIME_NONE;
+  client->new_connection = TRUE;
+  client->burst_min_format = min_format;
+  client->burst_min_value = min_value;
+  client->burst_max_format = max_format;
+  client->burst_max_value = max_value;
+  client->sync_method = sync_method;
+  client->currently_removing = FALSE;
+
+  /* update start time */
+  g_get_current_time (&now);
+  client->connect_time = GST_TIMEVAL_TO_TIME (now);
+  client->disconnect_time = 0;
+  /* set last activity time to connect time */
+  client->last_activity_time = client->connect_time;
+
+  CLIENTS_LOCK (sink);
+
+  /* check the hash to find a duplicate fd */
+  clink = g_hash_table_lookup (sink->socket_hash, socket);
+  if (clink != NULL)
+    goto duplicate;
+
+  /* we can add the fd now */
+  clink = sink->clients = g_list_prepend (sink->clients, client);
+  g_hash_table_insert (sink->socket_hash, socket, clink);
+  sink->clients_cookie++;
+
+  /* set the socket to non blocking */
+  g_socket_set_blocking (socket, FALSE);
+
+  /* we always read from a client */
+  if (sink->main_context) {
+    client->source =
+        g_socket_create_source (client->socket,
+        G_IO_IN | G_IO_OUT | G_IO_PRI | G_IO_ERR | G_IO_HUP, sink->cancellable);
+    g_source_set_callback (client->source,
+        (GSourceFunc) gst_multi_handle_sink_socket_condition,
+        gst_object_ref (sink), (GDestroyNotify) gst_object_unref);
+    g_source_attach (client->source, sink->main_context);
+  }
+
+  setup_dscp_client (sink, client);
+
+  CLIENTS_UNLOCK (sink);
+
+  g_signal_emit (G_OBJECT (sink),
+      gst_multi_handle_sink_signals[SIGNAL_CLIENT_ADDED], 0, socket);
+
+  return;
+
+  /* errors */
+wrong_limits:
+  {
+    GST_WARNING_OBJECT (sink,
+        "[socket %p] wrong values min =%" G_GUINT64_FORMAT ", max=%"
+        G_GUINT64_FORMAT ", format %d specified when adding client", socket,
+        min_value, max_value, min_format);
+    return;
+  }
+duplicate:
+  {
+    client->status = GST_CLIENT_STATUS_DUPLICATE;
+    CLIENTS_UNLOCK (sink);
+    GST_WARNING_OBJECT (sink, "[socket %p] duplicate client found, refusing",
+        socket);
+    g_signal_emit (G_OBJECT (sink),
+        gst_multi_handle_sink_signals[SIGNAL_CLIENT_REMOVED], 0, socket,
+        client->status);
+    g_free (client);
+    return;
+  }
+}
+
+/* "add" signal implementation */
+void
+gst_multi_handle_sink_add (GstMultiHandleSink * sink, GSocket * socket)
+{
+  gst_multi_handle_sink_add_full (sink, socket, sink->def_sync_method,
+      sink->def_burst_format, sink->def_burst_value, sink->def_burst_format,
+      -1);
+}
+
+/* "remove" signal implementation */
+void
+gst_multi_handle_sink_remove (GstMultiHandleSink * sink, GSocket * socket)
+{
+  GList *clink;
+
+  GST_DEBUG_OBJECT (sink, "[socket %p] removing client", socket);
+
+  CLIENTS_LOCK (sink);
+  clink = g_hash_table_lookup (sink->socket_hash, socket);
+  if (clink != NULL) {
+    GstSocketClient *client = clink->data;
+
+    if (client->status != GST_CLIENT_STATUS_OK) {
+      GST_INFO_OBJECT (sink,
+          "[socket %p] Client already disconnecting with status %d",
+          socket, client->status);
+      goto done;
+    }
+
+    client->status = GST_CLIENT_STATUS_REMOVED;
+    gst_multi_handle_sink_remove_client_link (sink, clink);
+  } else {
+    GST_WARNING_OBJECT (sink, "[socket %p] no client with this socket found!",
+        socket);
+  }
+
+done:
+  CLIENTS_UNLOCK (sink);
+}
+
+/* "remove-flush" signal implementation */
+void
+gst_multi_handle_sink_remove_flush (GstMultiHandleSink * sink, GSocket * socket)
+{
+  GList *clink;
+
+  GST_DEBUG_OBJECT (sink, "[socket %p] flushing client", socket);
+
+  CLIENTS_LOCK (sink);
+  clink = g_hash_table_lookup (sink->socket_hash, socket);
+  if (clink != NULL) {
+    GstSocketClient *client = clink->data;
+
+    if (client->status != GST_CLIENT_STATUS_OK) {
+      GST_INFO_OBJECT (sink,
+          "[socket %p] Client already disconnecting with status %d",
+          socket, client->status);
+      goto done;
+    }
+
+    /* take the position of the client as the number of buffers left to flush.
+     * If the client was at position -1, we flush 0 buffers, 0 == flush 1
+     * buffer, etc... */
+    client->flushcount = client->bufpos + 1;
+    /* mark client as flushing. We can not remove the client right away because
+     * it might have some buffers to flush in the ->sending queue. */
+    client->status = GST_CLIENT_STATUS_FLUSHING;
+  } else {
+    GST_WARNING_OBJECT (sink, "[socket %p] no client with this fd found!",
+        socket);
+  }
+done:
+  CLIENTS_UNLOCK (sink);
+}
+
+/* can be called both through the signal (i.e. from any thread) or when 
+ * stopping, after the writing thread has shut down */
+void
+gst_multi_handle_sink_clear (GstMultiHandleSink * sink)
+{
+  GList *clients;
+  guint32 cookie;
+
+  GST_DEBUG_OBJECT (sink, "clearing all clients");
+
+  CLIENTS_LOCK (sink);
+restart:
+  cookie = sink->clients_cookie;
+  for (clients = sink->clients; clients; clients = clients->next) {
+    GstSocketClient *client;
+
+    if (cookie != sink->clients_cookie) {
+      GST_DEBUG_OBJECT (sink, "cookie changed while removing all clients");
+      goto restart;
+    }
+
+    client = clients->data;
+    client->status = GST_CLIENT_STATUS_REMOVED;
+    gst_multi_handle_sink_remove_client_link (sink, clients);
+  }
+
+  CLIENTS_UNLOCK (sink);
+}
+
+/* "get-stats" signal implementation
+ */
+GstStructure *
+gst_multi_handle_sink_get_stats (GstMultiHandleSink * sink, GSocket * socket)
+{
+  GstSocketClient *client;
+  GstStructure *result = NULL;
+  GList *clink;
+
+  CLIENTS_LOCK (sink);
+  clink = g_hash_table_lookup (sink->socket_hash, socket);
+  if (clink == NULL)
+    goto noclient;
+
+  client = clink->data;
+  if (client != NULL) {
+    guint64 interval;
+
+    result = gst_structure_new_empty ("multisocketsink-stats");
+
+    if (client->disconnect_time == 0) {
+      GTimeVal nowtv;
+
+      g_get_current_time (&nowtv);
+
+      interval = GST_TIMEVAL_TO_TIME (nowtv) - client->connect_time;
+    } else {
+      interval = client->disconnect_time - client->connect_time;
+    }
+
+    gst_structure_set (result,
+        "bytes-sent", G_TYPE_UINT64, client->bytes_sent,
+        "connect-time", G_TYPE_UINT64, client->connect_time,
+        "disconnect-time", G_TYPE_UINT64, client->disconnect_time,
+        "connected-duration", G_TYPE_UINT64, interval,
+        "last-activatity-time", G_TYPE_UINT64, client->last_activity_time,
+        "dropped-buffers", G_TYPE_UINT64, client->dropped_buffers,
+        "first-buffer-ts", G_TYPE_UINT64, client->first_buffer_ts,
+        "last-buffer-ts", G_TYPE_UINT64, client->last_buffer_ts, NULL);
+  }
+
+noclient:
+  CLIENTS_UNLOCK (sink);
+
+  /* python doesn't like a NULL pointer yet */
+  if (result == NULL) {
+    GST_WARNING_OBJECT (sink, "[socket %p] no client with this found!", socket);
+    result = gst_structure_new_empty ("multisocketsink-stats");
+  }
+
+  return result;
+}
+
+/* should be called with the clientslock helt.
+ * Note that we don't close the fd as we didn't open it in the first
+ * place. An application should connect to the client-fd-removed signal and
+ * close the fd itself.
+ */
+static void
+gst_multi_handle_sink_remove_client_link (GstMultiHandleSink * sink,
+    GList * link)
+{
+  GSocket *socket;
+  GTimeVal now;
+  GstSocketClient *client = link->data;
+  GstMultiHandleSinkClass *fclass;
+
+  fclass = GST_MULTI_HANDLE_SINK_GET_CLASS (sink);
+
+  socket = client->socket;
+
+  if (client->currently_removing) {
+    GST_WARNING_OBJECT (sink, "[socket %p] client is already being removed",
+        socket);
+    return;
+  } else {
+    client->currently_removing = TRUE;
+  }
+
+  /* FIXME: if we keep track of ip we can log it here and signal */
+  switch (client->status) {
+    case GST_CLIENT_STATUS_OK:
+      GST_WARNING_OBJECT (sink, "[socket %p] removing client %p for no reason",
+          socket, client);
+      break;
+    case GST_CLIENT_STATUS_CLOSED:
+      GST_DEBUG_OBJECT (sink, "[socket %p] removing client %p because of close",
+          socket, client);
+      break;
+    case GST_CLIENT_STATUS_REMOVED:
+      GST_DEBUG_OBJECT (sink,
+          "[socket %p] removing client %p because the app removed it", socket,
+          client);
+      break;
+    case GST_CLIENT_STATUS_SLOW:
+      GST_INFO_OBJECT (sink,
+          "[socket %p] removing client %p because it was too slow", socket,
+          client);
+      break;
+    case GST_CLIENT_STATUS_ERROR:
+      GST_WARNING_OBJECT (sink,
+          "[socket %p] removing client %p because of error", socket, client);
+      break;
+    case GST_CLIENT_STATUS_FLUSHING:
+    default:
+      GST_WARNING_OBJECT (sink,
+          "[socket %p] removing client %p with invalid reason %d", socket,
+          client, client->status);
+      break;
+  }
+
+  if (client->source) {
+    g_source_destroy (client->source);
+    g_source_unref (client->source);
+    client->source = NULL;
+  }
+
+  g_get_current_time (&now);
+  client->disconnect_time = GST_TIMEVAL_TO_TIME (now);
+
+  /* free client buffers */
+  g_slist_foreach (client->sending, (GFunc) gst_mini_object_unref, NULL);
+  g_slist_free (client->sending);
+  client->sending = NULL;
+
+  if (client->caps)
+    gst_caps_unref (client->caps);
+  client->caps = NULL;
+
+  /* unlock the mutex before signaling because the signal handler
+   * might query some properties */
+  CLIENTS_UNLOCK (sink);
+
+  g_signal_emit (G_OBJECT (sink),
+      gst_multi_handle_sink_signals[SIGNAL_CLIENT_REMOVED], 0, socket,
+      client->status);
+
+  /* lock again before we remove the client completely */
+  CLIENTS_LOCK (sink);
+
+  /* fd cannot be reused in the above signal callback so we can safely
+   * remove it from the hashtable here */
+  if (!g_hash_table_remove (sink->socket_hash, socket)) {
+    GST_WARNING_OBJECT (sink,
+        "[socket %p] error removing client %p from hash", socket, client);
+  }
+  /* after releasing the lock above, the link could be invalid, more
+   * precisely, the next and prev pointers could point to invalid list
+   * links. One optimisation could be to add a cookie to the linked list
+   * and take a shortcut when it did not change between unlocking and locking
+   * our mutex. For now we just walk the list again. */
+  sink->clients = g_list_remove (sink->clients, client);
+  sink->clients_cookie++;
+
+  if (fclass->removed)
+    fclass->removed (sink, socket);
+
+  g_free (client);
+  CLIENTS_UNLOCK (sink);
+
+  /* and the fd is really gone now */
+  g_signal_emit (G_OBJECT (sink),
+      gst_multi_handle_sink_signals[SIGNAL_CLIENT_SOCKET_REMOVED], 0, socket);
+  g_object_unref (socket);
+
+  CLIENTS_LOCK (sink);
+}
+
+/* handle a read on a client socket,
+ * which either indicates a close or should be ignored
+ * returns FALSE if some error occured or the client closed. */
+static gboolean
+gst_multi_handle_sink_handle_client_read (GstMultiHandleSink * sink,
+    GstSocketClient * client)
+{
+  gboolean ret;
+  gchar dummy[256];
+  gssize nread;
+  GError *err = NULL;
+  gboolean first = TRUE;
+
+  GST_DEBUG_OBJECT (sink, "[socket %p] select reports client read",
+      client->socket);
+
+  ret = TRUE;
+
+  /* just Read 'n' Drop, could also just drop the client as it's not supposed
+   * to write to us except for closing the socket, I guess it's because we
+   * like to listen to our customers. */
+  do {
+    gssize navail;
+
+    GST_DEBUG_OBJECT (sink, "[socket %p] client wants us to read",
+        client->socket);
+
+    navail = g_socket_get_available_bytes (client->socket);
+    if (navail < 0)
+      break;
+
+    nread =
+        g_socket_receive (client->socket, dummy, MIN (navail, sizeof (dummy)),
+        sink->cancellable, &err);
+    if (first && nread == 0) {
+      /* client sent close, so remove it */
+      GST_DEBUG_OBJECT (sink, "[socket %p] client asked for close, removing",
+          client->socket);
+      client->status = GST_CLIENT_STATUS_CLOSED;
+      ret = FALSE;
+    } else if (nread < 0) {
+      GST_WARNING_OBJECT (sink, "[socket %p] could not read: %s",
+          client->socket, err->message);
+      client->status = GST_CLIENT_STATUS_ERROR;
+      ret = FALSE;
+      break;
+    }
+    first = FALSE;
+  } while (nread > 0);
+  g_clear_error (&err);
+
+  return ret;
+}
+
+static gboolean
+is_sync_frame (GstMultiHandleSink * sink, GstBuffer * buffer)
+{
+  if (GST_BUFFER_FLAG_IS_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT)) {
+    return FALSE;
+  } else if (!GST_BUFFER_FLAG_IS_SET (buffer, GST_BUFFER_FLAG_IN_CAPS)) {
+    return TRUE;
+  }
+
+  return FALSE;
+}
+
+/* queue the given buffer for the given client */
+static gboolean
+gst_multi_handle_sink_client_queue_buffer (GstMultiHandleSink * sink,
+    GstSocketClient * client, GstBuffer * buffer)
+{
+  GstCaps *caps;
+
+  /* TRUE: send them if the new caps have them */
+  gboolean send_streamheader = FALSE;
+  GstStructure *s;
+
+  /* before we queue the buffer, we check if we need to queue streamheader
+   * buffers (because it's a new client, or because they changed) */
+  caps = gst_pad_get_current_caps (GST_BASE_SINK_PAD (sink));
+
+  if (!client->caps) {
+    GST_DEBUG_OBJECT (sink,
+        "[socket %p] no previous caps for this client, send streamheader",
+        client->socket);
+    send_streamheader = TRUE;
+    client->caps = gst_caps_ref (caps);
+  } else {
+    /* there were previous caps recorded, so compare */
+    if (!gst_caps_is_equal (caps, client->caps)) {
+      const GValue *sh1, *sh2;
+
+      /* caps are not equal, but could still have the same streamheader */
+      s = gst_caps_get_structure (caps, 0);
+      if (!gst_structure_has_field (s, "streamheader")) {
+        /* no new streamheader, so nothing new to send */
+        GST_DEBUG_OBJECT (sink,
+            "[socket %p] new caps do not have streamheader, not sending",
+            client->socket);
+      } else {
+        /* there is a new streamheader */
+        s = gst_caps_get_structure (client->caps, 0);
+        if (!gst_structure_has_field (s, "streamheader")) {
+          /* no previous streamheader, so send the new one */
+          GST_DEBUG_OBJECT (sink,
+              "[socket %p] previous caps did not have streamheader, sending",
+              client->socket);
+          send_streamheader = TRUE;
+        } else {
+          /* both old and new caps have streamheader set */
+          if (!sink->resend_streamheader) {
+            GST_DEBUG_OBJECT (sink,
+                "[socket %p] asked to not resend the streamheader, not sending",
+                client->socket);
+            send_streamheader = FALSE;
+          } else {
+            sh1 = gst_structure_get_value (s, "streamheader");
+            s = gst_caps_get_structure (caps, 0);
+            sh2 = gst_structure_get_value (s, "streamheader");
+            if (gst_value_compare (sh1, sh2) != GST_VALUE_EQUAL) {
+              GST_DEBUG_OBJECT (sink,
+                  "[socket %p] new streamheader different from old, sending",
+                  client->socket);
+              send_streamheader = TRUE;
+            }
+          }
+        }
+      }
+    }
+    /* Replace the old caps */
+    gst_caps_unref (client->caps);
+    client->caps = gst_caps_ref (caps);
+  }
+
+  if (G_UNLIKELY (send_streamheader)) {
+    const GValue *sh;
+    GArray *buffers;
+    int i;
+
+    GST_LOG_OBJECT (sink,
+        "[socket %p] sending streamheader from caps %" GST_PTR_FORMAT,
+        client->socket, caps);
+    s = gst_caps_get_structure (caps, 0);
+    if (!gst_structure_has_field (s, "streamheader")) {
+      GST_DEBUG_OBJECT (sink,
+          "[socket %p] no new streamheader, so nothing to send",
+          client->socket);
+    } else {
+      GST_LOG_OBJECT (sink,
+          "[socket %p] sending streamheader from caps %" GST_PTR_FORMAT,
+          client->socket, caps);
+      sh = gst_structure_get_value (s, "streamheader");
+      g_assert (G_VALUE_TYPE (sh) == GST_TYPE_ARRAY);
+      buffers = g_value_peek_pointer (sh);
+      GST_DEBUG_OBJECT (sink, "%d streamheader buffers", buffers->len);
+      for (i = 0; i < buffers->len; ++i) {
+        GValue *bufval;
+        GstBuffer *buffer;
+
+        bufval = &g_array_index (buffers, GValue, i);
+        g_assert (G_VALUE_TYPE (bufval) == GST_TYPE_BUFFER);
+        buffer = g_value_peek_pointer (bufval);
+        GST_DEBUG_OBJECT (sink,
+            "[socket %p] queueing streamheader buffer of length %"
+            G_GSIZE_FORMAT, client->socket, gst_buffer_get_size (buffer));
+        gst_buffer_ref (buffer);
+
+        client->sending = g_slist_append (client->sending, buffer);
+      }
+    }
+  }
+
+  gst_caps_unref (caps);
+  caps = NULL;
+
+  GST_LOG_OBJECT (sink,
+      "[socket %p] queueing buffer of length %" G_GSIZE_FORMAT, client->socket,
+      gst_buffer_get_size (buffer));
+
+  gst_buffer_ref (buffer);
+  client->sending = g_slist_append (client->sending, buffer);
+
+  return TRUE;
+}
+
+/* find the keyframe in the list of buffers starting the
+ * search from @idx. @direction as -1 will search backwards, 
+ * 1 will search forwards.
+ * Returns: the index or -1 if there is no keyframe after idx.
+ */
+static gint
+find_syncframe (GstMultiHandleSink * sink, gint idx, gint direction)
+{
+  gint i, len, result;
+
+  /* take length of queued buffers */
+  len = sink->bufqueue->len;
+
+  /* assume we don't find a keyframe */
+  result = -1;
+
+  /* then loop over all buffers to find the first keyframe */
+  for (i = idx; i >= 0 && i < len; i += direction) {
+    GstBuffer *buf;
+
+    buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+    if (is_sync_frame (sink, buf)) {
+      GST_LOG_OBJECT (sink, "found keyframe at %d from %d, direction %d",
+          i, idx, direction);
+      result = i;
+      break;
+    }
+  }
+  return result;
+}
+
+#define find_next_syncframe(s,i)       find_syncframe(s,i,1)
+#define find_prev_syncframe(s,i)       find_syncframe(s,i,-1)
+
+/* Get the number of buffers from the buffer queue needed to satisfy
+ * the maximum max in the configured units.
+ * If units are not BUFFERS, and there are insufficient buffers in the
+ * queue to satify the limit, return len(queue) + 1 */
+static gint
+get_buffers_max (GstMultiHandleSink * sink, gint64 max)
+{
+  switch (sink->unit_type) {
+    case GST_FORMAT_BUFFERS:
+      return max;
+    case GST_FORMAT_TIME:
+    {
+      GstBuffer *buf;
+      int i;
+      int len;
+      gint64 diff;
+      GstClockTime first = GST_CLOCK_TIME_NONE;
+
+      len = sink->bufqueue->len;
+
+      for (i = 0; i < len; i++) {
+        buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+        if (GST_BUFFER_TIMESTAMP_IS_VALID (buf)) {
+          if (first == -1)
+            first = GST_BUFFER_TIMESTAMP (buf);
+
+          diff = first - GST_BUFFER_TIMESTAMP (buf);
+
+          if (diff > max)
+            return i + 1;
+        }
+      }
+      return len + 1;
+    }
+    case GST_FORMAT_BYTES:
+    {
+      GstBuffer *buf;
+      int i;
+      int len;
+      gint acc = 0;
+
+      len = sink->bufqueue->len;
+
+      for (i = 0; i < len; i++) {
+        buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+        acc += gst_buffer_get_size (buf);
+
+        if (acc > max)
+          return i + 1;
+      }
+      return len + 1;
+    }
+    default:
+      return max;
+  }
+}
+
+/* find the positions in the buffer queue where *_min and *_max
+ * is satisfied
+ */
+/* count the amount of data in the buffers and return the index
+ * that satifies the given limits.
+ *
+ * Returns: index @idx in the buffer queue so that the given limits are
+ * satisfied. TRUE if all the limits could be satisfied, FALSE if not
+ * enough data was in the queue.
+ *
+ * FIXME, this code might now work if any of the units is in buffers...
+ */
+static gboolean
+find_limits (GstMultiHandleSink * sink,
+    gint * min_idx, gint bytes_min, gint buffers_min, gint64 time_min,
+    gint * max_idx, gint bytes_max, gint buffers_max, gint64 time_max)
+{
+  GstClockTime first, time;
+  gint i, len, bytes;
+  gboolean result, max_hit;
+
+  /* take length of queue */
+  len = sink->bufqueue->len;
+
+  /* this must hold */
+  g_assert (len > 0);
+
+  GST_LOG_OBJECT (sink,
+      "bytes_min %d, buffers_min %d, time_min %" GST_TIME_FORMAT
+      ", bytes_max %d, buffers_max %d, time_max %" GST_TIME_FORMAT, bytes_min,
+      buffers_min, GST_TIME_ARGS (time_min), bytes_max, buffers_max,
+      GST_TIME_ARGS (time_max));
+
+  /* do the trivial buffer limit test */
+  if (buffers_min != -1 && len < buffers_min) {
+    *min_idx = len - 1;
+    *max_idx = len - 1;
+    return FALSE;
+  }
+
+  result = FALSE;
+  /* else count bytes and time */
+  first = -1;
+  bytes = 0;
+  /* unset limits */
+  *min_idx = -1;
+  *max_idx = -1;
+  max_hit = FALSE;
+
+  i = 0;
+  /* loop through the buffers, when a limit is ok, mark it 
+   * as -1, we have at least one buffer in the queue. */
+  do {
+    GstBuffer *buf;
+
+    /* if we checked all min limits, update result */
+    if (bytes_min == -1 && time_min == -1 && *min_idx == -1) {
+      /* don't go below 0 */
+      *min_idx = MAX (i - 1, 0);
+    }
+    /* if we reached one max limit break out */
+    if (max_hit) {
+      /* i > 0 when we get here, we subtract one to get the position
+       * of the previous buffer. */
+      *max_idx = i - 1;
+      /* we have valid complete result if we found a min_idx too */
+      result = *min_idx != -1;
+      break;
+    }
+    buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+
+    bytes += gst_buffer_get_size (buf);
+
+    /* take timestamp and save for the base first timestamp */
+    if ((time = GST_BUFFER_TIMESTAMP (buf)) != -1) {
+      GST_LOG_OBJECT (sink, "Ts %" GST_TIME_FORMAT " on buffer",
+          GST_TIME_ARGS (time));
+      if (first == -1)
+        first = time;
+
+      /* increase max usage if we did not fill enough. Note that
+       * buffers are sorted from new to old, so the first timestamp is
+       * bigger than the next one. */
+      if (time_min != -1 && first - time >= time_min)
+        time_min = -1;
+      if (time_max != -1 && first - time >= time_max)
+        max_hit = TRUE;
+    } else {
+      GST_LOG_OBJECT (sink, "No timestamp on buffer");
+    }
+    /* time is OK or unknown, check and increase if not enough bytes */
+    if (bytes_min != -1) {
+      if (bytes >= bytes_min)
+        bytes_min = -1;
+    }
+    if (bytes_max != -1) {
+      if (bytes >= bytes_max) {
+        max_hit = TRUE;
+      }
+    }
+    i++;
+  }
+  while (i < len);
+
+  /* if we did not hit the max or min limit, set to buffer size */
+  if (*max_idx == -1)
+    *max_idx = len - 1;
+  /* make sure min does not exceed max */
+  if (*min_idx == -1)
+    *min_idx = *max_idx;
+
+  return result;
+}
+
+/* parse the unit/value pair and assign it to the result value of the
+ * right type, leave the other values untouched 
+ *
+ * Returns: FALSE if the unit is unknown or undefined. TRUE otherwise.
+ */
+static gboolean
+assign_value (GstFormat format, guint64 value, gint * bytes, gint * buffers,
+    GstClockTime * time)
+{
+  gboolean res = TRUE;
+
+  /* set only the limit of the given format to the given value */
+  switch (format) {
+    case GST_FORMAT_BUFFERS:
+      *buffers = (gint) value;
+      break;
+    case GST_FORMAT_TIME:
+      *time = value;
+      break;
+    case GST_FORMAT_BYTES:
+      *bytes = (gint) value;
+      break;
+    case GST_FORMAT_UNDEFINED:
+    default:
+      res = FALSE;
+      break;
+  }
+  return res;
+}
+
+/* count the index in the buffer queue to satisfy the given unit
+ * and value pair starting from buffer at index 0.
+ *
+ * Returns: TRUE if there was enough data in the queue to satisfy the
+ * burst values. @idx contains the index in the buffer that contains enough
+ * data to satisfy the limits or the last buffer in the queue when the
+ * function returns FALSE.
+ */
+static gboolean
+count_burst_unit (GstMultiHandleSink * sink, gint * min_idx,
+    GstFormat min_format, guint64 min_value, gint * max_idx,
+    GstFormat max_format, guint64 max_value)
+{
+  gint bytes_min = -1, buffers_min = -1;
+  gint bytes_max = -1, buffers_max = -1;
+  GstClockTime time_min = GST_CLOCK_TIME_NONE, time_max = GST_CLOCK_TIME_NONE;
+
+  assign_value (min_format, min_value, &bytes_min, &buffers_min, &time_min);
+  assign_value (max_format, max_value, &bytes_max, &buffers_max, &time_max);
+
+  return find_limits (sink, min_idx, bytes_min, buffers_min, time_min,
+      max_idx, bytes_max, buffers_max, time_max);
+}
+
+/* decide where in the current buffer queue this new client should start
+ * receiving buffers from.
+ * This function is called whenever a client is connected and has not yet
+ * received a buffer.
+ * If this returns -1, it means that we haven't found a good point to
+ * start streaming from yet, and this function should be called again later
+ * when more buffers have arrived.
+ */
+static gint
+gst_multi_handle_sink_new_client (GstMultiHandleSink * sink,
+    GstSocketClient * client)
+{
+  gint result;
+
+  GST_DEBUG_OBJECT (sink,
+      "[socket %p] new client, deciding where to start in queue",
+      client->socket);
+  GST_DEBUG_OBJECT (sink, "queue is currently %d buffers long",
+      sink->bufqueue->len);
+  switch (client->sync_method) {
+    case GST_SYNC_METHOD_LATEST:
+      /* no syncing, we are happy with whatever the client is going to get */
+      result = client->bufpos;
+      GST_DEBUG_OBJECT (sink,
+          "[socket %p] SYNC_METHOD_LATEST, position %d", client->socket,
+          result);
+      break;
+    case GST_SYNC_METHOD_NEXT_KEYFRAME:
+    {
+      /* if one of the new buffers (between client->bufpos and 0) in the queue
+       * is a sync point, we can proceed, otherwise we need to keep waiting */
+      GST_LOG_OBJECT (sink,
+          "[socket %p] new client, bufpos %d, waiting for keyframe",
+          client->socket, client->bufpos);
+
+      result = find_prev_syncframe (sink, client->bufpos);
+      if (result != -1) {
+        GST_DEBUG_OBJECT (sink,
+            "[socket %p] SYNC_METHOD_NEXT_KEYFRAME: result %d",
+            client->socket, result);
+        break;
+      }
+
+      /* client is not on a syncbuffer, need to skip these buffers and
+       * wait some more */
+      GST_LOG_OBJECT (sink,
+          "[socket %p] new client, skipping buffer(s), no syncpoint found",
+          client->socket);
+      client->bufpos = -1;
+      break;
+    }
+    case GST_SYNC_METHOD_LATEST_KEYFRAME:
+    {
+      GST_DEBUG_OBJECT (sink,
+          "[socket %p] SYNC_METHOD_LATEST_KEYFRAME", client->socket);
+
+      /* for new clients we initially scan the complete buffer queue for
+       * a sync point when a buffer is added. If we don't find a keyframe,
+       * we need to wait for the next keyframe and so we change the client's
+       * sync method to GST_SYNC_METHOD_NEXT_KEYFRAME.
+       */
+      result = find_next_syncframe (sink, 0);
+      if (result != -1) {
+        GST_DEBUG_OBJECT (sink,
+            "[socket %p] SYNC_METHOD_LATEST_KEYFRAME: result %d",
+            client->socket, result);
+        break;
+      }
+
+      GST_DEBUG_OBJECT (sink,
+          "[socket %p] SYNC_METHOD_LATEST_KEYFRAME: no keyframe found, "
+          "switching to SYNC_METHOD_NEXT_KEYFRAME", client->socket);
+      /* throw client to the waiting state */
+      client->bufpos = -1;
+      /* and make client sync to next keyframe */
+      client->sync_method = GST_SYNC_METHOD_NEXT_KEYFRAME;
+      break;
+    }
+    case GST_SYNC_METHOD_BURST:
+    {
+      gboolean ok;
+      gint max;
+
+      /* move to the position where we satisfy the client's burst
+       * parameters. If we could not satisfy the parameters because there
+       * is not enough data, we just send what we have (which is in result).
+       * We use the max value to limit the search
+       */
+      ok = count_burst_unit (sink, &result, client->burst_min_format,
+          client->burst_min_value, &max, client->burst_max_format,
+          client->burst_max_value);
+      GST_DEBUG_OBJECT (sink,
+          "[socket %p] SYNC_METHOD_BURST: burst_unit returned %d, result %d",
+          client->socket, ok, result);
+
+      GST_LOG_OBJECT (sink, "min %d, max %d", result, max);
+
+      /* we hit the max and it is below the min, use that then */
+      if (max != -1 && max <= result) {
+        result = MAX (max - 1, 0);
+        GST_DEBUG_OBJECT (sink,
+            "[socket %p] SYNC_METHOD_BURST: result above max, taken down to %d",
+            client->socket, result);
+      }
+      break;
+    }
+    case GST_SYNC_METHOD_BURST_KEYFRAME:
+    {
+      gint min_idx, max_idx;
+      gint next_syncframe, prev_syncframe;
+
+      /* BURST_KEYFRAME:
+       *
+       * _always_ start sending a keyframe to the client. We first search
+       * a keyframe between min/max limits. If there is none, we send it the
+       * last keyframe before min. If there is none, the behaviour is like
+       * NEXT_KEYFRAME.
+       */
+      /* gather burst limits */
+      count_burst_unit (sink, &min_idx, client->burst_min_format,
+          client->burst_min_value, &max_idx, client->burst_max_format,
+          client->burst_max_value);
+
+      GST_LOG_OBJECT (sink, "min %d, max %d", min_idx, max_idx);
+
+      /* first find a keyframe after min_idx */
+      next_syncframe = find_next_syncframe (sink, min_idx);
+      if (next_syncframe != -1 && next_syncframe < max_idx) {
+        /* we have a valid keyframe and it's below the max */
+        GST_LOG_OBJECT (sink, "found keyframe in min/max limits");
+        result = next_syncframe;
+        break;
+      }
+
+      /* no valid keyframe, try to find one below min */
+      prev_syncframe = find_prev_syncframe (sink, min_idx);
+      if (prev_syncframe != -1) {
+        GST_WARNING_OBJECT (sink,
+            "using keyframe below min in BURST_KEYFRAME sync mode");
+        result = prev_syncframe;
+        break;
+      }
+
+      /* no prev keyframe or not enough data  */
+      GST_WARNING_OBJECT (sink,
+          "no prev keyframe found in BURST_KEYFRAME sync mode, waiting for next");
+
+      /* throw client to the waiting state */
+      client->bufpos = -1;
+      /* and make client sync to next keyframe */
+      client->sync_method = GST_SYNC_METHOD_NEXT_KEYFRAME;
+      result = -1;
+      break;
+    }
+    case GST_SYNC_METHOD_BURST_WITH_KEYFRAME:
+    {
+      gint min_idx, max_idx;
+      gint next_syncframe;
+
+      /* BURST_WITH_KEYFRAME:
+       *
+       * try to start sending a keyframe to the client. We first search
+       * a keyframe between min/max limits. If there is none, we send it the
+       * amount of data up 'till min.
+       */
+      /* gather enough data to burst */
+      count_burst_unit (sink, &min_idx, client->burst_min_format,
+          client->burst_min_value, &max_idx, client->burst_max_format,
+          client->burst_max_value);
+
+      GST_LOG_OBJECT (sink, "min %d, max %d", min_idx, max_idx);
+
+      /* first find a keyframe after min_idx */
+      next_syncframe = find_next_syncframe (sink, min_idx);
+      if (next_syncframe != -1 && next_syncframe < max_idx) {
+        /* we have a valid keyframe and it's below the max */
+        GST_LOG_OBJECT (sink, "found keyframe in min/max limits");
+        result = next_syncframe;
+        break;
+      }
+
+      /* no keyframe, send data from min_idx */
+      GST_WARNING_OBJECT (sink, "using min in BURST_WITH_KEYFRAME sync mode");
+
+      /* make sure we don't go over the max limit */
+      if (max_idx != -1 && max_idx <= min_idx) {
+        result = MAX (max_idx - 1, 0);
+      } else {
+        result = min_idx;
+      }
+
+      break;
+    }
+    default:
+      g_warning ("unknown sync method %d", client->sync_method);
+      result = client->bufpos;
+      break;
+  }
+  return result;
+}
+
+/* Handle a write on a client,
+ * which indicates a read request from a client.
+ *
+ * For each client we maintain a queue of GstBuffers that contain the raw bytes
+ * we need to send to the client.
+ *
+ * We first check to see if we need to send streamheaders. If so, we queue them.
+ *
+ * Then we run into the main loop that tries to send as many buffers as
+ * possible. It will first exhaust the client->sending queue and if the queue
+ * is empty, it will pick a buffer from the global queue.
+ *
+ * Sending the buffers from the client->sending queue is basically writing
+ * the bytes to the socket and maintaining a count of the bytes that were
+ * sent. When the buffer is completely sent, it is removed from the
+ * client->sending queue and we try to pick a new buffer for sending.
+ *
+ * When the sending returns a partial buffer we stop sending more data as
+ * the next send operation could block.
+ *
+ * This functions returns FALSE if some error occured.
+ */
+static gboolean
+gst_multi_handle_sink_handle_client_write (GstMultiHandleSink * sink,
+    GstSocketClient * client)
+{
+  GSocket *socket = client->socket;
+  gboolean more;
+  gboolean flushing;
+  GstClockTime now;
+  GTimeVal nowtv;
+  GError *err = NULL;
+
+  g_get_current_time (&nowtv);
+  now = GST_TIMEVAL_TO_TIME (nowtv);
+
+  flushing = client->status == GST_CLIENT_STATUS_FLUSHING;
+
+  more = TRUE;
+  do {
+    gint maxsize;
+
+    if (!client->sending) {
+      /* client is not working on a buffer */
+      if (client->bufpos == -1) {
+        /* client is too fast, remove from write queue until new buffer is
+         * available */
+        if (client->source) {
+          g_source_destroy (client->source);
+          g_source_unref (client->source);
+          client->source = NULL;
+        }
+        /* if we flushed out all of the client buffers, we can stop */
+        if (client->flushcount == 0)
+          goto flushed;
+
+        return TRUE;
+      } else {
+        /* client can pick a buffer from the global queue */
+        GstBuffer *buf;
+        GstClockTime timestamp;
+
+        /* for new connections, we need to find a good spot in the
+         * bufqueue to start streaming from */
+        if (client->new_connection && !flushing) {
+          gint position = gst_multi_handle_sink_new_client (sink, client);
+
+          if (position >= 0) {
+            /* we got a valid spot in the queue */
+            client->new_connection = FALSE;
+            client->bufpos = position;
+          } else {
+            /* cannot send data to this client yet */
+            if (client->source) {
+              g_source_destroy (client->source);
+              g_source_unref (client->source);
+              client->source = NULL;
+            }
+            return TRUE;
+          }
+        }
+
+        /* we flushed all remaining buffers, no need to get a new one */
+        if (client->flushcount == 0)
+          goto flushed;
+
+        /* grab buffer */
+        buf = g_array_index (sink->bufqueue, GstBuffer *, client->bufpos);
+        client->bufpos--;
+
+        /* update stats */
+        timestamp = GST_BUFFER_TIMESTAMP (buf);
+        if (client->first_buffer_ts == GST_CLOCK_TIME_NONE)
+          client->first_buffer_ts = timestamp;
+        if (timestamp != -1)
+          client->last_buffer_ts = timestamp;
+
+        /* decrease flushcount */
+        if (client->flushcount != -1)
+          client->flushcount--;
+
+        GST_LOG_OBJECT (sink, "[socket %p] client %p at position %d",
+            socket, client, client->bufpos);
+
+        /* queueing a buffer will ref it */
+        gst_multi_handle_sink_client_queue_buffer (sink, client, buf);
+
+        /* need to start from the first byte for this new buffer */
+        client->bufoffset = 0;
+      }
+    }
+
+    /* see if we need to send something */
+    if (client->sending) {
+      gssize wrote;
+      GstBuffer *head;
+      GstMapInfo map;
+
+      /* pick first buffer from list */
+      head = GST_BUFFER (client->sending->data);
+
+      gst_buffer_map (head, &map, GST_MAP_READ);
+      maxsize = map.size - client->bufoffset;
+
+      /* try to write the complete buffer */
+
+      wrote =
+          g_socket_send (socket, (gchar *) map.data + client->bufoffset,
+          maxsize, sink->cancellable, &err);
+      gst_buffer_unmap (head, &map);
+
+      if (wrote < 0) {
+        /* hmm error.. */
+        if (g_error_matches (err, G_IO_ERROR, G_IO_ERROR_CLOSED)) {
+          goto connection_reset;
+        } else {
+          goto write_error;
+        }
+      } else {
+        if (wrote < maxsize) {
+          /* partial write means that the client cannot read more and we should
+           * stop sending more */
+          GST_LOG_OBJECT (sink,
+              "partial write on %p of %" G_GSSIZE_FORMAT " bytes", socket,
+              wrote);
+          client->bufoffset += wrote;
+          more = FALSE;
+        } else {
+          /* complete buffer was written, we can proceed to the next one */
+          client->sending = g_slist_remove (client->sending, head);
+          gst_buffer_unref (head);
+          /* make sure we start from byte 0 for the next buffer */
+          client->bufoffset = 0;
+        }
+        /* update stats */
+        client->bytes_sent += wrote;
+        client->last_activity_time = now;
+        sink->bytes_served += wrote;
+      }
+    }
+  } while (more);
+
+  return TRUE;
+
+  /* ERRORS */
+flushed:
+  {
+    GST_DEBUG_OBJECT (sink, "[socket %p] flushed, removing", socket);
+    client->status = GST_CLIENT_STATUS_REMOVED;
+    return FALSE;
+  }
+connection_reset:
+  {
+    GST_DEBUG_OBJECT (sink, "[socket %p] connection reset by peer, removing",
+        socket);
+    client->status = GST_CLIENT_STATUS_CLOSED;
+    g_clear_error (&err);
+    return FALSE;
+  }
+write_error:
+  {
+    GST_WARNING_OBJECT (sink,
+        "[socket %p] could not write, removing client: %s", socket,
+        err->message);
+    g_clear_error (&err);
+    client->status = GST_CLIENT_STATUS_ERROR;
+    return FALSE;
+  }
+}
+
+/* calculate the new position for a client after recovery. This function
+ * does not update the client position but merely returns the required
+ * position.
+ */
+static gint
+gst_multi_handle_sink_recover_client (GstMultiHandleSink * sink,
+    GstSocketClient * client)
+{
+  gint newbufpos;
+
+  GST_WARNING_OBJECT (sink,
+      "[socket %p] client %p is lagging at %d, recover using policy %d",
+      client->socket, client, client->bufpos, sink->recover_policy);
+
+  switch (sink->recover_policy) {
+    case GST_RECOVER_POLICY_NONE:
+      /* do nothing, client will catch up or get kicked out when it reaches
+       * the hard max */
+      newbufpos = client->bufpos;
+      break;
+    case GST_RECOVER_POLICY_RESYNC_LATEST:
+      /* move to beginning of queue */
+      newbufpos = -1;
+      break;
+    case GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT:
+      /* move to beginning of soft max */
+      newbufpos = get_buffers_max (sink, sink->units_soft_max);
+      break;
+    case GST_RECOVER_POLICY_RESYNC_KEYFRAME:
+      /* find keyframe in buffers, we search backwards to find the
+       * closest keyframe relative to what this client already received. */
+      newbufpos = MIN (sink->bufqueue->len - 1,
+          get_buffers_max (sink, sink->units_soft_max) - 1);
+
+      while (newbufpos >= 0) {
+        GstBuffer *buf;
+
+        buf = g_array_index (sink->bufqueue, GstBuffer *, newbufpos);
+        if (is_sync_frame (sink, buf)) {
+          /* found a buffer that is not a delta unit */
+          break;
+        }
+        newbufpos--;
+      }
+      break;
+    default:
+      /* unknown recovery procedure */
+      newbufpos = get_buffers_max (sink, sink->units_soft_max);
+      break;
+  }
+  return newbufpos;
+}
+
+/* Queue a buffer on the global queue.
+ *
+ * This function adds the buffer to the front of a GArray. It removes the
+ * tail buffer if the max queue size is exceeded, unreffing the queued buffer.
+ * Note that unreffing the buffer is not a problem as clients who
+ * started writing out this buffer will still have a reference to it in the
+ * client->sending queue.
+ *
+ * After adding the buffer, we update all client positions in the queue. If
+ * a client moves over the soft max, we start the recovery procedure for this
+ * slow client. If it goes over the hard max, it is put into the slow list
+ * and removed.
+ *
+ * Special care is taken of clients that were waiting for a new buffer (they
+ * had a position of -1) because they can proceed after adding this new buffer.
+ * This is done by adding the client back into the write fd_set and signaling
+ * the select thread that the fd_set changed.
+ */
+static void
+gst_multi_handle_sink_queue_buffer (GstMultiHandleSink * sink, GstBuffer * buf)
+{
+  GList *clients, *next;
+  gint queuelen;
+  gint max_buffer_usage;
+  gint i;
+  GTimeVal nowtv;
+  GstClockTime now;
+  gint max_buffers, soft_max_buffers;
+  guint cookie;
+
+  g_get_current_time (&nowtv);
+  now = GST_TIMEVAL_TO_TIME (nowtv);
+
+  CLIENTS_LOCK (sink);
+  /* add buffer to queue */
+  gst_buffer_ref (buf);
+  g_array_prepend_val (sink->bufqueue, buf);
+  queuelen = sink->bufqueue->len;
+
+  if (sink->units_max > 0)
+    max_buffers = get_buffers_max (sink, sink->units_max);
+  else
+    max_buffers = -1;
+
+  if (sink->units_soft_max > 0)
+    soft_max_buffers = get_buffers_max (sink, sink->units_soft_max);
+  else
+    soft_max_buffers = -1;
+  GST_LOG_OBJECT (sink, "Using max %d, softmax %d", max_buffers,
+      soft_max_buffers);
+
+  /* then loop over the clients and update the positions */
+  max_buffer_usage = 0;
+
+restart:
+  cookie = sink->clients_cookie;
+  for (clients = sink->clients; clients; clients = next) {
+    GstSocketClient *client;
+
+    if (cookie != sink->clients_cookie) {
+      GST_DEBUG_OBJECT (sink, "Clients cookie outdated, restarting");
+      goto restart;
+    }
+
+    client = clients->data;
+    next = g_list_next (clients);
+
+    client->bufpos++;
+    GST_LOG_OBJECT (sink, "[socket %p] client %p at position %d",
+        client->socket, client, client->bufpos);
+    /* check soft max if needed, recover client */
+    if (soft_max_buffers > 0 && client->bufpos >= soft_max_buffers) {
+      gint newpos;
+
+      newpos = gst_multi_handle_sink_recover_client (sink, client);
+      if (newpos != client->bufpos) {
+        client->dropped_buffers += client->bufpos - newpos;
+        client->bufpos = newpos;
+        client->discont = TRUE;
+        GST_INFO_OBJECT (sink, "[socket %p] client %p position reset to %d",
+            client->socket, client, client->bufpos);
+      } else {
+        GST_INFO_OBJECT (sink,
+            "[socket %p] client %p not recovering position",
+            client->socket, client);
+      }
+    }
+    /* check hard max and timeout, remove client */
+    if ((max_buffers > 0 && client->bufpos >= max_buffers) ||
+        (sink->timeout > 0
+            && now - client->last_activity_time > sink->timeout)) {
+      /* remove client */
+      GST_WARNING_OBJECT (sink, "[socket %p] client %p is too slow, removing",
+          client->socket, client);
+      /* remove the client, the fd set will be cleared and the select thread
+       * will be signaled */
+      client->status = GST_CLIENT_STATUS_SLOW;
+      /* set client to invalid position while being removed */
+      client->bufpos = -1;
+      gst_multi_handle_sink_remove_client_link (sink, clients);
+      continue;
+    } else if (client->bufpos == 0 || client->new_connection) {
+      /* can send data to this client now. need to signal the select thread that
+       * the fd_set changed */
+      if (!client->source) {
+        client->source =
+            g_socket_create_source (client->socket,
+            G_IO_IN | G_IO_OUT | G_IO_PRI | G_IO_ERR | G_IO_HUP,
+            sink->cancellable);
+        g_source_set_callback (client->source,
+            (GSourceFunc) gst_multi_handle_sink_socket_condition,
+            gst_object_ref (sink), (GDestroyNotify) gst_object_unref);
+        g_source_attach (client->source, sink->main_context);
+      }
+    }
+    /* keep track of maximum buffer usage */
+    if (client->bufpos > max_buffer_usage) {
+      max_buffer_usage = client->bufpos;
+    }
+  }
+
+  /* make sure we respect bytes-min, buffers-min and time-min when they are set */
+  {
+    gint usage, max;
+
+    GST_LOG_OBJECT (sink,
+        "extending queue %d to respect time_min %" GST_TIME_FORMAT
+        ", bytes_min %d, buffers_min %d", max_buffer_usage,
+        GST_TIME_ARGS (sink->time_min), sink->bytes_min, sink->buffers_min);
+
+    /* get index where the limits are ok, we don't really care if all limits
+     * are ok, we just queue as much as we need. We also don't compare against
+     * the max limits. */
+    find_limits (sink, &usage, sink->bytes_min, sink->buffers_min,
+        sink->time_min, &max, -1, -1, -1);
+
+    max_buffer_usage = MAX (max_buffer_usage, usage + 1);
+    GST_LOG_OBJECT (sink, "extended queue to %d", max_buffer_usage);
+  }
+
+  /* now look for sync points and make sure there is at least one
+   * sync point in the queue. We only do this if the LATEST_KEYFRAME or 
+   * BURST_KEYFRAME mode is selected */
+  if (sink->def_sync_method == GST_SYNC_METHOD_LATEST_KEYFRAME ||
+      sink->def_sync_method == GST_SYNC_METHOD_BURST_KEYFRAME) {
+    /* no point in searching beyond the queue length */
+    gint limit = queuelen;
+    GstBuffer *buf;
+
+    /* no point in searching beyond the soft-max if any. */
+    if (soft_max_buffers > 0) {
+      limit = MIN (limit, soft_max_buffers);
+    }
+    GST_LOG_OBJECT (sink,
+        "extending queue to include sync point, now at %d, limit is %d",
+        max_buffer_usage, limit);
+    for (i = 0; i < limit; i++) {
+      buf = g_array_index (sink->bufqueue, GstBuffer *, i);
+      if (is_sync_frame (sink, buf)) {
+        /* found a sync frame, now extend the buffer usage to
+         * include at least this frame. */
+        max_buffer_usage = MAX (max_buffer_usage, i);
+        break;
+      }
+    }
+    GST_LOG_OBJECT (sink, "max buffer usage is now %d", max_buffer_usage);
+  }
+
+  GST_LOG_OBJECT (sink, "len %d, usage %d", queuelen, max_buffer_usage);
+
+  /* nobody is referencing units after max_buffer_usage so we can
+   * remove them from the queue. We remove them in reverse order as
+   * this is the most optimal for GArray. */
+  for (i = queuelen - 1; i > max_buffer_usage; i--) {
+    GstBuffer *old;
+
+    /* queue exceeded max size */
+    queuelen--;
+    old = g_array_index (sink->bufqueue, GstBuffer *, i);
+    sink->bufqueue = g_array_remove_index (sink->bufqueue, i);
+
+    /* unref tail buffer */
+    gst_buffer_unref (old);
+  }
+  /* save for stats */
+  sink->buffers_queued = max_buffer_usage;
+  CLIENTS_UNLOCK (sink);
+}
+
+/* Handle the clients. This is called when a socket becomes ready
+ * to read or writable. Badly behaving clients are put on a
+ * garbage list and removed.
+ */
+static gboolean
+gst_multi_handle_sink_socket_condition (GSocket * socket,
+    GIOCondition condition, GstMultiHandleSink * sink)
+{
+  GList *clink;
+  GstSocketClient *client;
+  gboolean ret = TRUE;
+
+  CLIENTS_LOCK (sink);
+  clink = g_hash_table_lookup (sink->socket_hash, socket);
+  if (clink == NULL) {
+    ret = FALSE;
+    goto done;
+  }
+
+  client = clink->data;
+
+  if (client->status != GST_CLIENT_STATUS_FLUSHING
+      && client->status != GST_CLIENT_STATUS_OK) {
+    gst_multi_handle_sink_remove_client_link (sink, clink);
+    ret = FALSE;
+    goto done;
+  }
+
+  if ((condition & G_IO_ERR)) {
+    GST_WARNING_OBJECT (sink, "Socket %p has error", client->socket);
+    client->status = GST_CLIENT_STATUS_ERROR;
+    gst_multi_handle_sink_remove_client_link (sink, clink);
+    ret = FALSE;
+    goto done;
+  } else if ((condition & G_IO_HUP)) {
+    client->status = GST_CLIENT_STATUS_CLOSED;
+    gst_multi_handle_sink_remove_client_link (sink, clink);
+    ret = FALSE;
+    goto done;
+  } else if ((condition & G_IO_IN) || (condition & G_IO_PRI)) {
+    /* handle client read */
+    if (!gst_multi_handle_sink_handle_client_read (sink, client)) {
+      gst_multi_handle_sink_remove_client_link (sink, clink);
+      ret = FALSE;
+      goto done;
+    }
+  } else if ((condition & G_IO_OUT)) {
+    /* handle client write */
+    if (!gst_multi_handle_sink_handle_client_write (sink, client)) {
+      gst_multi_handle_sink_remove_client_link (sink, clink);
+      ret = FALSE;
+      goto done;
+    }
+  }
+
+done:
+  CLIENTS_UNLOCK (sink);
+
+  return ret;
+}
+
+static gboolean
+gst_multi_handle_sink_timeout (GstMultiHandleSink * sink)
+{
+  GstClockTime now;
+  GTimeVal nowtv;
+  GList *clients;
+
+  g_get_current_time (&nowtv);
+  now = GST_TIMEVAL_TO_TIME (nowtv);
+
+  CLIENTS_LOCK (sink);
+  for (clients = sink->clients; clients; clients = clients->next) {
+    GstSocketClient *client;
+
+    client = clients->data;
+    if (sink->timeout > 0 && now - client->last_activity_time > sink->timeout) {
+      client->status = GST_CLIENT_STATUS_SLOW;
+      gst_multi_handle_sink_remove_client_link (sink, clients);
+    }
+  }
+  CLIENTS_UNLOCK (sink);
+
+  return FALSE;
+}
+
+/* we handle the client communication in another thread so that we do not block
+ * the gstreamer thread while we select() on the client fds */
+static gpointer
+gst_multi_handle_sink_thread (GstMultiHandleSink * sink)
+{
+  GSource *timeout = NULL;
+
+  while (sink->running) {
+    if (sink->timeout > 0) {
+      timeout = g_timeout_source_new (sink->timeout / GST_MSECOND);
+
+      g_source_set_callback (timeout,
+          (GSourceFunc) gst_multi_handle_sink_timeout, gst_object_ref (sink),
+          (GDestroyNotify) gst_object_unref);
+      g_source_attach (timeout, sink->main_context);
+    }
+
+    /* Returns after handling all pending events or when
+     * _wakeup() was called. In any case we have to add
+     * a new timeout because something happened.
+     */
+    g_main_context_iteration (sink->main_context, TRUE);
+
+    if (timeout) {
+      g_source_destroy (timeout);
+      g_source_unref (timeout);
+    }
+  }
+
+  return NULL;
+}
+
+static GstFlowReturn
+gst_multi_handle_sink_render (GstBaseSink * bsink, GstBuffer * buf)
+{
+  GstMultiHandleSink *sink;
+  gboolean in_caps;
+#if 0
+  GstCaps *bufcaps, *padcaps;
+#endif
+
+  sink = GST_MULTI_HANDLE_SINK (bsink);
+
+  g_return_val_if_fail (GST_OBJECT_FLAG_IS_SET (sink,
+          GST_MULTI_HANDLE_SINK_OPEN), GST_FLOW_WRONG_STATE);
+
+#if 0
+  /* since we check every buffer for streamheader caps, we need to make
+   * sure every buffer has caps set */
+  bufcaps = gst_buffer_get_caps (buf);
+  padcaps = GST_PAD_CAPS (GST_BASE_SINK_PAD (bsink));
+
+  /* make sure we have caps on the pad */
+  if (!padcaps && !bufcaps)
+    goto no_caps;
+#endif
+
+  /* get IN_CAPS first, code below might mess with the flags */
+  in_caps = GST_BUFFER_FLAG_IS_SET (buf, GST_BUFFER_FLAG_IN_CAPS);
+
+#if 0
+  /* stamp the buffer with previous caps if no caps set */
+  if (!bufcaps) {
+    if (!gst_buffer_is_writable (buf)) {
+      /* metadata is not writable, copy will be made and original buffer
+       * will be unreffed so we need to ref so that we don't lose the
+       * buffer in the render method. */
+      gst_buffer_ref (buf);
+      /* the new buffer is ours only, we keep it out of the scope of this
+       * function */
+      buf = gst_buffer_make_writable (buf);
+    } else {
+      /* else the metadata is writable, we ref because we keep the buffer
+       * out of the scope of this method */
+      gst_buffer_ref (buf);
+    }
+    /* buffer metadata is writable now, set the caps */
+    gst_buffer_set_caps (buf, padcaps);
+  } else {
+    gst_caps_unref (bufcaps);
+
+    /* since we keep this buffer out of the scope of this method */
+    gst_buffer_ref (buf);
+  }
+#endif
+
+  GST_LOG_OBJECT (sink, "received buffer %p, in_caps: %s, offset %"
+      G_GINT64_FORMAT ", offset_end %" G_GINT64_FORMAT
+      ", timestamp %" GST_TIME_FORMAT ", duration %" GST_TIME_FORMAT,
+      buf, in_caps ? "yes" : "no", GST_BUFFER_OFFSET (buf),
+      GST_BUFFER_OFFSET_END (buf),
+      GST_TIME_ARGS (GST_BUFFER_TIMESTAMP (buf)),
+      GST_TIME_ARGS (GST_BUFFER_DURATION (buf)));
+
+  /* if we get IN_CAPS buffers, but the previous buffer was not IN_CAPS,
+   * it means we're getting new streamheader buffers, and we should clear
+   * the old ones */
+  if (in_caps && sink->previous_buffer_in_caps == FALSE) {
+    GST_DEBUG_OBJECT (sink,
+        "receiving new IN_CAPS buffers, clearing old streamheader");
+    g_slist_foreach (sink->streamheader, (GFunc) gst_mini_object_unref, NULL);
+    g_slist_free (sink->streamheader);
+    sink->streamheader = NULL;
+  }
+
+  /* save the current in_caps */
+  sink->previous_buffer_in_caps = in_caps;
+
+  /* if the incoming buffer is marked as IN CAPS, then we assume for now
+   * it's a streamheader that needs to be sent to each new client, so we
+   * put it on our internal list of streamheader buffers.
+   * FIXME: we could check if the buffer's contents are in fact part of the
+   * current streamheader.
+   *
+   * We don't send the buffer to the client, since streamheaders are sent
+   * separately when necessary. */
+  if (in_caps) {
+    GST_DEBUG_OBJECT (sink, "appending IN_CAPS buffer with length %"
+        G_GSIZE_FORMAT " to streamheader", gst_buffer_get_size (buf));
+    sink->streamheader = g_slist_append (sink->streamheader, buf);
+  } else {
+    /* queue the buffer, this is a regular data buffer. */
+    gst_multi_handle_sink_queue_buffer (sink, buf);
+
+    sink->bytes_to_serve += gst_buffer_get_size (buf);
+  }
+  return GST_FLOW_OK;
+
+  /* ERRORS */
+#if 0
+no_caps:
+  {
+    GST_ELEMENT_ERROR (sink, CORE, NEGOTIATION, (NULL),
+        ("Received first buffer without caps set"));
+    return GST_FLOW_NOT_NEGOTIATED;
+  }
+#endif
+}
+
+static void
+gst_multi_handle_sink_set_property (GObject * object, guint prop_id,
+    const GValue * value, GParamSpec * pspec)
+{
+  GstMultiHandleSink *multisocketsink;
+
+  multisocketsink = GST_MULTI_HANDLE_SINK (object);
+
+  switch (prop_id) {
+    case PROP_BUFFERS_MAX:
+      multisocketsink->units_max = g_value_get_int (value);
+      break;
+    case PROP_BUFFERS_SOFT_MAX:
+      multisocketsink->units_soft_max = g_value_get_int (value);
+      break;
+    case PROP_TIME_MIN:
+      multisocketsink->time_min = g_value_get_int64 (value);
+      break;
+    case PROP_BYTES_MIN:
+      multisocketsink->bytes_min = g_value_get_int (value);
+      break;
+    case PROP_BUFFERS_MIN:
+      multisocketsink->buffers_min = g_value_get_int (value);
+      break;
+    case PROP_UNIT_TYPE:
+      multisocketsink->unit_type = g_value_get_enum (value);
+      break;
+    case PROP_UNITS_MAX:
+      multisocketsink->units_max = g_value_get_int64 (value);
+      break;
+    case PROP_UNITS_SOFT_MAX:
+      multisocketsink->units_soft_max = g_value_get_int64 (value);
+      break;
+    case PROP_RECOVER_POLICY:
+      multisocketsink->recover_policy = g_value_get_enum (value);
+      break;
+    case PROP_TIMEOUT:
+      multisocketsink->timeout = g_value_get_uint64 (value);
+      break;
+    case PROP_SYNC_METHOD:
+      multisocketsink->def_sync_method = g_value_get_enum (value);
+      break;
+    case PROP_BURST_FORMAT:
+      multisocketsink->def_burst_format = g_value_get_enum (value);
+      break;
+    case PROP_BURST_VALUE:
+      multisocketsink->def_burst_value = g_value_get_uint64 (value);
+      break;
+    case PROP_QOS_DSCP:
+      multisocketsink->qos_dscp = g_value_get_int (value);
+      setup_dscp (multisocketsink);
+      break;
+    case PROP_HANDLE_READ:
+      multisocketsink->handle_read = g_value_get_boolean (value);
+      break;
+    case PROP_RESEND_STREAMHEADER:
+      multisocketsink->resend_streamheader = g_value_get_boolean (value);
+      break;
+
+    default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
+      break;
+  }
+}
+
+static void
+gst_multi_handle_sink_get_property (GObject * object, guint prop_id,
+    GValue * value, GParamSpec * pspec)
+{
+  GstMultiHandleSink *multisocketsink;
+
+  multisocketsink = GST_MULTI_HANDLE_SINK (object);
+
+  switch (prop_id) {
+    case PROP_BUFFERS_MAX:
+      g_value_set_int (value, multisocketsink->units_max);
+      break;
+    case PROP_BUFFERS_SOFT_MAX:
+      g_value_set_int (value, multisocketsink->units_soft_max);
+      break;
+    case PROP_TIME_MIN:
+      g_value_set_int64 (value, multisocketsink->time_min);
+      break;
+    case PROP_BYTES_MIN:
+      g_value_set_int (value, multisocketsink->bytes_min);
+      break;
+    case PROP_BUFFERS_MIN:
+      g_value_set_int (value, multisocketsink->buffers_min);
+      break;
+    case PROP_BUFFERS_QUEUED:
+      g_value_set_uint (value, multisocketsink->buffers_queued);
+      break;
+    case PROP_BYTES_QUEUED:
+      g_value_set_uint (value, multisocketsink->bytes_queued);
+      break;
+    case PROP_TIME_QUEUED:
+      g_value_set_uint64 (value, multisocketsink->time_queued);
+      break;
+    case PROP_UNIT_TYPE:
+      g_value_set_enum (value, multisocketsink->unit_type);
+      break;
+    case PROP_UNITS_MAX:
+      g_value_set_int64 (value, multisocketsink->units_max);
+      break;
+    case PROP_UNITS_SOFT_MAX:
+      g_value_set_int64 (value, multisocketsink->units_soft_max);
+      break;
+    case PROP_RECOVER_POLICY:
+      g_value_set_enum (value, multisocketsink->recover_policy);
+      break;
+    case PROP_TIMEOUT:
+      g_value_set_uint64 (value, multisocketsink->timeout);
+      break;
+    case PROP_SYNC_METHOD:
+      g_value_set_enum (value, multisocketsink->def_sync_method);
+      break;
+    case PROP_BYTES_TO_SERVE:
+      g_value_set_uint64 (value, multisocketsink->bytes_to_serve);
+      break;
+    case PROP_BYTES_SERVED:
+      g_value_set_uint64 (value, multisocketsink->bytes_served);
+      break;
+    case PROP_BURST_FORMAT:
+      g_value_set_enum (value, multisocketsink->def_burst_format);
+      break;
+    case PROP_BURST_VALUE:
+      g_value_set_uint64 (value, multisocketsink->def_burst_value);
+      break;
+    case PROP_QOS_DSCP:
+      g_value_set_int (value, multisocketsink->qos_dscp);
+      break;
+    case PROP_HANDLE_READ:
+      g_value_set_boolean (value, multisocketsink->handle_read);
+      break;
+    case PROP_RESEND_STREAMHEADER:
+      g_value_set_boolean (value, multisocketsink->resend_streamheader);
+      break;
+    case PROP_NUM_SOCKETS:
+      g_value_set_uint (value,
+          g_hash_table_size (multisocketsink->socket_hash));
+      break;
+
+    default:
+      G_OBJECT_WARN_INVALID_PROPERTY_ID (object, prop_id, pspec);
+      break;
+  }
+}
+
+
+/* create a socket for sending to remote machine */
+static gboolean
+gst_multi_handle_sink_start (GstBaseSink * bsink)
+{
+  GstMultiHandleSinkClass *fclass;
+  GstMultiHandleSink *this;
+  GList *clients;
+
+  if (GST_OBJECT_FLAG_IS_SET (bsink, GST_MULTI_HANDLE_SINK_OPEN))
+    return TRUE;
+
+  this = GST_MULTI_HANDLE_SINK (bsink);
+  fclass = GST_MULTI_HANDLE_SINK_GET_CLASS (this);
+
+  GST_INFO_OBJECT (this, "starting");
+
+  this->main_context = g_main_context_new ();
+
+  CLIENTS_LOCK (this);
+  for (clients = this->clients; clients; clients = clients->next) {
+    GstSocketClient *client;
+
+    client = clients->data;
+    if (client->source)
+      continue;
+    client->source =
+        g_socket_create_source (client->socket,
+        G_IO_IN | G_IO_OUT | G_IO_PRI | G_IO_ERR | G_IO_HUP, this->cancellable);
+    g_source_set_callback (client->source,
+        (GSourceFunc) gst_multi_handle_sink_socket_condition,
+        gst_object_ref (this), (GDestroyNotify) gst_object_unref);
+    g_source_attach (client->source, this->main_context);
+  }
+  CLIENTS_UNLOCK (this);
+
+  this->streamheader = NULL;
+  this->bytes_to_serve = 0;
+  this->bytes_served = 0;
+
+  if (fclass->init) {
+    fclass->init (this);
+  }
+
+  this->running = TRUE;
+
+  this->thread = g_thread_new ("multisocketsink",
+      (GThreadFunc) gst_multi_handle_sink_thread, this);
+
+  GST_OBJECT_FLAG_SET (this, GST_MULTI_HANDLE_SINK_OPEN);
+
+  return TRUE;
+}
+
+static gboolean
+multisocketsink_hash_remove (gpointer key, gpointer value, gpointer data)
+{
+  return TRUE;
+}
+
+static gboolean
+gst_multi_handle_sink_stop (GstBaseSink * bsink)
+{
+  GstMultiHandleSinkClass *fclass;
+  GstMultiHandleSink *this;
+  GstBuffer *buf;
+  gint i;
+
+  this = GST_MULTI_HANDLE_SINK (bsink);
+  fclass = GST_MULTI_HANDLE_SINK_GET_CLASS (this);
+
+  if (!GST_OBJECT_FLAG_IS_SET (bsink, GST_MULTI_HANDLE_SINK_OPEN))
+    return TRUE;
+
+  this->running = FALSE;
+
+  if (this->main_context)
+    g_main_context_wakeup (this->main_context);
+
+  if (this->thread) {
+    GST_DEBUG_OBJECT (this, "joining thread");
+    g_thread_join (this->thread);
+    GST_DEBUG_OBJECT (this, "joined thread");
+    this->thread = NULL;
+  }
+
+  /* free the clients */
+  gst_multi_handle_sink_clear (this);
+
+  if (this->streamheader) {
+    g_slist_foreach (this->streamheader, (GFunc) gst_mini_object_unref, NULL);
+    g_slist_free (this->streamheader);
+    this->streamheader = NULL;
+  }
+
+  if (fclass->close)
+    fclass->close (this);
+
+  if (this->main_context) {
+    g_main_context_unref (this->main_context);
+    this->main_context = NULL;
+  }
+
+  g_hash_table_foreach_remove (this->socket_hash, multisocketsink_hash_remove,
+      this);
+
+  /* remove all queued buffers */
+  if (this->bufqueue) {
+    GST_DEBUG_OBJECT (this, "Emptying bufqueue with %d buffers",
+        this->bufqueue->len);
+    for (i = this->bufqueue->len - 1; i >= 0; --i) {
+      buf = g_array_index (this->bufqueue, GstBuffer *, i);
+      GST_LOG_OBJECT (this, "Removing buffer %p (%d) with refcount %d", buf, i,
+          GST_MINI_OBJECT_REFCOUNT (buf));
+      gst_buffer_unref (buf);
+      this->bufqueue = g_array_remove_index (this->bufqueue, i);
+    }
+    /* freeing the array is done in _finalize */
+  }
+  GST_OBJECT_FLAG_UNSET (this, GST_MULTI_HANDLE_SINK_OPEN);
+
+  return TRUE;
+}
+
+static GstStateChangeReturn
+gst_multi_handle_sink_change_state (GstElement * element,
+    GstStateChange transition)
+{
+  GstMultiHandleSink *sink;
+  GstStateChangeReturn ret;
+
+  sink = GST_MULTI_HANDLE_SINK (element);
+
+  /* we disallow changing the state from the streaming thread */
+  if (g_thread_self () == sink->thread)
+    return GST_STATE_CHANGE_FAILURE;
+
+
+  switch (transition) {
+    case GST_STATE_CHANGE_NULL_TO_READY:
+      if (!gst_multi_handle_sink_start (GST_BASE_SINK (sink)))
+        goto start_failed;
+      break;
+    case GST_STATE_CHANGE_READY_TO_PAUSED:
+      break;
+    case GST_STATE_CHANGE_PAUSED_TO_PLAYING:
+      break;
+    default:
+      break;
+  }
+
+  ret = GST_ELEMENT_CLASS (parent_class)->change_state (element, transition);
+
+  switch (transition) {
+    case GST_STATE_CHANGE_PLAYING_TO_PAUSED:
+      break;
+    case GST_STATE_CHANGE_PAUSED_TO_READY:
+      break;
+    case GST_STATE_CHANGE_READY_TO_NULL:
+      gst_multi_handle_sink_stop (GST_BASE_SINK (sink));
+      break;
+    default:
+      break;
+  }
+  return ret;
+
+  /* ERRORS */
+start_failed:
+  {
+    /* error message was posted */
+    return GST_STATE_CHANGE_FAILURE;
+  }
+}
+
+static gboolean
+gst_multi_handle_sink_unlock (GstBaseSink * bsink)
+{
+  GstMultiHandleSink *sink;
+
+  sink = GST_MULTI_HANDLE_SINK (bsink);
+
+  GST_DEBUG_OBJECT (sink, "set to flushing");
+  g_cancellable_cancel (sink->cancellable);
+  if (sink->main_context)
+    g_main_context_wakeup (sink->main_context);
+
+  return TRUE;
+}
+
+/* will be called only between calls to start() and stop() */
+static gboolean
+gst_multi_handle_sink_unlock_stop (GstBaseSink * bsink)
+{
+  GstMultiHandleSink *sink;
+
+  sink = GST_MULTI_HANDLE_SINK (bsink);
+
+  GST_DEBUG_OBJECT (sink, "unset flushing");
+  g_cancellable_reset (sink->cancellable);
+
+  return TRUE;
+}
diff --git a/gst/tcp/gstmultihandlesink.h b/gst/tcp/gstmultihandlesink.h
new file mode 100644 (file)
index 0000000..b5d917c
--- /dev/null
@@ -0,0 +1,273 @@
+/* GStreamer
+ * Copyright (C) <1999> Erik Walthinsen <omega@cse.ogi.edu>
+ * Copyright (C) <2004> Thomas Vander Stichele <thomas at apestaart dot org>
+ * Copyright (C) <2011> Collabora Ltd.
+ *     Author: Sebastian Dröge <sebastian.droege@collabora.co.uk>
+ *
+ * This library is free software; you can redistribute it and/or
+ * modify it under the terms of the GNU Library General Public
+ * License as published by the Free Software Foundation; either
+ * version 2 of the License, or (at your option) any later version.
+ *
+ * This library is distributed in the hope that it will be useful,
+ * but WITHOUT ANY WARRANTY; without even the implied warranty of
+ * MERCHANTABILITY or FITNESS FOR A PARTICULAR PURPOSE.  See the GNU
+ * Library General Public License for more details.
+ *
+ * You should have received a copy of the GNU Library General Public
+ * License along with this library; if not, write to the
+ * Free Software Foundation, Inc., 59 Temple Place - Suite 330,
+ * Boston, MA 02111-1307, USA.
+ */
+
+
+#ifndef __GST_MULTI_HANDLE_SINK_H__
+#define __GST_MULTI_HANDLE_SINK_H__
+
+#include <gst/gst.h>
+#include <gst/base/gstbasesink.h>
+#include <gio/gio.h>
+
+G_BEGIN_DECLS
+
+#define GST_TYPE_MULTI_HANDLE_SINK \
+  (gst_multi_handle_sink_get_type())
+#define GST_MULTI_HANDLE_SINK(obj) \
+  (G_TYPE_CHECK_INSTANCE_CAST((obj),GST_TYPE_MULTI_HANDLE_SINK,GstMultiHandleSink))
+#define GST_MULTI_HANDLE_SINK_CLASS(klass) \
+  (G_TYPE_CHECK_CLASS_CAST((klass),GST_TYPE_MULTI_HANDLE_SINK,GstMultiHandleSinkClass))
+#define GST_IS_MULTI_HANDLE_SINK(obj) \
+  (G_TYPE_CHECK_INSTANCE_TYPE((obj),GST_TYPE_MULTI_HANDLE_SINK))
+#define GST_IS_MULTI_HANDLE_SINK_CLASS(klass) \
+  (G_TYPE_CHECK_CLASS_TYPE((klass),GST_TYPE_MULTI_HANDLE_SINK))
+#define GST_MULTI_HANDLE_SINK_GET_CLASS(klass) \
+  (G_TYPE_INSTANCE_GET_CLASS ((klass), GST_TYPE_MULTI_HANDLE_SINK, GstMultiHandleSinkClass))
+
+
+typedef struct _GstMultiHandleSink GstMultiHandleSink;
+typedef struct _GstMultiHandleSinkClass GstMultiHandleSinkClass;
+
+#if 0
+typedef enum {
+  GST_MULTI_HANDLE_SINK_OPEN             = (GST_ELEMENT_FLAG_LAST << 0),
+
+  GST_MULTI_HANDLE_SINK_FLAG_LAST        = (GST_ELEMENT_FLAG_LAST << 2)
+} GstMultiHandleSinkFlags;
+#endif
+
+/**
+ * GstRecoverPolicy:
+ * @GST_RECOVER_POLICY_NONE             : no recovering is done
+ * @GST_RECOVER_POLICY_RESYNC_LATEST    : client is moved to last buffer
+ * @GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT: client is moved to the soft limit
+ * @GST_RECOVER_POLICY_RESYNC_KEYFRAME  : client is moved to latest keyframe
+ *
+ * Possible values for the recovery procedure to use when a client consumes
+ * data too slow and has a backlag of more that soft-limit buffers.
+ */
+typedef enum
+{
+  GST_RECOVER_POLICY_NONE,
+  GST_RECOVER_POLICY_RESYNC_LATEST,
+  GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT,
+  GST_RECOVER_POLICY_RESYNC_KEYFRAME
+} GstRecoverPolicy;
+
+/**
+ * GstSyncMethod:
+ * @GST_SYNC_METHOD_LATEST              : client receives most recent buffer
+ * @GST_SYNC_METHOD_NEXT_KEYFRAME       : client receives next keyframe
+ * @GST_SYNC_METHOD_LATEST_KEYFRAME     : client receives latest keyframe (burst)
+ * @GST_SYNC_METHOD_BURST               : client receives specific amount of data
+ * @GST_SYNC_METHOD_BURST_KEYFRAME      : client receives specific amount of data 
+ *                                        starting from latest keyframe
+ * @GST_SYNC_METHOD_BURST_WITH_KEYFRAME : client receives specific amount of data from
+ *                                        a keyframe, or if there is not enough data after
+ *                                        the keyframe, starting before the keyframe
+ *
+ * This enum defines the selection of the first buffer that is sent
+ * to a new client.
+ */
+typedef enum
+{
+  GST_SYNC_METHOD_LATEST,
+  GST_SYNC_METHOD_NEXT_KEYFRAME,
+  GST_SYNC_METHOD_LATEST_KEYFRAME,
+  GST_SYNC_METHOD_BURST,
+  GST_SYNC_METHOD_BURST_KEYFRAME,
+  GST_SYNC_METHOD_BURST_WITH_KEYFRAME
+} GstSyncMethod;
+
+/**
+ * GstClientStatus:
+ * @GST_CLIENT_STATUS_OK       : client is ok
+ * @GST_CLIENT_STATUS_CLOSED   : client closed the socket
+ * @GST_CLIENT_STATUS_REMOVED  : client is removed
+ * @GST_CLIENT_STATUS_SLOW     : client is too slow
+ * @GST_CLIENT_STATUS_ERROR    : client is in error
+ * @GST_CLIENT_STATUS_DUPLICATE: same client added twice
+ * @GST_CLIENT_STATUS_FLUSHING : client is flushing out the remaining buffers.
+ *
+ * This specifies the reason why a client was removed from
+ * multisocketsink and is received in the "client-removed" signal.
+ */
+typedef enum
+{
+  GST_CLIENT_STATUS_OK          = 0,
+  GST_CLIENT_STATUS_CLOSED      = 1,
+  GST_CLIENT_STATUS_REMOVED     = 2,
+  GST_CLIENT_STATUS_SLOW        = 3,
+  GST_CLIENT_STATUS_ERROR       = 4,
+  GST_CLIENT_STATUS_DUPLICATE   = 5,
+  GST_CLIENT_STATUS_FLUSHING    = 6
+} GstClientStatus;
+
+#if 0
+/* structure for a client
+ */
+typedef struct {
+  GSocket *socket;
+  GSource *source;
+
+  gint bufpos;                  /* position of this client in the global queue */
+  gint flushcount;              /* the remaining number of buffers to flush out or -1 if the 
+                                   client is not flushing. */
+
+  GstClientStatus status;
+
+  GSList *sending;              /* the buffers we need to send */
+  gint bufoffset;               /* offset in the first buffer */
+
+  gboolean discont;
+
+  gboolean new_connection;
+
+  gboolean currently_removing;
+
+  /* method to sync client when connecting */
+  GstSyncMethod sync_method;
+  GstFormat     burst_min_format;
+  guint64       burst_min_value;
+  GstFormat     burst_max_format;
+  guint64       burst_max_value;
+
+  GstCaps *caps;                /* caps of last queued buffer */
+
+  /* stats */
+  guint64 bytes_sent;
+  guint64 connect_time;
+  guint64 disconnect_time;
+  guint64 last_activity_time;
+  guint64 dropped_buffers;
+  guint64 avg_queue_size;
+  guint64 first_buffer_ts;
+  guint64 last_buffer_ts;
+} GstSocketClient;
+#endif
+
+#define CLIENTS_LOCK_INIT(socketsink)       (g_rec_mutex_init(&socketsink->clientslock))
+#define CLIENTS_LOCK_CLEAR(socketsink)      (g_rec_mutex_clear(&socketsink->clientslock))
+#define CLIENTS_LOCK(socketsink)            (g_rec_mutex_lock(&socketsink->clientslock))
+#define CLIENTS_UNLOCK(socketsink)          (g_rec_mutex_unlock(&socketsink->clientslock))
+
+/**
+ * GstMultiHandleSink:
+ *
+ * The multisocketsink object structure.
+ */
+struct _GstMultiHandleSink {
+  GstBaseSink element;
+
+  /*< private >*/
+  guint64 bytes_to_serve; /* how much bytes we must serve */
+  guint64 bytes_served; /* how much bytes have we served */
+
+  GRecMutex clientslock;  /* lock to protect the clients list */
+  GList *clients;       /* list of clients we are serving */
+  GHashTable *socket_hash;  /* index on socket to client */
+  guint clients_cookie; /* Cookie to detect changes to the clients list */
+
+  GMainContext *main_context;
+  GCancellable *cancellable;
+
+  GSList *streamheader; /* GSList of GstBuffers to use as streamheader */
+  gboolean previous_buffer_in_caps;
+
+  guint mtu;
+  gint qos_dscp;
+  gboolean handle_read;
+
+  GArray *bufqueue;     /* global queue of buffers */
+
+  gboolean running;     /* the thread state */
+  GThread *thread;      /* the sender thread */
+
+  /* these values are used to check if a client is reading fast
+   * enough and to control receovery */
+  GstFormat unit_type;/* the format of the units */
+  gint64 units_max;       /* max units to queue for a client */
+  gint64 units_soft_max;  /* max units a client can lag before recovery starts */
+  GstRecoverPolicy recover_policy;
+  GstClockTime timeout; /* max amount of nanoseconds to remain idle */
+
+  GstSyncMethod def_sync_method;    /* what method to use for connecting clients */
+  GstFormat     def_burst_format;
+  guint64       def_burst_value;
+
+  /* these values are used to control the amount of data
+   * kept in the queues. It allows clients to perform a burst
+   * on connect. */
+  gint   bytes_min;    /* min number of bytes to queue */
+  gint64 time_min;     /* min time to queue */
+  gint   buffers_min;   /* min number of buffers to queue */
+
+  gboolean resend_streamheader; /* resend streamheader if it changes */
+
+  /* stats */
+  gint buffers_queued;  /* number of queued buffers */
+  gint bytes_queued;    /* number of queued bytes */
+  gint time_queued;     /* number of queued time */
+
+  guint8 header_flags;
+};
+
+struct _GstMultiHandleSinkClass {
+  GstBaseSinkClass parent_class;
+
+  /* element methods */
+  void          (*add)          (GstMultiHandleSink *sink, GSocket *socket);
+  void          (*add_full)     (GstMultiHandleSink *sink, GSocket *socket, GstSyncMethod sync,
+                                GstFormat format, guint64 value, 
+                                GstFormat max_format, guint64 max_value);
+  void          (*remove)       (GstMultiHandleSink *sink, GSocket *socket);
+  void          (*remove_flush) (GstMultiHandleSink *sink, GSocket *socket);
+  void          (*clear)        (GstMultiHandleSink *sink);
+  GstStructure* (*get_stats)    (GstMultiHandleSink *sink, GSocket *socket);
+
+  /* vtable */
+  gboolean (*init)   (GstMultiHandleSink *sink);
+  gboolean (*close)  (GstMultiHandleSink *sink);
+  void (*removed) (GstMultiHandleSink *sink, GSocket *socket);
+
+  /* signals */
+  void (*client_added) (GstElement *element, GSocket *socket);
+  void (*client_removed) (GstElement *element, GSocket *socket, GstClientStatus status);
+  void (*client_socket_removed) (GstElement *element, GSocket *socket);
+};
+
+GType gst_multi_handle_sink_get_type (void);
+
+#if 0
+void          gst_multi_handle_sink_add          (GstMultiHandleSink *sink, GSocket *socket);
+void          gst_multi_handle_sink_add_full     (GstMultiHandleSink *sink, GSocket *socket, GstSyncMethod sync, 
+                                              GstFormat min_format, guint64 min_value,
+                                              GstFormat max_format, guint64 max_value);
+void          gst_multi_handle_sink_remove       (GstMultiHandleSink *sink, GSocket *socket);
+void          gst_multi_handle_sink_remove_flush (GstMultiHandleSink *sink, GSocket *socket);
+void          gst_multi_handle_sink_clear        (GstMultiHandleSink *sink);
+GstStructure*  gst_multi_handle_sink_get_stats    (GstMultiHandleSink *sink, GSocket *socket);
+
+G_END_DECLS
+#endif
+
+#endif /* __GST_MULTI_HANDLE_SINK_H__ */
index 746777a..a447862 100644 (file)
 #ifndef __GST_MULTI_SOCKET_SINK_H__
 #define __GST_MULTI_SOCKET_SINK_H__
 
+#include <gio/gio.h>
+
 #include <gst/gst.h>
 #include <gst/base/gstbasesink.h>
-#include <gio/gio.h>
+
+#include "gstmultihandlesink.h"
 
 G_BEGIN_DECLS
 
@@ -53,73 +56,6 @@ typedef enum {
   GST_MULTI_SOCKET_SINK_FLAG_LAST        = (GST_ELEMENT_FLAG_LAST << 2)
 } GstMultiSocketSinkFlags;
 
-/**
- * GstRecoverPolicy:
- * @GST_RECOVER_POLICY_NONE             : no recovering is done
- * @GST_RECOVER_POLICY_RESYNC_LATEST    : client is moved to last buffer
- * @GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT: client is moved to the soft limit
- * @GST_RECOVER_POLICY_RESYNC_KEYFRAME  : client is moved to latest keyframe
- *
- * Possible values for the recovery procedure to use when a client consumes
- * data too slow and has a backlag of more that soft-limit buffers.
- */
-typedef enum
-{
-  GST_RECOVER_POLICY_NONE,
-  GST_RECOVER_POLICY_RESYNC_LATEST,
-  GST_RECOVER_POLICY_RESYNC_SOFT_LIMIT,
-  GST_RECOVER_POLICY_RESYNC_KEYFRAME
-} GstRecoverPolicy;
-
-/**
- * GstSyncMethod:
- * @GST_SYNC_METHOD_LATEST              : client receives most recent buffer
- * @GST_SYNC_METHOD_NEXT_KEYFRAME       : client receives next keyframe
- * @GST_SYNC_METHOD_LATEST_KEYFRAME     : client receives latest keyframe (burst)
- * @GST_SYNC_METHOD_BURST               : client receives specific amount of data
- * @GST_SYNC_METHOD_BURST_KEYFRAME      : client receives specific amount of data 
- *                                        starting from latest keyframe
- * @GST_SYNC_METHOD_BURST_WITH_KEYFRAME : client receives specific amount of data from
- *                                        a keyframe, or if there is not enough data after
- *                                        the keyframe, starting before the keyframe
- *
- * This enum defines the selection of the first buffer that is sent
- * to a new client.
- */
-typedef enum
-{
-  GST_SYNC_METHOD_LATEST,
-  GST_SYNC_METHOD_NEXT_KEYFRAME,
-  GST_SYNC_METHOD_LATEST_KEYFRAME,
-  GST_SYNC_METHOD_BURST,
-  GST_SYNC_METHOD_BURST_KEYFRAME,
-  GST_SYNC_METHOD_BURST_WITH_KEYFRAME
-} GstSyncMethod;
-
-/**
- * GstClientStatus:
- * @GST_CLIENT_STATUS_OK       : client is ok
- * @GST_CLIENT_STATUS_CLOSED   : client closed the socket
- * @GST_CLIENT_STATUS_REMOVED  : client is removed
- * @GST_CLIENT_STATUS_SLOW     : client is too slow
- * @GST_CLIENT_STATUS_ERROR    : client is in error
- * @GST_CLIENT_STATUS_DUPLICATE: same client added twice
- * @GST_CLIENT_STATUS_FLUSHING : client is flushing out the remaining buffers.
- *
- * This specifies the reason why a client was removed from
- * multisocketsink and is received in the "client-removed" signal.
- */
-typedef enum
-{
-  GST_CLIENT_STATUS_OK          = 0,
-  GST_CLIENT_STATUS_CLOSED      = 1,
-  GST_CLIENT_STATUS_REMOVED     = 2,
-  GST_CLIENT_STATUS_SLOW        = 3,
-  GST_CLIENT_STATUS_ERROR       = 4,
-  GST_CLIENT_STATUS_DUPLICATE   = 5,
-  GST_CLIENT_STATUS_FLUSHING    = 6
-} GstClientStatus;
-
 /* structure for a client
  */
 typedef struct {
index 07ecb9e..1fc3831 100644 (file)
@@ -2,3 +2,7 @@ VOID:STRING,UINT
 VOID:OBJECT,ENUM
 VOID:OBJECT,ENUM,ENUM,UINT64,ENUM,UINT64
 BOXED:OBJECT
+VOID:INT,ENUM,INT,UINT64,INT,UINT64
+VOID:INT
+VOID:INT,ENUM
+BOXED:INT
index d5b2866..f2a5953 100644 (file)
@@ -25,6 +25,7 @@
 #include "gsttcpclientsink.h"
 #include "gsttcpserversrc.h"
 #include "gsttcpserversink.h"
+#include "gstmultifdsink.h"
 #include "gstmultisocketsink.h"
 
 GST_DEBUG_CATEGORY (tcp_debug);
@@ -44,6 +45,9 @@ plugin_init (GstPlugin * plugin)
   if (!gst_element_register (plugin, "tcpserversrc", GST_RANK_NONE,
           GST_TYPE_TCP_SERVER_SRC))
     return FALSE;
+  if (!gst_element_register (plugin, "multifdsink", GST_RANK_NONE,
+          GST_TYPE_MULTI_FD_SINK))
+    return FALSE;
   if (!gst_element_register (plugin, "multisocketsink", GST_RANK_NONE,
           GST_TYPE_MULTI_SOCKET_SINK))
     return FALSE;
index 6d5a252..ace3c49 100644 (file)
@@ -42,6 +42,7 @@ setup_multifdsink (void)
   GST_DEBUG ("setup_multifdsink");
   multifdsink = gst_check_setup_element ("multifdsink");
   mysrcpad = gst_check_setup_src_pad (multifdsink, &srctemplate);
+  GST_PAD_UNSET_FLUSHING (mysrcpad);
 
   return multifdsink;
 }
@@ -117,10 +118,11 @@ GST_START_TEST (test_add_client)
   g_signal_emit_by_name (sink, "add", pfd[1]);
 
   caps = gst_caps_from_string ("application/x-gst-check");
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
   GST_DEBUG ("Created test caps %p %" GST_PTR_FORMAT, caps, caps);
   buffer = gst_buffer_new_and_alloc (4);
   gst_pad_set_caps (mysrcpad, caps);
-  ASSERT_CAPS_REFCOUNT (caps, "caps", 2);
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 3);
   gst_buffer_fill (buffer, 0, "dead", 4);
   fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
 
@@ -247,9 +249,10 @@ GST_START_TEST (test_streamheader)
    * buffers */
   gst_multifdsink_create_streamheader ("babe", "deadbeef", &hbuf1, &hbuf2,
       &caps);
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
   fail_unless (gst_pad_set_caps (mysrcpad, caps));
-  /* one is ours, two on the buffers, and one now on the pad */
-  ASSERT_CAPS_REFCOUNT (caps, "caps", 4);
+  /* one is ours, two from set_caps */
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 3);
 
   fail_unless (gst_pad_push (mysrcpad, hbuf1) == GST_FLOW_OK);
   fail_unless (gst_pad_push (mysrcpad, hbuf2) == GST_FLOW_OK);
@@ -334,9 +337,10 @@ GST_START_TEST (test_change_streamheader)
    * buffers */
   gst_multifdsink_create_streamheader ("first", "header", &hbuf1, &hbuf2,
       &caps);
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 1);
   fail_unless (gst_pad_set_caps (mysrcpad, caps));
-  /* one is ours, two on the buffers, and one now on the pad */
-  ASSERT_CAPS_REFCOUNT (caps, "caps", 4);
+  /* one is ours, two from set_caps */
+  ASSERT_CAPS_REFCOUNT (caps, "caps", 3);
 
   /* one to hold for the test and one to give away */
   ASSERT_BUFFER_REFCOUNT (hbuf1, "hbuf1", 2);
@@ -427,11 +431,28 @@ GST_START_TEST (test_change_streamheader)
 
 GST_END_TEST;
 
+static GstBuffer *
+gst_new_buffer (int i)
+{
+  GstMapInfo info;
+  gchar *data;
+
+  GstBuffer *buffer = gst_buffer_new_and_alloc (16);
+
+  /* copy some id */
+  g_assert (gst_buffer_map (buffer, &info, GST_MAP_WRITE));
+  data = (gchar *) info.data;
+  g_snprintf (data, 16, "deadbee%08x", i);
+  gst_buffer_unmap (buffer, &info);
+
+  return buffer;
+}
+
+
 /* keep 100 bytes and burst 80 bytes to clients */
 GST_START_TEST (test_burst_client_bytes)
 {
   GstElement *sink;
-  GstBuffer *buffer;
   GstCaps *caps;
   int pfd1[2];
   int pfd2[2];
@@ -459,14 +480,7 @@ GST_START_TEST (test_burst_client_bytes)
 
   /* push buffers in, 9 * 16 bytes = 144 bytes */
   for (i = 0; i < 9; i++) {
-    gchar *data;
-
-    buffer = gst_buffer_new_and_alloc (16);
-
-    /* copy some id */
-    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
-    g_snprintf (data, 16, "deadbee%08x", i);
-    gst_buffer_unmap (buffer, data, 16);
+    GstBuffer *buffer = gst_new_buffer (i);
 
     fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
   }
@@ -484,14 +498,7 @@ GST_START_TEST (test_burst_client_bytes)
 
   /* push last buffer to make client fds ready for reading */
   for (i = 9; i < 10; i++) {
-    gchar *data;
-
-    buffer = gst_buffer_new_and_alloc (16);
-
-    /* copy some id */
-    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
-    g_snprintf (data, 16, "deadbee%08x", i);
-    gst_buffer_unmap (buffer, data, 16);
+    GstBuffer *buffer = gst_new_buffer (i);
 
     fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
   }
@@ -545,7 +552,6 @@ GST_END_TEST;
 GST_START_TEST (test_burst_client_bytes_keyframe)
 {
   GstElement *sink;
-  GstBuffer *buffer;
   GstCaps *caps;
   int pfd1[2];
   int pfd2[2];
@@ -573,19 +579,12 @@ GST_START_TEST (test_burst_client_bytes_keyframe)
 
   /* push buffers in, 9 * 16 bytes = 144 bytes */
   for (i = 0; i < 9; i++) {
-    gchar *data;
-
-    buffer = gst_buffer_new_and_alloc (16);
+    GstBuffer *buffer = gst_new_buffer (i);
 
     /* mark most buffers as delta */
     if (i != 0 && i != 4 && i != 8)
       GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
 
-    /* copy some id */
-    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
-    g_snprintf (data, 16, "deadbee%08x", i);
-    gst_buffer_unmap (buffer, data, 16);
-
     fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
   }
 
@@ -602,16 +601,10 @@ GST_START_TEST (test_burst_client_bytes_keyframe)
 
   /* push last buffer to make client fds ready for reading */
   for (i = 9; i < 10; i++) {
-    gchar *data;
+    GstBuffer *buffer = gst_new_buffer (i);
 
-    buffer = gst_buffer_new_and_alloc (16);
     GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
 
-    /* copy some id */
-    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
-    g_snprintf (data, 16, "deadbee%08x", i);
-    gst_buffer_unmap (buffer, data, 16);
-
     fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
   }
 
@@ -661,7 +654,6 @@ GST_END_TEST;
 GST_START_TEST (test_burst_client_bytes_with_keyframe)
 {
   GstElement *sink;
-  GstBuffer *buffer;
   GstCaps *caps;
   int pfd1[2];
   int pfd2[2];
@@ -689,19 +681,12 @@ GST_START_TEST (test_burst_client_bytes_with_keyframe)
 
   /* push buffers in, 9 * 16 bytes = 144 bytes */
   for (i = 0; i < 9; i++) {
-    gchar *data;
-
-    buffer = gst_buffer_new_and_alloc (16);
+    GstBuffer *buffer = gst_new_buffer (i);
 
     /* mark most buffers as delta */
     if (i != 0 && i != 4 && i != 8)
       GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
 
-    /* copy some id */
-    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
-    g_snprintf (data, 16, "deadbee%08x", i);
-    gst_buffer_unmap (buffer, data, 16);
-
     fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
   }
 
@@ -718,16 +703,10 @@ GST_START_TEST (test_burst_client_bytes_with_keyframe)
 
   /* push last buffer to make client fds ready for reading */
   for (i = 9; i < 10; i++) {
-    gchar *data;
+    GstBuffer *buffer = gst_new_buffer (i);
 
-    buffer = gst_buffer_new_and_alloc (16);
     GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);
 
-    /* copy some id */
-    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
-    g_snprintf (data, 16, "deadbee%08x", i);
-    gst_buffer_unmap (buffer, data, 16);
-
     fail_unless (gst_pad_push (mysrcpad, buffer) == GST_FLOW_OK);
   }
 
@@ -784,7 +763,6 @@ GST_END_TEST;
 GST_START_TEST (test_client_next_keyframe)
 {
   GstElement *sink;
-  GstBuffer *buffer;
   GstCaps *caps;
   int pfd1[2];
   gchar data[16];
@@ -806,14 +784,7 @@ GST_START_TEST (test_client_next_keyframe)
 
   /* push buffers in: keyframe, then non-keyframe */
   for (i = 0; i < 2; i++) {
-    gchar *data;
-
-    buffer = gst_buffer_new_and_alloc (16);
-
-    /* copy some id */
-    data = gst_buffer_map (buffer, NULL, NULL, GST_MAP_WRITE);
-    g_snprintf (data, 16, "deadbee%08x", i);
-    gst_buffer_unmap (buffer, data, 16);
+    GstBuffer *buffer = gst_new_buffer (i);
     if (i > 0)
       GST_BUFFER_FLAG_SET (buffer, GST_BUFFER_FLAG_DELTA_UNIT);