Opened 7 years ago
Last modified 8 months ago
#12011 accepted Patch - Bug Fix
upnp NOTIFY messages are not sent
Reported by: | Owned by: | stuartm | |
---|---|---|---|
Priority: | minor | Milestone: | needs_triage |
Component: | MythTV - UPnP | Version: | Master Head |
Severity: | medium | Keywords: | soap upnp |
Cc: | Ticket locked: | no |
Description
Myth is only visible to clients who actively request are you there.
libs/libmythupnp/mmulticastsocketdevice.h does not initialize m_local_addresses during instantiation and is never set anywhere else.
The result is that no interfaces are found and no messages sent. This behavior can be seen with any client that does not actively request (PS2's, Samsungs, some LG's). This fix puts the initialize in the header QList<QHostAddress> m_local_addresses = QNetworkInterface::allAddresses(); It could be put in the routines that use it with a "m_local_addresses = QNetworkInterface::allAddresses();" What I havent checked yet is that this message is redone periodical in case of router failure or the max-age expiring in the router.
build is v0.28-pre-678-g6530ac6-dirty
Attachments (1)
Change History (4)
Changed 7 years ago by
Attachment: | multicast.patch added |
---|
comment:1 Changed 6 years ago by
Type: | Bug Report - General → Patch - Bug Fix |
---|
comment:2 Changed 6 years ago by
Owner: | changed from dblain to stuartm |
---|---|
Status: | new → accepted |
Version: | Unspecified → Master Head |
comment:3 Changed 8 months ago by
Milestone: | unknown → needs_triage |
---|
Patch for fixing upnp notify messages