[Wayland-bugs] [Bug 91872] libinput_unref() frees memory even when clients hold references to devices or seats

[bugzilla-daemon at freedesktop.org]

https://bugs.freedesktop.org/show_bug.cgi?id=91872

--- Comment #1 from Peter Hutterer <peter.hutterer at who-t.net> ---
The options for fixing this issue introduce incompatible behavior, so I'm
closing this as a WONTFIX, this bug is mostly for archival purposes only.

If libinput_unref() changes to only do the work when all references are
accounted for, callers have to release all existing references manually. This
may cause a memory leak in existing callers.
In addition, the call to libinput_suspend() would not happen until the
resources are released, causing wakeups on top of the memory leaks.

If libinput_unref() calls libinput_suspend() but leaves the memory until all
references are released, the behaviour becomes unpredictable, and we get memory
leaks in existing callers.
Requiring an explicit call to libinput_suspend() breaks existing callers.


So really, no good fix here and since the current behavior is also that of
other libraries (e.g. libudev) we'll leave it as-is.

-- 
You are receiving this mail because:
You are the assignee for the bug.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.freedesktop.org/archives/wayland-bugs/attachments/20150904/79b137cb/attachment.html>