Re: [PATCH v2 weston] log: Open log file CLOEXEC so child processes don't get the fd

2015-07-16 Thread Bryce Harrington
On Mon, Jun 08, 2015 at 11:37:31AM -0500, Derek Foreman wrote: > Reviewed-by: Pekka Paalanen > Signed-off-by: Derek Foreman Thanks, pushed: To ssh://git.freedesktop.org/git/wayland/weston 4a8a3a1..6bc33d6 master -> master > --- > > Nested the os_fd_set_cloexec() call so calling weston_log

[PATCH v2 weston] log: Open log file CLOEXEC so child processes don't get the fd

2015-06-08 Thread Derek Foreman
Reviewed-by: Pekka Paalanen Signed-off-by: Derek Foreman --- Nested the os_fd_set_cloexec() call so calling weston_log_file_open() multiple times behaves as expected. shared/os-compatibility.c | 22 ++ shared/os-compatibility.h | 3 +++ src/log.c | 7 +

Re: [PATCH v2 weston] log: Open log file CLOEXEC so child processes don't get the fd

2015-06-08 Thread Pekka Paalanen
On Thu, 4 Jun 2015 11:37:52 -0500 Derek Foreman wrote: > Signed-off-by: Derek Foreman > --- > > Refactor and re-use the code from set_cloexec_or_close() instead of > using fopen "e" > > shared/os-compatibility.c | 22 ++ > shared/os-compatibility.h | 3 +++ > src/log.c

[PATCH v2 weston] log: Open log file CLOEXEC so child processes don't get the fd

2015-06-04 Thread Derek Foreman
Signed-off-by: Derek Foreman --- Refactor and re-use the code from set_cloexec_or_close() instead of using fopen "e" shared/os-compatibility.c | 22 ++ shared/os-compatibility.h | 3 +++ src/log.c | 5 + 3 files changed, 22 insertions(+), 8 deletions(-)