D8887: Fix leaking of FDs requested from logind

Martin Flöser noreply at phabricator.kde.org
Sun Nov 19 11:10:34 UTC 2017


graesslin added a comment.


  In https://phabricator.kde.org/D8887#169520, @fvogt wrote:
  
  > Should this go into Plasma/5.11 (probably even 5.8) as well?
  
  
  5.11 yes! Wayland support is not covered by LTS, so I don't see a need to fix in 5.8.

INLINE COMMENTS

> fvogt wrote in logind.cpp:387
> FD_DUPFD_CLOEXEC is Linux specific. I don't know whether this code path could get used on BSDs.

AFAIK libinput is Linux specific, so this code path can be Linux specific as well.

REPOSITORY
  R108 KWin

REVISION DETAIL
  https://phabricator.kde.org/D8887

To: fvogt, #plasma, graesslin
Cc: kwin, plasma-devel, #kwin, bwowk, ZrenBot, progwolff, lesliezhai, ali-mohamed, hardening, jensreuterberg, abetts, sebas, apol, mart
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://mail.kde.org/pipermail/plasma-devel/attachments/20171119/518c8aae/attachment.html>


More information about the Plasma-devel mailing list