Opened 8 years ago

Closed 2 months ago

#12011 closed Patch - Bug Fix (Trac EOL)

upnp NOTIFY messages are not sent

Reported by: steve_g@… 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)

multicast.patch​ (615 bytes) - added by steve_g@… 8 years ago.
Patch for fixing upnp notify messages

Download all attachments as: .zip

Change History (5)

Changed 8 years ago by steve_g@…

Attachment: multicast.patch​ added

Patch for fixing upnp notify messages

comment:1 Changed 8 years ago by Karl Egly

Type: Bug Report - General → Patch - Bug Fix

comment:2 Changed 8 years ago by stuartm

Owner: changed from dblain to stuartm
Status: new → accepted
Version: Unspecified → Master Head

comment:3 Changed 2 years ago by Stuart Auchterlonie

Milestone: unknown → needs_triage

comment:4 Changed 2 months ago by Stuart Auchterlonie

Resolution: → Trac EOL
Status: accepted → closed

We have moved all bug tracking to github [1]

If you continue to have this issue, please open a new issue at github, referencing this ticket.

[1] - ​https://github.com/MythTV/mythtv/issues

Note: See TracTickets for help on using tickets.