Re: [PATCH weston] logind: actually close fd in launcher_logind_close

2017-07-27 Thread Pekka Paalanen
On Wed, 26 Jul 2017 18:04:19 +0300 Pekka Paalanen wrote: > On Tue, 25 Jul 2017 16:17:36 -0500 > Derek Foreman wrote: > > > We still need to close fds passed to us - or we leak quite a few fds > > on VC switch. > > > > Regression, originally fixed in 8f5acc2f3a29c3831af4ddd6bed57f703c98dc77 > >

Re: [PATCH weston] logind: actually close fd in launcher_logind_close

2017-07-26 Thread Pekka Paalanen
On Tue, 25 Jul 2017 16:17:36 -0500 Derek Foreman wrote: > We still need to close fds passed to us - or we leak quite a few fds > on VC switch. > > Regression, originally fixed in 8f5acc2f3a29c3831af4ddd6bed57f703c98dc77 > and re-broken in commit 72dea06d7952e3ce8dd8057f7106186da4fa2678 > but onl

[PATCH weston] logind: actually close fd in launcher_logind_close

2017-07-25 Thread Derek Foreman
We still need to close fds passed to us - or we leak quite a few fds on VC switch. Regression, originally fixed in 8f5acc2f3a29c3831af4ddd6bed57f703c98dc77 and re-broken in commit 72dea06d7952e3ce8dd8057f7106186da4fa2678 but only for the logind launcher. Signed-off-by: Derek Foreman --- libwest