On 17 June 2015 at 17:28, Derek Foreman <der...@osg.samsung.com> wrote: > This series catches a couple of places where we forget to set CLOEXEC on > file descriptors, and makes a helper function for the necessarily ugly > way we have to open fds to make sure CLOEXEC is set. > > Derek Foreman (4): > egl/drm: Duplicate fd with F_DUPFD_CLOEXEC to prevent leak > loader: Rename drm_open_device() to loader_open_device() and share it > glx: Use loader_open_device() helper > egl: Use the loader_open_device() helper to do open with CLOEXEC > > src/egl/drivers/dri2/platform_drm.c | 6 +++--- > src/egl/drivers/dri2/platform_surfaceless.c | 11 +---------- > src/egl/drivers/dri2/platform_wayland.c | 11 +---------- > src/egl/drivers/dri2/platform_x11.c | 12 ++---------- > src/glx/dri2_glx.c | 10 +--------- > src/loader/loader.c | 6 +++--- > src/loader/loader.h | 3 +++ > 7 files changed, 14 insertions(+), 45 deletions(-) Series looks great. Thanks for the help Derek. Reviewed-by: Emil Velikov <emil.l.veli...@gmail.com>
Will give it a couple of days, and barring any objections I'll push this to master. -Emil _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
