[mythtv-users] 0.26 -> 0.27 UPnP Failure?

[Nicolas Riendeau]

Hi!

On 2013-10-07 7:10 AM, John Pilkington wrote:
> On 06/10/13 20:43, Nigel Jewell wrote:
>> On 06/10/2013 20:20, Bill Meek wrote:
>>> You could try running:
>>>
>>>     mythbackend --setverbose upnp
>>
>> Thanks for this Bill.  It got me a little closer.  I've just got to work
>> out what is causing the 404s I guess.

A MythTV dev, Stuart Morgan, analyzed the communication and found out it 
was caused by improper commands sent by VLC (actually libupnp which they 
use).

>>
>> 2013-10-06 20:41:14.713689 I [1316/15251] HttpServer120
>> httprequest.cpp:1366 (ExtractMethodFromURL) - ExtractMethodFromURL(end)
>> : getDeviceDesc : /

This is OK.

>> 2013-10-06 20:41:14.723720 I [1316/15251] HttpServer120
>> httprequest.cpp:1366 (ExtractMethodFromURL) - ExtractMethodFromURL(end)
>> : CDS_EventDesc : /

This is not...

>> httpserver.cpp:185 (DelegateRequest) - m_sBaseUrl: /CDS_Controlsc
>> 2013-10-06 20:41:14.733150 I [1316/15252] HttpServer120
>> httprequest.cpp:764 (FormatFileResponse) -
>> HTTPRequest::FormatFileResponse() - cannot find file

And neither is this....

Neither CDS_EventDesc nor CDS_Controlsc are acceptable, it's supposed to 
be CDS_Event and CDS_Control.

Now let's put them one over the other:

GetDeviceDesc
CDS_EventDesc
CDS_Controlsc

They are sending the last characters of the previous one which corrupts 
them (either because they haven't cleared their buffer properly or are 
always sending the same numbers of characters not taking into account 
that they are not all of the same length).

> vlc versions have changed too, of course.

and this is where problem comes from...

I do believe Stuart intends to report this to the libupnp guys... When 
will it be fixed in VLC, I have no idea...

Have a nice day!

Nicolas