Opened 11 years ago
Closed 3 years ago
#12011 closed Patch - Bug Fix (Trac EOL)
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 (5)
Changed 11 years ago by
| Attachment: | multicast.patch added |
|---|
comment:1 Changed 11 years ago by
| Type: | Bug Report - General → Patch - Bug Fix |
|---|
comment:2 Changed 11 years ago by
| Owner: | changed from dblain to stuartm |
|---|---|
| Status: | new → accepted |
| Version: | Unspecified → Master Head |
comment:3 Changed 5 years ago by
| Milestone: | unknown → needs_triage |
|---|
Patch for fixing upnp notify messages