[Mesa-dev] [PATCH 0/4] CLOEXEC fix-ups

[Kristian Høgsberg]

On Wed, Jun 17, 2015 at 9:28 AM, Derek Foreman <derekf at 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

Reviewed-by: Kristian Høgsberg <krh at bitplanet.net>

>  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(-)
>
> --
> 2.1.4
>
> _______________________________________________
> mesa-dev mailing list
> mesa-dev at lists.freedesktop.org
> http://lists.freedesktop.org/mailman/listinfo/mesa-dev