]> git.sesse.net Git - vlc/blobdiff - modules/services_discovery/sap.c
Removes trailing spaces. Removes tabs.
[vlc] / modules / services_discovery / sap.c
index f671119be7d27f96528585ae5d0c66f536cc32f0..23b8c946238a85387737fb9b505ce951f5f1d5a7 100644 (file)
@@ -1,10 +1,12 @@
 /*****************************************************************************
  * sap.c :  SAP interface module
  *****************************************************************************
- * Copyright (C) 2004-2005 VideoLAN
+ * Copyright (C) 2004-2005 the VideoLAN team
+ * Copyright © 2007 Rémi Denis-Courmont
  * $Id$
  *
  * Authors: Clément Stenac <zorglub@videolan.org>
+ *          Rémi Denis-Courmont
  *
  * This program is free software; you can redistribute it and/or modify
  * it under the terms of the GNU General Public License as published by
  *
  * You should have received a copy of the GNU General Public License
  * along with this program; if not, write to the Free Software
- * Foundation, Inc., 59 Temple Place - Suite 330, Boston, MA  02111, USA.
+ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston MA 02110-1301, USA.
  *****************************************************************************/
 
 /*****************************************************************************
  * Includes
  *****************************************************************************/
-#include <stdlib.h>                                      /* malloc(), free() */
-
 #include <vlc/vlc.h>
-#include <vlc/intf.h>
+#include <assert.h>
 
-#include <vlc/input.h>
+#include <vlc_playlist.h>
+#include <vlc_demux.h>
 
-#include "network.h"
-#include "charset.h"
+#include <vlc_network.h>
+#include <vlc_charset.h>
 
 #include <ctype.h>
 #include <errno.h>
 #   include <zlib.h>
 #endif
 
+#ifndef WIN32
+#   include <net/if.h>
+#endif
+
 /************************************************************************
  * Macros and definitions
  ************************************************************************/
 #define SAP_V4_LINK_ADDRESS     "224.0.0.255"
 #define ADD_SESSION 1
 
-#define SAP_V6_1 "FF0"
-/* Scope is inserted between them */
-#define SAP_V6_2 "::2:7FFE"
-/* See RFC3513 for list of valid scopes */
-static const char ipv6_scopes[] = "12456789ABCDE";
-
-
 /*****************************************************************************
  * Module descriptor
  *****************************************************************************/
 #define SAP_ADDR_TEXT N_( "SAP multicast address" )
-#define SAP_ADDR_LONGTEXT N_( "Listen for SAP announces on another address" )
-#define SAP_IPV4_TEXT N_( "IPv4-SAP listening" )
+#define SAP_ADDR_LONGTEXT N_( "The SAP module normally chooses itself the " \
+                              "right addresses to listen to. However, you " \
+                              "can specify a specific address." )
+#define SAP_IPV4_TEXT N_( "IPv4 SAP" )
 #define SAP_IPV4_LONGTEXT N_( \
-      "Set this if you want the SAP module to listen to IPv4 announces " \
-      "on the standard address." )
-#define SAP_IPV6_TEXT N_( "IPv6-SAP listening" )
+      "Listen to IPv4 announcements on the standard addresses." )
+#define SAP_IPV6_TEXT N_( "IPv6 SAP" )
 #define SAP_IPV6_LONGTEXT N_( \
-      "Set this if you want the SAP module to listen to IPv6 announces " \
-      "on the standard address." )
+      "Listen to IPv6 announcements on the standard addresses." )
 #define SAP_SCOPE_TEXT N_( "IPv6 SAP scope" )
 #define SAP_SCOPE_LONGTEXT N_( \
-       "Sets the scope for IPv6 announces (default is 8)." )
+       "Scope for IPv6 announcements (default is 8)." )
 #define SAP_TIMEOUT_TEXT N_( "SAP timeout (seconds)" )
 #define SAP_TIMEOUT_LONGTEXT N_( \
-       "Sets the time before SAP items get deleted if no new announce " \
+       "Delay after which SAP items get deleted if no new announcement " \
        "is received." )
-#define SAP_PARSE_TEXT N_( "Try to parse the SAP" )
+#define SAP_PARSE_TEXT N_( "Try to parse the announce" )
 #define SAP_PARSE_LONGTEXT N_( \
-       "When SAP can it will try to parse the SAP. If you don't select " \
-       "this, all announces will be parsed by the livedotcom module." )
+       "This enables actual parsing of the announces by the SAP module. " \
+       "Otherwise, all announcements are parsed by the \"livedotcom\" " \
+       "(RTP/RTSP) module." )
 #define SAP_STRICT_TEXT N_( "SAP Strict mode" )
 #define SAP_STRICT_LONGTEXT N_( \
        "When this is set, the SAP parser will discard some non-compliant " \
-       "announces." )
+       "announcements." )
 #define SAP_CACHE_TEXT N_("Use SAP cache")
 #define SAP_CACHE_LONGTEXT N_( \
-       "If this option is selected, a SAP caching mechanism will be used. " \
+       "This enables a SAP caching mechanism. " \
        "This will result in lower SAP startup time, but you could end up " \
-        "with items corresponding to legacy streams." )
+       "with items corresponding to legacy streams." )
+#define SAP_TIMESHIFT_TEXT N_("Allow timeshifting")
+#define SAP_TIMESHIFT_LONGTEXT N_( "This automatically enables timeshifting " \
+        "for streams discovered through SAP announcements." )
 
 /* Callbacks */
     static int  Open ( vlc_object_t * );
@@ -115,7 +117,7 @@ static const char ipv6_scopes[] = "12456789ABCDE";
 
 vlc_module_begin();
     set_shortname( _("SAP"));
-    set_description( _("SAP announces") );
+    set_description( _("SAP Announcements") );
     set_category( CAT_PLAYLIST );
     set_subcategory( SUBCAT_PLAYLIST_SD );
 
@@ -131,14 +133,18 @@ vlc_module_begin();
                SAP_PARSE_TEXT,SAP_PARSE_LONGTEXT, VLC_TRUE );
     add_bool( "sap-strict", 0 , NULL,
                SAP_STRICT_TEXT,SAP_STRICT_LONGTEXT, VLC_TRUE );
+#if 0
     add_bool( "sap-cache", 0 , NULL,
                SAP_CACHE_TEXT,SAP_CACHE_LONGTEXT, VLC_TRUE );
+#endif
+    add_bool( "sap-timeshift", 0 , NULL,
+              SAP_TIMESHIFT_TEXT,SAP_TIMESHIFT_LONGTEXT, VLC_TRUE );
 
     set_capability( "services_discovery", 0 );
     set_callbacks( Open, Close );
 
     add_submodule();
-        set_description( _("SDP file parser for UDP") );
+        set_description( _("SDP Descriptions parser") );
         add_shortcut( "sdp" );
         set_capability( "demux2", 51 );
         set_callbacks( OpenDemux, CloseDemux );
@@ -153,41 +159,52 @@ typedef struct sdp_t sdp_t;
 typedef struct attribute_t attribute_t;
 typedef struct sap_announce_t sap_announce_t;
 
+
+struct sdp_media_t
+{
+    struct sdp_t           *parent;
+    char                   *fmt;
+    struct sockaddr_storage addr;
+    socklen_t               addrlen;
+    unsigned                n_addr;
+    int           i_attributes;
+    attribute_t  **pp_attributes;
+};
+
+
 /* The structure that contains sdp information */
 struct  sdp_t
 {
-    char *psz_sdp;
+    const char *psz_sdp;
+
+    /* o field */
+    char     username[64];
+    uint64_t session_id;
+    uint64_t session_version;
+    unsigned orig_ip_version;
+    char     orig_host[1024];
 
     /* s= field */
     char *psz_sessionname;
 
-    /* Raw m= and c= fields */
-    char *psz_connection;
-    char *psz_media;
-
-    /* o field */
-    char *psz_username;
-    char *psz_network_type;
-    char *psz_address_type;
-    char *psz_address;
-    int64_t i_session_id;
-
+    /* old cruft */
     /* "computed" URI */
     char *psz_uri;
-
-    int         i_in; /* IP version */
-
-    int           i_media;
     int           i_media_type;
 
+    /* a= global attributes */
     int           i_attributes;
     attribute_t  **pp_attributes;
+
+    /* medias (well, we only support one atm) */
+    unsigned            mediac;
+    struct sdp_media_t *mediav;
 };
 
 struct attribute_t
 {
-    char *psz_field;
-    char *psz_value;
+    const char *value;
+    char name[0];
 };
 
 struct sap_announce_t
@@ -200,8 +217,7 @@ struct sap_announce_t
     /* SAP annnounces must only contain one SDP */
     sdp_t       *p_sdp;
 
-    int i_item_id;
-//    playlist_item_t *p_item;
+    int i_input_id;
 };
 
 struct services_discovery_sys_t
@@ -210,13 +226,6 @@ struct services_discovery_sys_t
     int i_fd;
     int *pi_fd;
 
-    /* playlist node */
-    playlist_item_t *p_node;
-    playlist_t *p_playlist;
-
-    /* charset conversion */
-    vlc_iconv_t iconvHandle;
-
     /* Table of announces */
     int i_announces;
     struct sap_announce_t **pp_announces;
@@ -224,6 +233,7 @@ struct services_discovery_sys_t
     /* Modes */
     vlc_bool_t  b_strict;
     vlc_bool_t  b_parse;
+    vlc_bool_t  b_timeshift;
 
     int i_timeout;
 };
@@ -245,30 +255,23 @@ struct demux_sys_t
 
 /* Main parsing functions */
     static int ParseConnection( vlc_object_t *p_obj, sdp_t *p_sdp );
-    static int ParseSAP( services_discovery_t *p_sd, uint8_t *p_buffer, int i_read );
-    static sdp_t *  ParseSDP( vlc_object_t *p_sd, char* psz_sdp );
+    static int ParseSAP( services_discovery_t *p_sd, const uint8_t *p_buffer, size_t i_read );
+    static sdp_t *ParseSDP (vlc_object_t *p_sd, const char *psz_sdp);
     static sap_announce_t *CreateAnnounce( services_discovery_t *, uint16_t, sdp_t * );
     static int RemoveAnnounce( services_discovery_t *p_sd, sap_announce_t *p_announce );
 
-/* Cache */
-    static void CacheLoad( services_discovery_t *p_sd );
-    static void CacheSave( services_discovery_t *p_sd );
 /* Helper functions */
-    static char *GetAttribute( sdp_t *p_sdp, const char *psz_search );
+    static inline attribute_t *MakeAttribute (const char *str);
+    static const char *GetAttribute (attribute_t **tab, unsigned n, const char *name);
+    static inline void FreeAttribute (attribute_t *a);
+    static const char *FindAttribute (const sdp_t *sdp, unsigned media,
+                                      const char *name);
+
     static vlc_bool_t IsSameSession( sdp_t *p_sdp1, sdp_t *p_sdp2 );
-    static char *convert_from_utf8( struct services_discovery_t *p_sd,
-                                   char *psz_unicode );
-    static int InitSocket( services_discovery_t *p_sd, char *psz_address, int i_port );
-#ifdef HAVE_ZLIB_H
-    static int Decompress( unsigned char *psz_src, unsigned char **_dst, int i_len );
-#endif
+    static int InitSocket( services_discovery_t *p_sd, const char *psz_address, int i_port );
+    static int Decompress( const unsigned char *psz_src, unsigned char **_dst, int i_len );
     static void FreeSDP( sdp_t *p_sdp );
 
-/* Detect multicast addresses */
-static vlc_bool_t ismult( char * );
-
-#define FREE( p ) \
-    if( p ) { free( p ); (p) = NULL; }
 /*****************************************************************************
  * Open: initialize and create stuff
  *****************************************************************************/
@@ -278,21 +281,8 @@ static int Open( vlc_object_t *p_this )
     services_discovery_sys_t *p_sys  = (services_discovery_sys_t *)
                                 malloc( sizeof( services_discovery_sys_t ) );
 
-    playlist_view_t     *p_view;
-    char                *psz_addr, *psz_charset;
-    vlc_value_t         val;
-
     p_sys->i_timeout = var_CreateGetInteger( p_sd, "sap-timeout" );
 
-    vlc_current_charset( &psz_charset );
-    p_sys->iconvHandle = vlc_iconv_open( psz_charset, "UTF-8" );
-    free( psz_charset );
-    if( p_sys->iconvHandle == (vlc_iconv_t)(-1) )
-    {
-        msg_Err( p_sd, "unable to perform characters conversion" );
-       return VLC_EGENERIC;
-    }
-
     p_sd->pf_run = Run;
     p_sd->p_sys  = p_sys;
 
@@ -302,59 +292,18 @@ static int Open( vlc_object_t *p_this )
     p_sys->b_strict = var_CreateGetInteger( p_sd, "sap-strict");
     p_sys->b_parse = var_CreateGetInteger( p_sd, "sap-parse" );
 
+#if 0
     if( var_CreateGetInteger( p_sd, "sap-cache" ) )
     {
         CacheLoad( p_sd );
     }
+#endif
 
-    if( var_CreateGetInteger( p_sd, "sap-ipv4" ) )
-    {
-        InitSocket( p_sd, SAP_V4_GLOBAL_ADDRESS, SAP_PORT );
-        InitSocket( p_sd, SAP_V4_ORG_ADDRESS, SAP_PORT );
-        InitSocket( p_sd, SAP_V4_LOCAL_ADDRESS, SAP_PORT );
-        InitSocket( p_sd, SAP_V4_LINK_ADDRESS, SAP_PORT );
-    }
-    if( var_CreateGetInteger( p_sd, "sap-ipv6" ) )
-    {
-        char psz_address[] = SAP_V6_1"0"SAP_V6_2;
-        const char *c_scope;
-
-        for( c_scope = ipv6_scopes; *c_scope; c_scope++ )
-        {
-            psz_address[sizeof(SAP_V6_1) - 1] = *c_scope;
-            InitSocket( p_sd, psz_address, SAP_PORT );
-        }
-    }
-
-    psz_addr = var_CreateGetString( p_sd, "sap-addr" );
-    if( psz_addr && *psz_addr )
-    {
-        InitSocket( p_sd, psz_addr, SAP_PORT );
-    }
-
-    if( p_sys->i_fd == 0 )
-    {
-        msg_Err( p_sd, "unable to read on any address" );
-        return VLC_EGENERIC;
-    }
-
-    /* Create our playlist node */
-    p_sys->p_playlist = (playlist_t *)vlc_object_find( p_sd,
-                                                       VLC_OBJECT_PLAYLIST,
-                                                       FIND_ANYWHERE );
-    if( !p_sys->p_playlist )
-    {
-        msg_Warn( p_sd, "unable to find playlist, cancelling SAP listening");
-        return VLC_EGENERIC;
-    }
+    /* Cache sap_timeshift value */
+    p_sys->b_timeshift = var_CreateGetInteger( p_sd, "sap-timeshift" );
 
-    p_view = playlist_ViewFind( p_sys->p_playlist, VIEW_CATEGORY );
-    p_sys->p_node = playlist_NodeCreate( p_sys->p_playlist, VIEW_CATEGORY,
-                                         _("SAP"), p_view->p_root );
-    p_sys->p_node->i_flags |= PLAYLIST_RO_FLAG;
-    p_sys->p_node->i_flags =~ PLAYLIST_SKIP_FLAG;
-    val.b_bool = VLC_TRUE;
-    var_Set( p_sys->p_playlist, "intf-change", val );
+    /* Set our name */
+    services_discovery_SetLocalizedName( p_sd, _("SAP") );
 
     p_sys->i_announces = 0;
     p_sys->pp_announces = NULL;
@@ -368,50 +317,52 @@ static int Open( vlc_object_t *p_this )
 static int OpenDemux( vlc_object_t *p_this )
 {
     demux_t *p_demux = (demux_t *)p_this;
-    uint8_t *p_peek;
-    int i_max_sdp = 1024;
-    int i_sdp = 0;
+    const uint8_t *p_peek;
     char *psz_sdp = NULL;
     sdp_t *p_sdp = NULL;
+    int errval = VLC_EGENERIC;
+    size_t i_len;
 
-    /* Probe for SDP */
-    if( p_demux->s )
+    if( !var_CreateGetInteger( p_demux, "sap-parse" ) )
     {
-        if( stream_Peek( p_demux->s, &p_peek, 7 ) < 7 ) return VLC_EGENERIC;
-
-        if( strncmp( (char*)p_peek, "v=0\r\n", 5 ) &&
-            strncmp( (char*)p_peek, "v=0\n", 4 ) &&
-            ( p_peek[0] < 'a' || p_peek[0] > 'z' || p_peek[1] != '=' ) )
-        {
-            return VLC_EGENERIC;
-        }
+        /* We want livedotcom module to parse this SDP file */
+        return VLC_EGENERIC;
     }
 
-    psz_sdp = (char *)malloc( i_max_sdp );
-    if( !psz_sdp ) return VLC_EGENERIC;
+    assert( p_demux->s ); /* this is NOT an access_demux */
+
+    /* Probe for SDP */
+    if( stream_Peek( p_demux->s, &p_peek, 7 ) < 7 )
+        return VLC_EGENERIC;
+
+    if( memcmp( p_peek, "v=0\r\no=", 7 ) && memcmp( p_peek, "v=0\no=", 6 ) )
+        return VLC_EGENERIC;
 
     /* Gather the complete sdp file */
-    for( ;; )
+    for( i_len = 0, psz_sdp = NULL; i_len < 65536; )
     {
-        int i_read = stream_Read( p_demux->s,
-                                  &psz_sdp[i_sdp], i_max_sdp - i_sdp - 1 );
-
-        if( i_read < 0 )
+        const int i_read_max = 1024;
+        char *psz_sdp_new = realloc( psz_sdp, i_len + i_read_max );
+        size_t i_read;
+        if( psz_sdp_new == NULL )
         {
-            msg_Err( p_demux, "failed to read SDP" );
+            errval = VLC_ENOMEM;
             goto error;
         }
+        psz_sdp = psz_sdp_new;
 
-        i_sdp += i_read;
-
-        if( i_read < i_max_sdp - i_sdp - 1 )
+        i_read = stream_Read( p_demux->s, &psz_sdp[i_len], i_read_max );
+        if( i_read < 0 )
         {
-            psz_sdp[i_sdp] = '\0';
-            break;
+            msg_Err( p_demux, "cannot read SDP" );
+            goto error;
         }
+        i_len += i_read;
+
+        psz_sdp[i_len] = '\0';
 
-        i_max_sdp += 1000;
-        psz_sdp = (uint8_t*)realloc( psz_sdp, i_max_sdp );
+        if( i_read < i_read_max )
+            break; // EOF
     }
 
     p_sdp = ParseSDP( VLC_OBJECT(p_demux), psz_sdp );
@@ -422,11 +373,6 @@ static int OpenDemux( vlc_object_t *p_this )
         goto error;
     }
 
-    if( p_sdp->i_media > 1 )
-    {
-        goto error;
-    }
-
     if( ParseConnection( VLC_OBJECT( p_demux ), p_sdp ) )
     {
         p_sdp->psz_uri = NULL;
@@ -442,14 +388,14 @@ static int OpenDemux( vlc_object_t *p_this )
     p_demux->pf_control = Control;
     p_demux->pf_demux = Demux;
 
-    free( psz_sdp );
+    FREENULL( psz_sdp );
     return VLC_SUCCESS;
 
 error:
-    free( psz_sdp );
-    if( p_sdp ) FreeSDP( p_sdp );
+    FREENULL( psz_sdp );
+    if( p_sdp ) FreeSDP( p_sdp ); p_sdp = NULL;
     stream_Seek( p_demux->s, 0 );
-    return VLC_EGENERIC;    
+    return errval;
 }
 
 /*****************************************************************************
@@ -466,28 +412,22 @@ static void Close( vlc_object_t *p_this )
     {
         net_Close( p_sys->pi_fd[i] );
     }
-    FREE( p_sys->pi_fd );
+    FREENULL( p_sys->pi_fd );
 
+#if 0
     if( config_GetInt( p_sd, "sap-cache" ) )
     {
         CacheSave( p_sd );
     }
+#endif
 
     for( i = p_sys->i_announces  - 1;  i>= 0; i-- )
     {
         RemoveAnnounce( p_sd, p_sys->pp_announces[i] );
     }
-    FREE( p_sys->pp_announces );
-
-    if( p_sys->p_playlist )
-    {
-        playlist_NodeDelete( p_sys->p_playlist, p_sys->p_node, VLC_TRUE,
-                             VLC_TRUE );
-        vlc_object_release( p_sys->p_playlist );
-    }
-
-    vlc_iconv_close( p_sys->iconvHandle );
+    FREENULL( p_sys->pp_announces );
 
+    pl_Release( p_sd );
     free( p_sys );
 }
 
@@ -496,7 +436,12 @@ static void Close( vlc_object_t *p_this )
  *****************************************************************************/
 static void CloseDemux( vlc_object_t *p_this )
 {
-
+    demux_t *p_demux = (demux_t *)p_this;
+    if( p_demux->p_sys )
+    {
+        if( p_demux->p_sys->p_sdp ) { FreeSDP( p_demux->p_sys->p_sdp ); p_demux->p_sys->p_sdp = NULL; }
+        free( p_demux->p_sys );
+    }
 }
 
 /*****************************************************************************
@@ -508,17 +453,94 @@ static void CloseDemux( vlc_object_t *p_this )
 
 static void Run( services_discovery_t *p_sd )
 {
-    int                i;
+    char *psz_addr;
+    int i;
+
+    /* Braindead Winsock DNS resolver will get stuck over 2 seconds per failed
+     * DNS queries, even if the DNS server returns an error with milliseconds.
+     * You don't want to know why the bug (as of XP SP2) wasn't fixed since
+     * Winsock 1.1 from Windows 95, if not Windows 3.1.
+     * Anyway, to avoid a 30 seconds delay for failed IPv6 socket creation,
+     * we have to open sockets in Run() rather than Open(). */
+    if( var_CreateGetInteger( p_sd, "sap-ipv4" ) )
+    {
+        InitSocket( p_sd, SAP_V4_GLOBAL_ADDRESS, SAP_PORT );
+        InitSocket( p_sd, SAP_V4_ORG_ADDRESS, SAP_PORT );
+        InitSocket( p_sd, SAP_V4_LOCAL_ADDRESS, SAP_PORT );
+        InitSocket( p_sd, SAP_V4_LINK_ADDRESS, SAP_PORT );
+    }
+    if( var_CreateGetInteger( p_sd, "sap-ipv6" ) )
+    {
+        char psz_address[NI_MAXNUMERICHOST] = "ff02::2:7ffe%";
+
+#ifndef WIN32
+        struct if_nameindex *l = if_nameindex ();
+        if (l != NULL)
+        {
+            char *ptr = strchr (psz_address, '%') + 1;
+            for (unsigned i = 0; l[i].if_index; i++)
+            {
+                strcpy (ptr, l[i].if_name);
+                InitSocket (p_sd, psz_address, SAP_PORT);
+            }
+            if_freenameindex (l);
+        }
+#else
+        /* this is the Winsock2 equivalant of SIOCGIFCONF on BSD stacks,
+           which if_nameindex uses internally anyway */
+
+        // first create a dummy socket to pin down the protocol family
+        SOCKET s = socket(PF_INET6, SOCK_DGRAM, IPPROTO_UDP);
+        if( s != INVALID_SOCKET )
+        {
+            INTERFACE_INFO ifaces[10]; // Assume there will be no more than 10 IP interfaces
+            size_t len = sizeof(ifaces);
+
+            if( SOCKET_ERROR != WSAIoctl(s, SIO_GET_INTERFACE_LIST, NULL, 0, &ifaces, len, &len, NULL, NULL) )
+            {
+                unsigned ifcount = len/sizeof(INTERFACE_INFO);
+                char *ptr = strchr (psz_address, '%') + 1;
+                for(unsigned i = 1; i<=ifcount; ++i )
+                {
+                    // append link-local zone identifier
+                    sprintf(ptr, "%d", i);
+                }
+            }
+            closesocket(s);
+        }
+#endif
+        *strchr (psz_address, '%') = '\0';
+
+        static const char ipv6_scopes[] = "1456789ABCDE";
+        for (const char *c_scope = ipv6_scopes; *c_scope; c_scope++)
+        {
+            psz_address[3] = *c_scope;
+            InitSocket( p_sd, psz_address, SAP_PORT );
+        }
+    }
+
+    psz_addr = var_CreateGetString( p_sd, "sap-addr" );
+    if( psz_addr && *psz_addr )
+    {
+        InitSocket( p_sd, psz_addr, SAP_PORT );
+        free( psz_addr );
+    }
+
+    if( p_sd->p_sys->i_fd == 0 )
+    {
+        msg_Err( p_sd, "unable to listen on any address" );
+        return;
+    }
 
     /* read SAP packets */
     while( !p_sd->b_die )
     {
         int i_read;
-       uint8_t p_buffer[MAX_SAP_BUFFER];
+        uint8_t p_buffer[MAX_SAP_BUFFER+1];
 
-        i_read = net_Select( p_sd, p_sd->p_sys->pi_fd, NULL,
+        i_read = net_Select( p_sd, p_sd->p_sys->pi_fd,
                              p_sd->p_sys->i_fd, p_buffer,
-                             MAX_SAP_BUFFER, 500000 );
+                             MAX_SAP_BUFFER );
 
         /* Check for items that need deletion */
         for( i = 0; i < p_sd->p_sys->i_announces; i++ )
@@ -527,18 +549,7 @@ static void Run( services_discovery_t *p_sd )
 
             if( mdate() - p_sd->p_sys->pp_announces[i]->i_last > i_timeout )
             {
-                struct sap_announce_t *p_announce;
-                p_announce = p_sd->p_sys->pp_announces[i];
-
-                /* Remove the playlist item */
-                playlist_LockDelete( p_sd->p_sys->p_playlist,
-                                     p_announce->i_item_id );
-
-                /* Remove the sap_announce from the array */
-                REMOVE_ELEM( p_sd->p_sys->pp_announces,
-                           p_sd->p_sys->i_announces, i );
-
-                free( p_announce );
+                RemoveAnnounce( p_sd, p_sd->p_sys->pp_announces[i] );
             }
         }
 
@@ -566,18 +577,38 @@ static void Run( services_discovery_t *p_sd )
 static int Demux( demux_t *p_demux )
 {
     sdp_t *p_sdp = p_demux->p_sys->p_sdp;
-    playlist_t *p_playlist;
+    input_thread_t *p_input;
+    input_item_t *p_parent_input;
+
+    playlist_t *p_playlist = pl_Yield( p_demux );
+    p_input = (input_thread_t *)vlc_object_find( p_demux, VLC_OBJECT_INPUT,
+                                                 FIND_PARENT );
+    assert( p_input );
+    if( !p_input )
+    {
+        msg_Err( p_demux, "parent input could not be found" );
+        return VLC_EGENERIC;
+    }
+
+    p_parent_input = input_GetItem( p_input );
 
-    p_playlist = (playlist_t *)vlc_object_find( p_demux, VLC_OBJECT_PLAYLIST,
-                                               FIND_ANYWHERE );
+    input_item_SetURI( p_parent_input, p_sdp->psz_uri );
+    input_item_SetName( p_parent_input, p_sdp->psz_sessionname );
 
-    p_playlist->status.p_item->i_flags |= PLAYLIST_DEL_FLAG;
+    vlc_mutex_lock( &p_parent_input->lock );
 
-    playlist_Add( p_playlist, p_sdp->psz_uri, p_sdp->psz_sessionname,
-                 PLAYLIST_APPEND, PLAYLIST_END );
+    p_parent_input->i_type = ITEM_TYPE_NET;
+
+    if( p_playlist->status.p_item &&
+             p_playlist->status.p_item->p_input == p_parent_input )
+    {
+        playlist_Control( p_playlist, PLAYLIST_VIEWPLAY, VLC_TRUE,
+                          p_playlist->status.p_node, p_playlist->status.p_item );
+    }
 
+    vlc_mutex_unlock( &p_parent_input->lock );
+    vlc_object_release( p_input );
     vlc_object_release( p_playlist );
-    if( p_sdp ) FreeSDP( p_sdp );
 
     return VLC_SUCCESS;
 }
@@ -592,50 +623,37 @@ static int Control( demux_t *p_demux, int i_query, va_list args )
  **************************************************************/
 
 /* i_read is at least > 6 */
-static int ParseSAP( services_discovery_t *p_sd, uint8_t *p_buffer, int i_read )
+static int ParseSAP( services_discovery_t *p_sd, const uint8_t *buf,
+                     size_t len )
 {
-    int                 i_version, i_address_type, i_hash, i;
-    uint8_t             *psz_sdp;
-    uint8_t             *psz_initial_sdp;
+    int i;
+    const char          *psz_sdp;
+    const uint8_t *end = buf + len;
     sdp_t               *p_sdp;
-    vlc_bool_t          b_compressed;
-    vlc_bool_t          b_need_delete = VLC_FALSE;
-#ifdef HAVE_ZLIB_H
-    int                 i_decompressed_size;
-    uint8_t             *p_decompressed_buffer;
-#endif
-    uint8_t             *psz_foo;
 
-    /* First, check the sap announce is correct */
-    i_version = p_buffer[0] >> 5;
-    if( i_version != 1 )
-    {
-       msg_Dbg( p_sd, "strange sap version %d found", i_version );
-    }
+    assert (buf[len] == '\0');
 
-    i_address_type = p_buffer[0] & 0x10;
+    if (len < 4)
+        return VLC_EGENERIC;
 
-    if( (p_buffer[0] & 0x08) != 0 )
-    {
-        msg_Dbg( p_sd, "reserved bit incorrectly set" );
+    uint8_t flags = buf[0];
+
+    /* First, check the sap announce is correct */
+    if ((flags >> 5) != 1)
         return VLC_EGENERIC;
-    }
 
-    if( (p_buffer[0] & 0x04) != 0 )
-    {
-        msg_Dbg( p_sd, "session deletion packet" );
-        b_need_delete = VLC_TRUE;
-    }
+    vlc_bool_t b_ipv6 = (flags & 0x10) != 0;
+    vlc_bool_t b_need_delete = (flags & 0x04) != 0;
 
-    if( p_buffer[0] & 0x02  )
+    if (flags & 0x02)
     {
         msg_Dbg( p_sd, "encrypted packet, unsupported" );
         return VLC_EGENERIC;
     }
 
-    b_compressed = p_buffer[0] & 0x01;
+    vlc_bool_t b_compressed = (flags & 0x01) != 0;
 
-    i_hash = ( p_buffer[2] << 8 ) + p_buffer[3];
+    uint16_t i_hash = U16_AT (buf + 2);
 
     if( p_sd->p_sys->b_strict && i_hash == 0 )
     {
@@ -643,102 +661,77 @@ static int ParseSAP( services_discovery_t *p_sd, uint8_t *p_buffer, int i_read )
         return VLC_EGENERIC;
     }
 
-    psz_sdp  = &p_buffer[4];
-    psz_initial_sdp = psz_sdp;
+    // Skips source address and auth data
+    buf += 4 + (b_ipv6 ? 16 : 4) + buf[1];
+    if (buf > end)
+        return VLC_EGENERIC;
 
-    if( i_address_type == 0 ) /* ipv4 source address */
-    {
-        psz_sdp += 4;
-        if( i_read <= 9 )
-        {
-            msg_Warn( p_sd, "too short SAP packet\n" );
-            return VLC_EGENERIC;
-        }
-    }
-    else /* ipv6 source address */
+    uint8_t *decomp = NULL;
+    if( b_compressed )
     {
-        psz_sdp += 16;
-        if( i_read <= 21 )
+        int newsize = Decompress (buf, &decomp, end - buf);
+        if (newsize < 0)
         {
-            msg_Warn( p_sd, "too short SAP packet\n" );
+            msg_Dbg( p_sd, "decompression of SAP packet failed" );
             return VLC_EGENERIC;
         }
-    }
 
-    if( b_compressed )
-    {
-#ifdef HAVE_ZLIB_H
-        i_decompressed_size = Decompress( psz_sdp,
-                   &p_decompressed_buffer,i_read - ( psz_sdp - p_buffer ) );
-        if( i_decompressed_size > 0 && i_decompressed_size < MAX_SAP_BUFFER )
-        {
-            memcpy( psz_sdp, p_decompressed_buffer, i_decompressed_size );
-            psz_sdp[i_decompressed_size] = '\0';
-            free( p_decompressed_buffer );
-        }
-#else
-        msg_Warn( p_sd, "Ignoring compressed sap packet" );
-        return VLC_EGENERIC;
-#endif
+        decomp = realloc (decomp, newsize + 1);
+        decomp[newsize] = '\0';
+        psz_sdp = (const char *)decomp;
+        len = newsize;
     }
-
-    /* Add the size of authentification info */
-    if( i_read < p_buffer[1] + (psz_sdp - psz_initial_sdp ) )
+    else
     {
-        msg_Warn( p_sd, "too short SAP packet\n");
-        return VLC_EGENERIC;
+        psz_sdp = (const char *)buf;
+        len = end - buf;
     }
-    psz_sdp += p_buffer[1];
-    psz_foo = psz_sdp;
+
+    /* len is a strlen here here. both buf and decomp are len+1 where the 1 should be a \0 */
+    assert( psz_sdp[len] == '\0');
 
     /* Skip payload type */
-    /* Handle announces without \0 between SAP and SDP */
-    while( *psz_sdp != '\0' && ( psz_sdp[0] != 'v' && psz_sdp[1] != '=' ) )
+    /* SAPv1 has implicit "application/sdp" payload type: first line is v=0 */
+    if (strncmp (psz_sdp, "v=0", 3))
     {
-        if( psz_sdp - psz_initial_sdp >= i_read - 5 )
+        size_t clen = strlen (psz_sdp) + 1;
+
+        if (strcmp (psz_sdp, "application/sdp"))
         {
-            msg_Warn( p_sd, "empty SDP ?");
+            msg_Dbg (p_sd, "unsupported content type: %s", psz_sdp);
+            return VLC_EGENERIC;
         }
-        psz_sdp++;
-    }
 
-    if( *psz_sdp == '\0' )
-    {
-        psz_sdp++;
-    }
-    if( psz_sdp != psz_foo && strcasecmp( psz_foo, "application/sdp" ) )
-    {
-        msg_Dbg( p_sd, "unhandled content type: %s", psz_foo );        
-    }
-    if( psz_sdp -p_buffer >= i_read )
-    {
-        msg_Warn( p_sd, "package without content" );
-        return VLC_EGENERIC;
+        // skips content type
+        if (len <= clen)
+            return VLC_EGENERIC;
+
+        len -= clen;
+        psz_sdp += clen;
     }
 
     /* Parse SDP info */
     p_sdp = ParseSDP( VLC_OBJECT(p_sd), psz_sdp );
 
     if( p_sdp == NULL )
-    {
         return VLC_EGENERIC;
-    }
+
+    p_sdp->psz_sdp = psz_sdp;
 
     /* Decide whether we should add a playlist item for this SDP */
     /* Parse connection information (c= & m= ) */
     if( ParseConnection( VLC_OBJECT(p_sd), p_sdp ) )
-    {
         p_sdp->psz_uri = NULL;
-    }
 
     /* Multi-media or no-parse -> pass to LIVE.COM */
-    if( p_sdp->i_media > 1 || ( p_sdp->i_media_type != 14 &&
-                                p_sdp->i_media_type != 32 &&
-                                p_sdp->i_media_type != 33) ||
-        p_sd->p_sys->b_parse == VLC_FALSE )
-    {
-        if( p_sdp->psz_uri ) free( p_sdp->psz_uri );
-        asprintf( &p_sdp->psz_uri, "sdp://%s", p_sdp->psz_sdp );
+    if( ( p_sdp->i_media_type != 14
+       && p_sdp->i_media_type != 32
+       && p_sdp->i_media_type != 33)
+     || p_sd->p_sys->b_parse == VLC_FALSE )
+    {
+        free( p_sdp->psz_uri );
+        if (asprintf( &p_sdp->psz_uri, "sdp://%s", p_sdp->psz_sdp ) == -1)
+            p_sdp->psz_uri = NULL;
     }
 
     if( p_sdp->psz_uri == NULL ) return VLC_EGENERIC;
@@ -752,618 +745,628 @@ static int ParseSAP( services_discovery_t *p_sd, uint8_t *p_buffer, int i_read )
             if( b_need_delete )
             {
                 RemoveAnnounce( p_sd, p_sd->p_sys->pp_announces[i]);
-                return VLC_SUCCESS;
             }
             else
             {
                 p_sd->p_sys->pp_announces[i]->i_last = mdate();
-                FreeSDP( p_sdp );
-                return VLC_SUCCESS;
             }
+            FreeSDP( p_sdp ); p_sdp = NULL;
+            return VLC_SUCCESS;
         }
     }
-    /* Add item */
-    if( p_sdp->i_media > 1 )
-    {
-        msg_Dbg( p_sd, "passing to LIVE.COM" );
-    }
 
     CreateAnnounce( p_sd, i_hash, p_sdp );
 
+    FREENULL (decomp);
     return VLC_SUCCESS;
 }
 
 sap_announce_t *CreateAnnounce( services_discovery_t *p_sd, uint16_t i_hash,
                                 sdp_t *p_sdp )
 {
-    playlist_item_t     *p_item, *p_child;
-    char                *psz_value;
+    input_item_t *p_input;
+    const char *psz_value;
     sap_announce_t *p_sap = (sap_announce_t *)malloc(
                                         sizeof(sap_announce_t ) );
-
-    psz_value = convert_from_utf8( p_sd, p_sdp->psz_sessionname );
-    if( p_sap == NULL || psz_value == NULL )
-    {
-        FREE( p_sap );
-        FREE( psz_value );
+    services_discovery_sys_t *p_sys;
+    if( p_sap == NULL )
         return NULL;
-    }
+
+    p_sys = p_sd->p_sys;
+
     p_sap->i_last = mdate();
     p_sap->i_hash = i_hash;
     p_sap->p_sdp = p_sdp;
-    p_sap->i_item_id = -1;
-
-    /* Create the playlist item here */
-    p_item = playlist_ItemNew( p_sd, p_sap->p_sdp->psz_uri, psz_value );
-    free( psz_value );
 
-    if( !p_item )
+    /* Create the actual playlist item here */
+    p_input = input_ItemNewWithType( VLC_OBJECT(p_sd),
+                                     p_sap->p_sdp->psz_uri,
+                                     p_sdp->psz_sessionname,
+                                     0, NULL, -1, ITEM_TYPE_NET );
+    p_sap->i_input_id = p_input->i_id;
+    if( !p_input )
     {
         free( p_sap );
         return NULL;
     }
 
-    psz_value = GetAttribute( p_sap->p_sdp, "tool" );
-    if( psz_value != NULL )
-    {
-        vlc_input_item_AddInfo( &p_item->input, _("Session"),
-                                _("Tool"), psz_value );
-    }
-    if( strcmp( p_sdp->psz_username, "-" ) )
-    {
-        vlc_input_item_AddInfo( &p_item->input, _("Session"),
-                                _("User"), p_sdp->psz_username );
-    }
-
-    psz_value = GetAttribute( p_sap->p_sdp, "x-plgroup" );
-
-    if( psz_value == NULL )
-    {
-        psz_value = GetAttribute( p_sap->p_sdp, "plgroup" );
-    }
+    if( p_sys->b_timeshift )
+        input_ItemAddOption( p_input, ":access-filter=timeshift" );
 
+    psz_value = GetAttribute( p_sap->p_sdp->pp_attributes, p_sap->p_sdp->i_attributes, "tool" );
     if( psz_value != NULL )
     {
-        char *psz_grp = convert_from_utf8( p_sd, psz_value );
-
-        if( psz_grp != NULL )
-        {
-            p_child = playlist_ChildSearchName( p_sd->p_sys->p_node,
-                                                psz_grp );
-
-            if( p_child == NULL )
-            {
-                p_child = playlist_NodeCreate( p_sd->p_sys->p_playlist,
-                                               VIEW_CATEGORY, psz_grp,
-                                               p_sd->p_sys->p_node );
-                p_child->i_flags =~ PLAYLIST_SKIP_FLAG;
-            }
-            free( psz_grp );
-        }
-        else
-        {
-            msg_Err( p_sd, "out of memory");
-            free( p_sap );
-            return NULL;
-        }
+        input_ItemAddInfo( p_input, _("Session"), _("Tool"), "%s",
+                           psz_value );
     }
-    else
+    if( strcmp( p_sdp->username, "-" ) )
     {
-        p_child = p_sd->p_sys->p_node;
+        input_ItemAddInfo( p_input, _("Session"), _("User"), "%s",
+                           p_sdp->username );
     }
 
-    p_item->i_flags &= ~PLAYLIST_SKIP_FLAG;
-    p_item->i_flags &= ~PLAYLIST_SAVE_FLAG;
-
-    playlist_NodeAddItem( p_sd->p_sys->p_playlist, p_item, VIEW_CATEGORY,
-                              p_child, PLAYLIST_APPEND, PLAYLIST_END );
+    /* Handle group */
+    if (p_sap->p_sdp->mediac >= 1)
+        psz_value = FindAttribute (p_sap->p_sdp, 0, "x-plgroup");
+    else
+        psz_value = GetAttribute( p_sap->p_sdp->pp_attributes, p_sap->p_sdp->i_attributes, "x-plgroup" );
 
-    p_sap->i_item_id = p_item->input.i_id;
+    services_discovery_AddItem( p_sd, p_input, psz_value /* category name */ );
 
-    TAB_APPEND( p_sd->p_sys->i_announces,
-                p_sd->p_sys->pp_announces, p_sap );
+    TAB_APPEND( p_sys->i_announces, p_sys->pp_announces, p_sap );
 
     return p_sap;
 }
 
-static char *GetAttribute( sdp_t *p_sdp, const char *psz_search )
-{
-    int i;
 
-    for( i = 0 ; i< p_sdp->i_attributes; i++ )
-    {
-        if( !strncmp( p_sdp->pp_attributes[i]->psz_field, psz_search,
-                      strlen( p_sdp->pp_attributes[i]->psz_field ) ) )
-        {
-            return p_sdp->pp_attributes[i]->psz_value;
-        }
-    }
-    return NULL;
+static const char *FindAttribute (const sdp_t *sdp, unsigned media,
+                                  const char *name)
+{
+    /* Look for media attribute, and fallback to session */
+    return GetAttribute (sdp->mediav[media].pp_attributes,
+                         sdp->mediav[media].i_attributes, name)
+        ?: GetAttribute (sdp->pp_attributes, sdp->i_attributes, name);
 }
 
 
 /* Fill p_sdp->psz_uri */
 static int ParseConnection( vlc_object_t *p_obj, sdp_t *p_sdp )
 {
-    char *psz_eof;
-    char *psz_parse;
-    char *psz_uri = NULL;
-    char *psz_proto = NULL;
-    int i_port = 0;
-
-    /* Parse c= field */
-    if( p_sdp->psz_connection )
+    if (p_sdp->mediac == 0)
     {
-        psz_parse = p_sdp->psz_connection;
-
-        psz_eof = strchr( psz_parse, ' ' );
+        msg_Dbg (p_obj, "Ignoring SDP with no media");
+        return VLC_EGENERIC;
+    }
 
-        if( psz_eof )
-        {
-            *psz_eof = '\0';
-            psz_parse = psz_eof + 1;
-        }
-        else
+    for (unsigned i = 1; i < p_sdp->mediac; i++)
+    {
+        if ((p_sdp->mediav[i].n_addr != p_sdp->mediav->n_addr)
+         || (p_sdp->mediav[i].addrlen != p_sdp->mediav->addrlen)
+         || memcmp (&p_sdp->mediav[i].addr, &p_sdp->mediav->addr,
+                    p_sdp->mediav->addrlen))
         {
-            msg_Warn( p_obj, "unable to parse c field (1)");
+            msg_Dbg (p_obj, "Multiple media ports not supported -> live555");
             return VLC_EGENERIC;
         }
+    }
 
-        psz_eof = strchr( psz_parse, ' ' );
-
-        if( psz_eof )
-        {
-            *psz_eof = '\0';
-            if( !strncmp( psz_parse, "IP4", 3 ) )
-            {
-                p_sdp->i_in = 4;
-            }
-            else if( !strncmp( psz_parse, "IP6", 3 ) )
-            {
-                p_sdp->i_in = 6;
-            }
-            else
-            {
-                p_sdp->i_in = 0;
-            }
-            psz_parse = psz_eof + 1;
-        }
-        else
-        {
-            msg_Warn( p_obj, "unable to parse c field (2)");
-            return VLC_EGENERIC;
-        }
+    if (p_sdp->mediav->n_addr != 1)
+    {
+        msg_Dbg (p_obj, "Layered encoding not supported -> live555");
+        return VLC_EGENERIC;
+    }
 
-        psz_eof = strchr( psz_parse, '/' );
+    char psz_uri[1026];
+    const char *host;
+    int port;
 
-        if( psz_eof )
-        {
-            *psz_eof = 0;
-        }
-        else
-        {
-            msg_Dbg( p_obj, "incorrect c field, %s", p_sdp->psz_connection );
-        }
-        psz_uri = strdup( psz_parse );
+    psz_uri[0] = '[';
+    if (vlc_getnameinfo ((struct sockaddr *)&(p_sdp->mediav->addr),
+                         p_sdp->mediav->addrlen, psz_uri + 1,
+                         sizeof (psz_uri) - 2, &port, NI_NUMERICHOST))
+        return VLC_EGENERIC;
 
+    if (strchr (psz_uri + 1, ':'))
+    {
+        host = psz_uri;
+        psz_uri[strlen (psz_uri)] = ']';
     }
+    else
+        host = psz_uri + 1;
 
     /* Parse m= field */
-    if( p_sdp->psz_media )
-    {
-        psz_parse = p_sdp->psz_media;
-
-        psz_eof = strchr( psz_parse, ' ' );
+    char *sdp_proto = strdup (p_sdp->mediav[0].fmt);
+    if (sdp_proto == NULL)
+        return VLC_ENOMEM;
 
-        if( psz_eof )
+    char *subtype = strchr (sdp_proto, ' ');
+    if (subtype == NULL)
+    {
+        msg_Dbg (p_obj, "missing SDP media subtype: %s", sdp_proto);
+        p_sdp->i_media_type = 0;
+    }
+    else
+    {
+        *subtype++ = '\0';
+        p_sdp->i_media_type = atoi (subtype);
+    }
+    if (p_sdp->i_media_type == 0)
+         p_sdp->i_media_type = 33;
+
+    /* RTP protocol, nul, VLC shortcut, nul, flags byte as follow:
+     * 0x1: Connection-Oriented media. */
+    static const char proto_match[] =
+        "udp\0"             "udp\0\0"
+        "RTP/AVP\0"         "rtp\0\0"
+        "UDPLite/RTP/AVP\0" "udplite\0\0"
+        "DCCP/RTP/AVP\0"    "dccp\0\1"
+        "TCP/RTP/AVP\0"     "rtptcp\0\1"
+        "\0";
+
+    const char *vlc_proto = NULL;
+    uint8_t flags = 0;
+    for (const char *proto = proto_match; *proto;)
+    {
+        if (strcasecmp (proto, sdp_proto) == 0)
         {
-            *psz_eof = '\0';
+            vlc_proto = proto + strlen (proto) + 1;
+            flags = vlc_proto[strlen (vlc_proto) + 1];
+            break;
+        }
+        proto += strlen (proto) + 1;
+        proto += strlen (proto) + 2;
+    }
 
-            if( strncmp( psz_parse, "audio", 5 )  &&
-                strncmp( psz_parse, "video",5 ) )
-            {
-                msg_Warn( p_obj, "unhandled media type -%s-", psz_parse );
-                FREE( psz_uri );
-                return VLC_EGENERIC;
-            }
+    free (sdp_proto);
+    if (vlc_proto == NULL)
+    {
+        msg_Dbg (p_obj, "unknown SDP media protocol: %s",
+                 p_sdp->mediav[0].fmt);
+        return VLC_EGENERIC;
+    }
 
-            psz_parse = psz_eof + 1;
-        }
-        else
+    if (flags & 1)
+    {
+        /* Connection-oriented media */
+        const char *setup = FindAttribute (p_sdp, 0, "setup");
+        if (setup == NULL)
+            setup = "active"; /* default value */
+
+        if (strcmp (setup, "actpass") && strcmp (setup, "passive"))
         {
-            msg_Warn( p_obj, "unable to parse m field (1)");
-            FREE( psz_uri );
+            msg_Dbg (p_obj, "unsupported COMEDIA mode: %s", setup);
             return VLC_EGENERIC;
         }
 
-        psz_eof = strchr( psz_parse, ' ' );
+        if (asprintf (&p_sdp->psz_uri, "%s://%s:%d", vlc_proto,
+                      host, port) == -1)
+            return VLC_ENOMEM;
+    }
+    else
+    {
+        /* Non-connected (normally multicast) media */
 
-        if( psz_eof )
+        char psz_source[258] = "";
+        const char *sfilter = FindAttribute (p_sdp, 0, "source-filter");
+        if (sfilter != NULL)
         {
-            *psz_eof = '\0';
-
-            /* FIXME : multiple port ! */
-            i_port = atoi( psz_parse );
+            char psz_source_ip[256];
+            unsigned ipv;
 
-            if( i_port <= 0 || i_port >= 65536 )
+            if (sscanf (sfilter, " incl IN IP%u %*s %255s ", &ipv,
+                        psz_source_ip) == 2)
             {
-                msg_Warn( p_obj, "invalid transport port %i", i_port );
+                /* According to RFC4570, FQDNs can be used for source-filters,
+                * but -seriously- this is impractical */
+                switch (ipv)
+                {
+#ifdef AF_INET6
+                    case 6:
+                    {
+                        struct in6_addr addr;
+                        if ((inet_pton (AF_INET6, psz_source_ip, &addr) > 0)
+                        && (inet_ntop (AF_INET6, &addr, psz_source + 1,
+                                        sizeof (psz_source) - 2) != NULL))
+                        {
+                            psz_source[0] = '[';
+                            psz_source[strlen (psz_source)] = ']';
+                        }
+                        break;
+                    }
+#endif
+                    case 4:
+                    {
+                        struct in_addr addr;
+                        if ((inet_pton (AF_INET, psz_source_ip, &addr) > 0)
+                        && (inet_ntop (AF_INET, &addr, psz_source,
+                                        sizeof (psz_source)) == NULL))
+                            *psz_source = '\0';
+                        break;
+                    }
+                }
             }
-
-            psz_parse = psz_eof + 1;
-        }
-        else
-        {
-            msg_Warn( p_obj, "unable to parse m field (2)");
-            FREE( psz_uri );
-            return VLC_EGENERIC;
         }
 
-        psz_eof = strchr( psz_parse, ' ' );
+        if (asprintf (&p_sdp->psz_uri, "%s://%s@%s:%i", vlc_proto, psz_source,
+                     host, port) == -1)
+            return VLC_ENOMEM;
+    }
 
-        if( psz_eof )
-        {
-            *psz_eof = '\0';
-            psz_proto = strdup( psz_parse );
+    return VLC_SUCCESS;
+}
 
-            psz_parse = psz_eof + 1;
-            p_sdp->i_media_type = atoi( psz_parse );
-            
-        }
-        else
-        {
-            msg_Dbg( p_obj, "incorrect m field, %s", p_sdp->psz_media );
-            p_sdp->i_media_type = 33;
-            psz_proto = strdup( psz_parse );
-        }
-    }
 
-    if( psz_proto && !strncmp( psz_proto, "RTP/AVP", 7 ) )
-    {
-        free( psz_proto );
-        psz_proto = strdup( "rtp" );
-    }
-    if( psz_proto && !strncasecmp( psz_proto, "UDP", 3 ) )
-    {
-        free( psz_proto );
-        psz_proto = strdup( "udp" );
-    }
+static int ParseSDPConnection (const char *str, struct sockaddr_storage *addr,
+                               socklen_t *addrlen, unsigned *number)
+{
+    char host[60];
+    unsigned fam, n1, n2;
 
-    /* FIXME: HTTP support */
+    int res = sscanf (str, "IN IP%u %59[^/]/%u/%u", &fam, host, &n1, &n2);
+    if (res < 2)
+        return -1;
 
-    if( i_port == 0 )
+    switch (fam)
     {
-        i_port = 1234;
-    }
+#ifdef AF_INET6
+        case 6:
+            addr->ss_family = AF_INET6;
+# ifdef HAVE_SA_LEN
+            addr->ss_len =
+# endif
+           *addrlen = sizeof (struct sockaddr_in6);
 
-    if( ismult( psz_uri ) )
-    {
-        asprintf( &p_sdp->psz_uri, "%s://@%s:%i", psz_proto, psz_uri, i_port );
-    }
-    else
-    {
-        asprintf( &p_sdp->psz_uri, "%s://%s:%i", psz_proto, psz_uri, i_port );
+            if (inet_pton (AF_INET6, host,
+                           &((struct sockaddr_in6 *)addr)->sin6_addr) <= 0)
+                return -1;
+
+            *number = (res >= 3) ? n1 : 1;
+            break;
+#endif
+
+        case 4:
+            addr->ss_family = AF_INET;
+# ifdef HAVE_SA_LEN
+            addr->ss_len =
+# endif
+           *addrlen = sizeof (struct sockaddr_in);
+
+            if (inet_pton (AF_INET, host,
+                           &((struct sockaddr_in *)addr)->sin_addr) <= 0)
+                return -1;
+
+            *number = (res >= 4) ? n2 : 1;
+            break;
+
+        default:
+            return -1;
     }
-    FREE( psz_uri );
-    FREE( psz_proto );
-    return VLC_SUCCESS;
+    return 0;
 }
 
+
 /***********************************************************************
  * ParseSDP : SDP parsing
  * *********************************************************************
  * Validate SDP and parse all fields
  ***********************************************************************/
-static sdp_t *  ParseSDP( vlc_object_t *p_obj, char* psz_sdp )
+static sdp_t *ParseSDP (vlc_object_t *p_obj, const char *psz_sdp)
 {
-    sdp_t *p_sdp;
-    vlc_bool_t b_invalid = VLC_FALSE;
-    vlc_bool_t b_end = VLC_FALSE;
     if( psz_sdp == NULL )
-    {
         return NULL;
-    }
 
-    if( psz_sdp[0] != 'v' || psz_sdp[1] != '=' )
-    {
-        msg_Warn( p_obj, "Bad packet" );
+    sdp_t *p_sdp = calloc (1, sizeof (*p_sdp));
+    if (p_sdp == NULL)
         return NULL;
-    }
 
-    p_sdp = (sdp_t *)malloc( sizeof( sdp_t ) );
+    char expect = 'V';
+    struct sockaddr_storage glob_addr;
+    memset (&glob_addr, 0, sizeof (glob_addr));
+    socklen_t glob_len = 0;
+    unsigned glob_count = 1;
+    int port = 0;
 
-    p_sdp->psz_sdp = strdup( psz_sdp );
-
-    p_sdp->psz_sessionname = NULL;
-    p_sdp->psz_media       = NULL;
-    p_sdp->psz_connection  = NULL;
-    p_sdp->psz_uri         = NULL;
-    p_sdp->psz_address     = NULL;
-    p_sdp->psz_address_type= NULL;
-
-    p_sdp->i_media         = 0;
-    p_sdp->i_attributes    = 0;
-    p_sdp->pp_attributes   = NULL;
-
-    while( *psz_sdp != '\0' && b_end == VLC_FALSE  )
+    /* TODO: use iconv and charset attribute instead of EnsureUTF8 */
+    while (*psz_sdp)
     {
-        char *psz_eol;
-        char *psz_eof;
-        char *psz_parse;
-        char *psz_sess_id;
+        /* Extract one line */
+        char *eol = strchr (psz_sdp, '\n');
+        size_t linelen = eol ? (size_t)(eol - psz_sdp) : strlen (psz_sdp);
+        char line[linelen + 1];
+        memcpy (line, psz_sdp, linelen);
+        line[linelen] = '\0';
 
-        while( *psz_sdp == '\r' || *psz_sdp == '\n' ||
-               *psz_sdp == ' ' || *psz_sdp == '\t' )
-        {
-            psz_sdp++;
-        }
+        psz_sdp += linelen + 1;
 
-        if( ( psz_eol = strchr( psz_sdp, '\n' ) ) == NULL )
-        {
-            psz_eol = psz_sdp + strlen( psz_sdp );
-            b_end = VLC_TRUE;
-        }
-        if( psz_eol > psz_sdp && *( psz_eol - 1 ) == '\r' )
+        /* Remove carriage return if present */
+        eol = strchr (line, '\r');
+        if (eol != NULL)
         {
-            psz_eol--;
+            linelen = eol - line;
+            line[linelen] = '\0';
         }
 
-        if( psz_eol <= psz_sdp )
+        /* Validate line */
+        char cat = line[0], *data = line + 2;
+        if (!cat || (strchr ("vosiuepcbtrzkam", cat) == NULL))
         {
-            break;
+            /* MUST ignore SDP with unknown line type */
+            msg_Dbg (p_obj, "unknown SDP line type: 0x%02x", (int)cat);
+            goto error;
         }
-        *psz_eol++ = '\0';
-
-        /* no space allowed between fields */
-        if( psz_sdp[1] != '=' )
+        if (line[1] != '=')
         {
-            msg_Warn( p_obj, "invalid packet" ) ;
-            /* MEMLEAK ! */
-            return NULL;
+            msg_Dbg (p_obj, "invalid SDP line: %s", line);
+            goto error;
         }
 
-        /* Now parse each line */
-        switch( psz_sdp[0] )
+        assert (linelen >= 2);
+
+        /* SDP parsing state machine
+         * We INTERNALLY use uppercase for session, lowercase for media
+         */
+        switch (expect)
         {
-            case( 'v' ):
-                break;
-            case( 's' ):
-                p_sdp->psz_sessionname = strdup( &psz_sdp[2] );
+            /* Session description */
+            case 'V':
+                expect = 'O';
+                if (cat != 'v')
+                {
+                    msg_Dbg (p_obj, "missing SDP version");
+                    goto error;
+                }
+                if (strcmp (data, "0"))
+                {
+                    msg_Dbg (p_obj, "unknown SDP version: %s", data);
+                    goto error;
+                }
                 break;
-            case ( 'o' ):
-            {
-                int i_field = 0;
-                /* o field is <username> <session id> <version>
-                 *  <network type> <address type> <address> */
-
-#define GET_FIELD( store ) \
-                psz_eof = strchr( psz_parse, ' ' ); \
-                if( psz_eof ) \
-                { \
-                    *psz_eof=0; store = strdup( psz_parse ); \
-                } \
-                else \
-                { \
-                    if( i_field != 5 ) \
-                    { \
-                        b_invalid = VLC_TRUE; break; \
-                    } \
-                    else \
-                    { \
-                        store = strdup( psz_parse ); \
-                    } \
-                }; \
-                psz_parse = psz_eof + 1; i_field++;
-
-
-                psz_parse = &psz_sdp[2];
-                GET_FIELD( p_sdp->psz_username );
-                GET_FIELD( psz_sess_id );
-
-                p_sdp->i_session_id = atoll( psz_sess_id );
-
-                FREE( psz_sess_id );
-
-                GET_FIELD( psz_sess_id );
-                FREE( psz_sess_id );
-
-                GET_FIELD( p_sdp->psz_network_type );
-                GET_FIELD( p_sdp->psz_address_type );
-                GET_FIELD( p_sdp->psz_address );
 
-                break;
-            }
-            case( 'i' ):
-            case( 'u' ):
-            case( 'e' ):
-            case( 'p' ):
-            case( 't' ):
-            case( 'r' ):
-                break;
-            case( 'a' ): /* attribute */
+            case 'O':
             {
-                char *psz_eon = strchr( &psz_sdp[2], ':' );
-                attribute_t *p_attr = malloc( sizeof( attribute_t ) );
-
-                /* Attribute with value */
-                if( psz_eon )
+                expect = 'S';
+                if (cat != 'o')
                 {
-                    *psz_eon++ = '\0';
-
-                    p_attr->psz_field = strdup( &psz_sdp[2] );
-                    p_attr->psz_value = strdup( psz_eon );
+                    msg_Dbg (p_obj, "missing SDP originator");
+                    goto error;
                 }
-                else /* Attribute without value */
+
+                if ((sscanf (data, "%63s "I64Fu" "I64Fu" IN IP%u %1023s",
+                             p_sdp->username, &p_sdp->session_id,
+                             &p_sdp->session_version, &p_sdp->orig_ip_version,
+                             p_sdp->orig_host) != 5)
+                 || ((p_sdp->orig_ip_version != 4)
+                  && (p_sdp->orig_ip_version != 6)))
                 {
-                    p_attr->psz_field = strdup( &psz_sdp[2] );
-                    p_attr->psz_value = NULL;
+                    msg_Dbg (p_obj, "SDP origin not supported: %s\n", data);
+                    /* Or maybe out-of-range, but this looks suspicious */
+                    return NULL;
                 }
-
-                TAB_APPEND( p_sdp->i_attributes, p_sdp->pp_attributes, p_attr );
+                EnsureUTF8 (p_sdp->orig_host);
                 break;
             }
 
-            case( 'm' ): /* Media announcement */
+            case 'S':
             {
-                /* If we have several medias, we pass the announcement to
-                 * LIVE.COM, so just count them */
-                p_sdp->i_media++;
-                if( p_sdp->i_media == 1 )
+                expect = 'I';
+                if ((cat != 's') || !*data)
                 {
-                    p_sdp->psz_media = strdup( &psz_sdp[2] );
+                    /* MUST be present AND non-empty */
+                    msg_Dbg (p_obj, "missing SDP session name");
+                    goto error;
                 }
+                assert (p_sdp->psz_sessionname == NULL); // no memleak here
+                p_sdp->psz_sessionname = strdup (data);
+                EnsureUTF8 (p_sdp->psz_sessionname);
+                if (p_sdp->psz_sessionname == NULL)
+                    goto error;
                 break;
             }
 
-            case( 'c' ):
-            {
-                if( p_sdp->i_media > 1 )
+            case 'I':
+                expect = 'U';
+                if (cat == 'i')
                     break;
-
-                p_sdp->psz_connection = strdup( &psz_sdp[2] );
+            case 'U':
+                expect = 'E';
+                if (cat == 'u')
+                    break;
+            case 'E':
+                expect = 'E';
+                if (cat == 'e')
+                    break;
+            case 'P':
+                expect = 'P';
+                if (cat == 'p')
+                    break;
+            case 'C':
+                expect = 'B';
+                if (cat == 'c')
+                {
+                    if (ParseSDPConnection (data, &glob_addr, &glob_len,
+                                            &glob_count))
+                    {
+                        msg_Dbg (p_obj, "SDP connection infos not supported: "
+                                 "%s", data);
+                        goto error;
+                    }
+                    break;
+                }
+            case 'B':
+                assert (expect == 'B');
+                if (cat == 'b')
+                    break;
+            case 'T':
+                expect = 'R';
+                if (cat != 't')
+                {
+                    msg_Dbg (p_obj, "missing SDP time description");
+                    goto error;
+                }
                 break;
-            }
-
-            default:
-               break;
-        }
 
-        if( b_invalid )
-        {
-            FreeSDP( p_sdp );
-            return NULL;
-        }
+            case 'R':
+                if ((cat == 't') || (cat == 'r'))
+                    break;
 
-        psz_sdp = psz_eol;
-    }
+            case 'Z':
+                expect = 'K';
+                if (cat == 'z')
+                    break;
+            case 'K':
+                expect = 'A';
+                if (cat == 'k')
+                    break;
+            case 'A':
+                //expect = 'A';
+                if (cat == 'a')
+                {
+                    attribute_t *p_attr = MakeAttribute (data);
+                    TAB_APPEND( p_sdp->i_attributes, p_sdp->pp_attributes, p_attr );
+                    break;
+                }
 
-    return p_sdp;
-}
+            /* Media description */
+            case 'm':
+            media:
+            {
+                expect = 'i';
+                if (cat != 'm')
+                {
+                    msg_Dbg (p_obj, "missing SDP media description");
+                    goto error;
+                }
+                struct sdp_media_t *m;
+                m = realloc (p_sdp->mediav, (p_sdp->mediac + 1) * sizeof (*m));
+                if (m == NULL)
+                    goto error;
+
+                p_sdp->mediav = m;
+                m += p_sdp->mediac;
+                p_sdp->mediac++;
+
+                memset (m, 0, sizeof (*m));
+                memcpy (&m->addr, &glob_addr, m->addrlen = glob_len);
+                m->n_addr = glob_count;
+
+                /* TODO: remember media type (if we need multiple medias) */
+                data = strchr (data, ' ');
+                if (data == NULL)
+                {
+                    msg_Dbg (p_obj, "missing SDP media port");
+                    goto error;
+                }
+                port = atoi (++data);
+                if (port <= 0 || port >= 65536)
+                {
+                    msg_Dbg (p_obj, "invalid transport port %d", port);
+                    goto error;
+                }
+                net_SetPort ((struct sockaddr *)&m->addr, htons (port));
 
+                data = strchr (data, ' ');
+                if (data == NULL)
+                {
+                    msg_Dbg (p_obj, "missing SDP media format");
+                    goto error;
+                }
+                m->fmt = strdup (++data);
+                if (m->fmt == NULL)
+                    goto error;
 
-static char *convert_from_utf8( struct services_discovery_t *p_sd,
-                                char *psz_unicode )
-{
-    char *psz_local, *psz_in, *psz_out;
-    size_t ret, i_in, i_out;
-    vlc_bool_t b_warn = VLC_FALSE;
+                break;
+            }
+            case 'i':
+                expect = 'c';
+                if (cat == 'i')
+                    break;
+            case 'c':
+                expect = 'b';
+                if (cat == 'c')
+                {
+                    struct sdp_media_t *m = p_sdp->mediav + p_sdp->mediac - 1;
+                    if (ParseSDPConnection (data, &m->addr, &m->addrlen,
+                                            &m->n_addr))
+                    {
+                        msg_Dbg (p_obj, "SDP connection infos not supported: "
+                                 "%s", data);
+                        goto error;
+                    }
+                    net_SetPort ((struct sockaddr *)&m->addr, htons (port));
+                    break;
+                }
+            case 'b':
+                expect = 'b';
+                if (cat == 'b')
+                    break;
+            case 'k':
+                expect = 'a';
+                if (cat == 'k')
+                    break;
+            case 'a':
+                assert (expect == 'a');
+                if (cat == 'a')
+                {
+                    attribute_t *p_attr = MakeAttribute (data);
+                    if (p_attr == NULL)
+                        goto error;
 
-    if( psz_unicode == NULL )
-        return NULL;
+                    TAB_APPEND (p_sdp->mediav[p_sdp->mediac - 1].i_attributes,
+                                p_sdp->mediav[p_sdp->mediac - 1].pp_attributes, p_attr);
+                    break;
+                }
 
-    psz_in = psz_unicode;
-    i_in = strlen( psz_unicode );
+                if (cat == 'm')
+                    goto media;
 
-#ifndef MB_CUR_MAX
-    i_out = 6 * i_in;
-#else
-    i_out = MB_CUR_MAX * i_in;
-#endif
-    psz_local = malloc( i_out + 1 );
-    if( psz_local == NULL )
-        return NULL;
-    psz_out = psz_local;
+                if (cat != 'm')
+                {
+                    msg_Dbg (p_obj, "unexpected SDP line: 0x%02x", (int)cat);
+                    goto error;
+                }
+                break;
 
-    do
-    {
-        ret = vlc_iconv( p_sd->p_sys->iconvHandle,
-                         &psz_in, &i_in, &psz_out, &i_out);
-        if( i_in )
-        {
-            *psz_in = '?';
-            b_warn = VLC_TRUE;
-        }
-        else
-        if( ret == (size_t)(-1) )
-        {
-            msg_Err( p_sd, "character conversion failure : %s",
-                     strerror( errno ) );
-            free( psz_local );
-            return NULL;
+            default:
+                msg_Err (p_obj, "*** BUG in SDP parser! ***");
+                goto error;
         }
     }
-    while( i_in );
-
-    if( b_warn )
-        msg_Warn( p_sd, "in \"%s\" : %s", psz_unicode, 
-                  strerror( errno ) );
-
-    *psz_out = '\0';
-    return psz_local;
-}
-
 
-/***********************************************************************
- * ismult: returns true if we have a multicast address
- ***********************************************************************/
-static vlc_bool_t ismult( char *psz_uri )
-{
-    char *psz_end;
-    int  i_value;
-
-    /* IPv6 */
-    if( psz_uri[0] == '[')
-    {
-      if( strncasecmp( &psz_uri[1], "FF0" , 3) ||
-          ( !isalnum( psz_uri[1]) && strncasecmp( &psz_uri[2], "FF0" , 3) ) )
-            return( VLC_TRUE );
-        else
-            return( VLC_FALSE );
-    }
-    
-    i_value = strtol( psz_uri, &psz_end, 0 );
-
-    if( *psz_end != '.' ) { return( VLC_FALSE ); }
+    return p_sdp;
 
-    return ( ( i_value < 224 ) || ( i_value >= 240 ) ) ? VLC_FALSE : VLC_TRUE;
+error:
+    FreeSDP (p_sdp);
+    return NULL;
 }
 
-static int InitSocket( services_discovery_t *p_sd, char *psz_address,
+static int InitSocket( services_discovery_t *p_sd, const char *psz_address,
                        int i_port )
 {
-    int i_fd = net_OpenUDP( p_sd, psz_address, i_port, "", 0 );
-
-    if( i_fd != -1 )
-    {
-        INSERT_ELEM(  p_sd->p_sys->pi_fd, p_sd->p_sys->i_fd,
-                      p_sd->p_sys->i_fd, i_fd );
-        return VLC_SUCCESS;
-    }
+    int i_fd = net_ListenUDP1 ((vlc_object_t *)p_sd, psz_address, i_port);
+    if (i_fd == -1)
+        return VLC_EGENERIC;
 
-    return VLC_EGENERIC;
+    shutdown( i_fd, SHUT_WR );
+    INSERT_ELEM (p_sd->p_sys->pi_fd, p_sd->p_sys->i_fd,
+                 p_sd->p_sys->i_fd, i_fd);
+    return VLC_SUCCESS;
 }
 
-#ifdef HAVE_ZLIB_H
-static int Decompress( unsigned char *psz_src, unsigned char **_dst, int i_len )
+static int Decompress( const unsigned char *psz_src, unsigned char **_dst, int i_len )
 {
-    int i_result, i_dstsize, n;
-    unsigned char *psz_dst;
+#ifdef HAVE_ZLIB_H
+    int i_result, i_dstsize, n = 0;
+    unsigned char *psz_dst = NULL;
     z_stream d_stream;
 
-    d_stream.zalloc = (alloc_func)0;
-    d_stream.zfree = (free_func)0;
-    d_stream.opaque = (voidpf)0;
+    memset (&d_stream, 0, sizeof (d_stream));
 
     i_result = inflateInit(&d_stream);
     if( i_result != Z_OK )
-    {
-        printf( "inflateInit() failed. Result: %d\n", i_result );
         return( -1 );
-    }
-#if 0
-    p_playlist->pp_items[p_playlist->i_index]->b_autodeletion = VLC_TRUE;
-    i_position = p_playlist->i_index;
 
-    /* Gather the complete sdp file */
-    for( ;; )
-    {
-        int i_read = stream_Read( p_demux->s, &p_sdp[i_sdp], i_sdp_max - i_sdp - 1 );
-#endif
     d_stream.next_in = (Bytef *)psz_src;
     d_stream.avail_in = i_len;
-    n = 0;
-
-    psz_dst = NULL;
 
     do
     {
@@ -1375,7 +1378,7 @@ static int Decompress( unsigned char *psz_src, unsigned char **_dst, int i_len )
         i_result = inflate(&d_stream, Z_NO_FLUSH);
         if( ( i_result != Z_OK ) && ( i_result != Z_STREAM_END ) )
         {
-            printf( "Zlib decompression failed. Result: %d\n", i_result );
+            inflateEnd( &d_stream );
             return( -1 );
         }
     }
@@ -1388,33 +1391,34 @@ static int Decompress( unsigned char *psz_src, unsigned char **_dst, int i_len )
     *_dst = (unsigned char *)realloc( psz_dst, i_dstsize );
 
     return i_dstsize;
-}
+#else
+    (void)psz_src;
+    (void)_dst;
+    (void)i_len;
+    return -1;
 #endif
+}
 
 
 static void FreeSDP( sdp_t *p_sdp )
 {
-    int i;
-    FREE( p_sdp->psz_sdp );
-    FREE( p_sdp->psz_sessionname );
-    FREE( p_sdp->psz_connection );
-    FREE( p_sdp->psz_media );
-    FREE( p_sdp->psz_uri );
-    FREE( p_sdp->psz_username );
-    FREE( p_sdp->psz_network_type );
-
-    FREE( p_sdp->psz_address );
-    FREE( p_sdp->psz_address_type );
-
-    for( i= p_sdp->i_attributes - 1; i >= 0 ; i-- )
+    free( p_sdp->psz_sessionname );
+    free( p_sdp->psz_uri );
+
+    for (unsigned j = 0; j < p_sdp->mediac; j++)
     {
-        struct attribute_t *p_attr = p_sdp->pp_attributes[i];
-        FREE( p_sdp->pp_attributes[i]->psz_field );
-        FREE( p_sdp->pp_attributes[i]->psz_value );
-        REMOVE_ELEM( p_sdp->pp_attributes, p_sdp->i_attributes, i);
-        FREE( p_attr );
+        free (p_sdp->mediav[j].fmt);
+        for (int i = 0; i < p_sdp->mediav[j].i_attributes; i++)
+            FreeAttribute (p_sdp->mediav[j].pp_attributes[i]);
+        free (p_sdp->mediav[j].pp_attributes);
     }
-    free( p_sdp );
+    free (p_sdp->mediav);
+
+    for (int i = 0; i < p_sdp->i_attributes; i++)
+        FreeAttribute (p_sdp->pp_attributes[i]);
+
+    free (p_sdp->pp_attributes);
+    free (p_sdp);
 }
 
 static int RemoveAnnounce( services_discovery_t *p_sd,
@@ -1422,13 +1426,15 @@ static int RemoveAnnounce( services_discovery_t *p_sd,
 {
     int i;
 
-    if( p_announce->p_sdp ) FreeSDP( p_announce->p_sdp );
-
-    if( p_announce->i_item_id > -1 )
+    if( p_announce->p_sdp )
     {
-        playlist_LockDelete( p_sd->p_sys->p_playlist, p_announce->i_item_id );
+        FreeSDP( p_announce->p_sdp );
+        p_announce->p_sdp = NULL;
     }
 
+    if( p_announce->i_input_id > -1 )
+        playlist_DeleteFromInput( pl_Get(p_sd), p_announce->i_input_id, VLC_FALSE );
+
     for( i = 0; i< p_sd->p_sys->i_announces; i++)
     {
         if( p_sd->p_sys->pp_announces[i] == p_announce )
@@ -1447,38 +1453,53 @@ static int RemoveAnnounce( services_discovery_t *p_sd,
 static vlc_bool_t IsSameSession( sdp_t *p_sdp1, sdp_t *p_sdp2 )
 {
     /* A session is identified by
-     * username, session_id, network type, address type and address */
-    if( p_sdp1->psz_username && p_sdp2->psz_username &&
-        p_sdp1->psz_network_type && p_sdp2->psz_network_type &&
-        p_sdp1->psz_address_type && p_sdp2->psz_address_type &&
-        p_sdp1->psz_address &&  p_sdp2->psz_address )
+     * - username,
+     * - session_id,
+     * - network type (which is always IN),
+     * - address type (currently, this means IP version),
+     * - and hostname.
+     */
+    if (strcmp (p_sdp1->username, p_sdp2->username)
+     || (p_sdp1->session_id != p_sdp2->session_id)
+     || (p_sdp1->orig_ip_version != p_sdp2->orig_ip_version)
+     || strcmp (p_sdp1->orig_host, p_sdp2->orig_host))
+        return VLC_FALSE;
+
+    return VLC_TRUE;
+}
+
+
+static inline attribute_t *MakeAttribute (const char *str)
+{
+    attribute_t *a = malloc (sizeof (*a) + strlen (str) + 1);
+    if (a == NULL)
+        return NULL;
+
+    strcpy (a->name, str);
+    EnsureUTF8 (a->name);
+    char *value = strchr (a->name, ':');
+    if (value != NULL)
     {
-        if(!strcmp( p_sdp1->psz_username , p_sdp2->psz_username ) &&
-           !strcmp( p_sdp1->psz_network_type , p_sdp2->psz_network_type ) &&
-           !strcmp( p_sdp1->psz_address_type , p_sdp2->psz_address_type ) &&
-           !strcmp( p_sdp1->psz_address , p_sdp2->psz_address ) &&
-           p_sdp1->i_session_id == p_sdp2->i_session_id )
-        {
-            return VLC_TRUE;
-        }
-        else
-        {
-            return VLC_FALSE;
-        }
+        *value++ = '\0';
+        a->value = value;
     }
     else
-    {
-        return VLC_FALSE;
-    }
+        a->value = "";
+    return a;
 }
 
 
-static void CacheLoad( services_discovery_t *p_sd )
+static const char *GetAttribute (attribute_t **tab, unsigned n,
+                                 const char *name)
 {
-    msg_Warn( p_sd, "Cache not implemented") ;
+    for (unsigned i = 0; i < n; i++)
+        if (strcasecmp (tab[i]->name, name) == 0)
+            return tab[i]->value;
+    return NULL;
 }
 
-static void CacheSave( services_discovery_t *p_sd )
+
+static inline void FreeAttribute (attribute_t *a)
 {
-    msg_Warn( p_sd, "Cache not implemented") ;
+    free (a);
 }