On Mon, 2009-03-09 at 15:04 -0300, Johan Hedberg wrote: > 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 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! And the final patch for now. This one should be fixing printer discovery with BlueZ 4.x. Next time it gets broken, could someone please let me know so I don't realise it after 6 months? Cheers