transmission icon indicating copy to clipboard operation
transmission copied to clipboard

two event objects leaked during shutdown of transmission-daemon

Open icy17 opened this issue 3 years ago • 1 comments

What is the issue?

Potential memory leak in daemon-posix.cc line 73 Calling event_new() without calling event_free() to free the memory will cause a memory leak. Doc says "Deallocate a struct event * returned by event_new()." image

Which application of Transmission?

transmission-daemon

Which version of Transmission?

No response

icy17 avatar Feb 04 '23 10:02 icy17

This is correct and the bug ought to get fixed for cleanliness' sake & to make leak detectors happy -- but FWIW but in practice this is called twice during daemon setup and the allocated events exist for the lifespan of the daemon session, so end users won't see any loss of memory due to this bug.

ckerr avatar Feb 06 '23 05:02 ckerr

Fixed by #5695.

tearfur avatar Jul 14 '23 17:07 tearfur