Re: Broken SDP parsing?

[Johan Hedberg <johan.hedberg@gmail.com>]

Hi Bastien,

On Mon, Mar 09, 2009, Bastien Nocera wrote:
> On Mon, 2009-03-09 at 13:40 -0300, Luiz Augusto von Dentz wrote:
> > Hi Bastien,
> > 
> > On Mon, Mar 9, 2009 at 12:25 PM, Bastien Nocera <hadess@hadess.net> wrote:
> > > On Mon, 2009-03-09 at 11:32 -0300, Luiz Augusto von Dentz wrote:
> > > Problem was sdp_data_alloc() falling back to doing an strlen() on the
> > > string, instead of taking its existing length into account. That would
> > > break any strings with NULLs embedded.
> > >
> > > The attached patch fixes this. I'm not certain that making this public
> > > is useful, but feel free to make it so if you feel it's needed.
> > >
> > > Cheers
> > >
> > 
> > Sounds good to me, I will check with Marcel or Johan if they can get
> > this changes upstream.
> 
> Apparently &#00; isn't an allowed character in XML, so the attached
> patch just swaps it out for a space. Much easier.

Thanks for the patches! Could you still fix a couple of coding syle
issues please:

1. Keep the lines max 79 characters
2. Have the new function start like:
{
        sdp_data_t *d;

        d = sdp_data_alloc_with_length(dtd, value, len);
        if (!d)
                return -1;

Also, please prepare the patches with git format-patch so that I don't
have to play with GIT_AUTHOR_NAME, commit messages, etc. but can use
simply git am.

Johan