Opened 12 years ago

Closed 11 years ago

Last modified 11 years ago

#11111 closed Patch - Bug Fix (Fixed)

Fix UUID generation in UPNP

Reported by: Rune Petersen <rune@…> Owned by: stuartm
Priority: minor Milestone: 0.27
Component: MythTV - UPnP Version: Master Head
Severity: medium Keywords: upnp fix udn
Cc: Ticket locked: no

Description

currently LookupUDN() will generate invalid UUID which is then used to generate a invalid UDN. this may cause problems with UPNP devices that parses the UDN - resulting in failed discovery.

Currently LookupUDN() generates UUID like this:

"{74c5afe6-181b-46b1-9a4b-ae9e53d0e47"

this results in a UDN looking like this:

"uuid:{74c5afe6-181b-46b1-9a4b-ae9e53d0e47"

The patch changes the code to remove the leading '{' and trailing '}' maiing the UUID look like this:

"74c5afe6-181b-46b1-9a4b-ae9e53d0e479"

and a valid UDN:

"uuid:74c5afe6-181b-46b1-9a4b-ae9e53d0e479"

Attachments (2)

mythtv_upnp_fix_uuid_generation.patch (722 bytes) - added by Rune Petersen <rune@…> 12 years ago.
fix UUID generation - applies to MASTER and 0.25
upnp.txt (238.2 KB) - added by spamme@… 12 years ago.
mythbackend -v upnp

Download all attachments as: .zip

Change History (8)

Changed 12 years ago by Rune Petersen <rune@…>

fix UUID generation - applies to MASTER and 0.25

comment:1 Changed 12 years ago by spamme@…

Nope sorry no change. I even recompiled every thing.

upnp-inspector

does not even acknowledge myth as upnp device even though it is on the same pc as the mythbackend.

comment:2 Changed 12 years ago by spamme@…

<?xml version="1.0" encoding="utf-8"?>
<root xmlns="urn:schemas-upnp-org:device-1-0"  xmlns:mythtv="mythtv.org">
<specVersion>
<major>1</major>
<minor>0</minor>
</specVersion>
<URLBase>http://<Server IP hidden by me>:6544/</URLBase>
<device>
<deviceType>urn:schemas-upnp-org:device:MediaServer:1</deviceType>
<friendlyName>mythdvr: MythTV AV Media Server</friendlyName>
<manufacturer>MythTV</manufacturer>
<modelURL>http://www.mythtv.org/</modelURL>
<modelName>MythTV AV Media Server</modelName>
<manufacturerURL>http://www.mythtv.org/</manufacturerURL>
<modelDescription>MythTV AV Media Server</modelDescription>
<modelNumber>0.26.20120822-1</modelNumber>
<serialNumber>v0.26-rc2-30-g272fb61-dirty</serialNumber>
<presentationURL>/</presentationURL>
<mythtv:X_secure>false</mythtv:X_secure>
<mythtv:X_protocol>75</mythtv:X_protocol>
<dlna:X_DLNADOC xmlns:dlna="urn:schemas-dlna-org:device-1-0">DMS-1.00</dlna:X_DLNADOC>
<UDN>uuid:0705d747-7b55-415d-b2cf-d9517014cb7d</UDN>
<serviceList>
<service>
<serviceType>urn:schemas-upnp-org:service:ConnectionManager:1</serviceType>
<serviceId>urn:upnp-org:serviceId:CMGR_1-0</serviceId>
<SCPDURL>CMGR_Control/GetServDesc</SCPDURL>
<controlURL>CMGR_Control</controlURL>
<eventSubURL>CMGR_Event</eventSubURL>
</service>
<service>
<serviceType>urn:schemas-upnp-org:service:ContentDirectory:1</serviceType>
<serviceId>urn:upnp-org:serviceId:CDS_1-0</serviceId>
<SCPDURL>CDS_Control/GetServDesc</SCPDURL>
<controlURL>CDS_Control</controlURL>
<eventSubURL>CDS_Event</eventSubURL>
</service>
</serviceList>
</device>
</root>

Changed 12 years ago by spamme@…

Attachment: upnp.txt added

mythbackend -v upnp

comment:3 Changed 11 years ago by stuartm

Milestone: unknown0.27
Owner: changed from dblain to stuartm
Status: newaccepted

comment:4 Changed 11 years ago by stuartm

Resolution: Fixed
Status: acceptedclosed

In 6a172c5b70f24cb1daa6123f3a67acd5a3fcd056/mythtv:

Fix UUID. QUuid returns the UUID encapsulated in braces which need to be stripped before it can be used for upnp. Fix #11111

comment:5 Changed 11 years ago by stuartm

Thank you for the patch Rune!

comment:6 Changed 11 years ago by Rune Petersen <rune@…>

Thank you for doing the heavy lifting Stuart

Note: See TracTickets for help on using tickets.