Opened 19 years ago

Last modified 19 years ago

#1783 closed patch

Memory Leak - StreamDataListener Handling — at Version 1

Reported by: Stuart Auchterlonie Owned by: danielk
Priority: minor Milestone: 0.20
Component: mythtv Version: head
Severity: medium Keywords: memory leak
Cc: Ticket locked: no

Description (last modified by danielk)

Found a memory leak in the stream data Listener handling in siparser.

Patch attached.

Change History (3)

by Stuart Auchterlonie, 19 years ago

Attachment: streamdata-memory-leak.diff added

Patch to fix memory leak in streamdata listener handling

comment:1 by danielk, 19 years ago

Description: modified (diff)
Type: patchdefect

This patch won't do since the SIParser is going away and this would segfault when both SIParser and DVBRecorder delete the same object.

But the bug is valid and I'll write a proper fix within the next few weeks if no one beats me to it.

The fix will either transfer ownership of the stream data to TVRec, or just delete the stream data when we teardown a DTVSignalMonitor with a stream data that a recorder does not reference.

by Janne <janne-mythtv@…>, 19 years ago

Attachment: streamdata_leak_fix.2.diff added
Note: See TracTickets for help on using tickets.