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@xxxxxxxxxx> 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 � 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
--
To unsubscribe from this list: send the line "unsubscribe linux-bluetooth" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
