diff options
author | Matt Whitlock <freedesktop@mattwhitlock.name> | 2016-10-01 23:49:45 -0400 |
---|---|---|
committer | Nicolai Hähnle <nicolai.haehnle@amd.com> | 2016-10-04 11:09:03 +0200 |
commit | 42ed8a6c9ccfb8b6b632c3be90fb03d292f53d25 (patch) | |
tree | 40dd06c9d73546f182915bdb13c00653cc5d458e /src/gallium/winsys/radeon | |
parent | ac6064f91898065c8b82255190a0508204d3f88c (diff) | |
download | external_mesa3d-42ed8a6c9ccfb8b6b632c3be90fb03d292f53d25.zip external_mesa3d-42ed8a6c9ccfb8b6b632c3be90fb03d292f53d25.tar.gz external_mesa3d-42ed8a6c9ccfb8b6b632c3be90fb03d292f53d25.tar.bz2 |
gallium/winsys: replace calls to dup(2) with fcntl(F_DUPFD_CLOEXEC)
Without this fix, duplicated file descriptors leak into child processes.
See commit aaac913e901229d11a1894f6aaf646de6b1a542c for one instance
where the same fix was employed.
Cc: <mesa-stable@lists.freedesktop.org>
Signed-off-by: Matt Whitlock <freedesktop@mattwhitlock.name>
Reviewed-by: Nicolai Hähnle <nicolai.haehnle@amd.com>
Diffstat (limited to 'src/gallium/winsys/radeon')
-rw-r--r-- | src/gallium/winsys/radeon/drm/radeon_drm_winsys.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c b/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c index 16e4408..c7ceee2 100644 --- a/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c +++ b/src/gallium/winsys/radeon/drm/radeon_drm_winsys.c @@ -43,6 +43,7 @@ #include <sys/types.h> #include <sys/stat.h> #include <unistd.h> +#include <fcntl.h> #include <radeon_surface.h> #ifndef RADEON_INFO_ACTIVE_CU_COUNT @@ -751,7 +752,7 @@ radeon_drm_winsys_create(int fd, radeon_screen_create_t screen_create) return NULL; } - ws->fd = dup(fd); + ws->fd = fcntl(fd, F_DUPFD_CLOEXEC, 3); if (!do_winsys_init(ws)) goto fail1; |