[vlc-devel] Re: [PATCH] Issues with SDP parsing

[Christophe Mutricy]

On Mon, Mar 06, 06 at 17:05 +0100, Alexander Gall wrote:
> I believe that the "m=" field is mandatory in SDP.  Currently,
[...]
>                  msg_Warn( p_obj, "invalid transport port %i", i_port );
> +                FREE( psz_uri );
> +                return VLC_EGENERIC;
[...]
>          }
> +    } else {
> +      msg_Warn( p_obj, "missing m field" );
> +      FREE( psz_uri );
> +      return VLC_EGENERIC;

If we return a VLC_EGENERIC i think we could go for a msg_Err().

For the m= field i know nothing to sdp myself. but maybe the VLC
implemetation don't relly on it, or just not co;plain when it's absent
to support brocken annonces.

my 2¢

-- 
Xtophe

-- 
This is the vlc-devel mailing-list, see http://www.videolan.org/vlc/
To unsubscribe, please read http://developers.videolan.org/lists.html