From: Emil Velikov <emil.veli...@collabora.com> As the newly introduced comment says: The pipe loader takes ownership of the fd
Thus, there's no need to close it again. Cc: Patrick Rudolph <s...@das-labor.org> Cc: Axel Davy <davyax...@gmail.com> Cc: mesa-sta...@lists.freedesktop.org> Signed-off-by: Emil Velikov <emil.veli...@collabora.com> --- src/gallium/targets/d3dadapter9/drm.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/src/gallium/targets/d3dadapter9/drm.c b/src/gallium/targets/d3dadapter9/drm.c index 85b3e10633e..a2a36dbbda9 100644 --- a/src/gallium/targets/d3dadapter9/drm.c +++ b/src/gallium/targets/d3dadapter9/drm.c @@ -107,7 +107,7 @@ drm_destroy( struct d3dadapter9_context *ctx ) if (drm->dev) pipe_loader_release(&drm->dev, 1); - close(drm->fd); + /* The pipe loader takes ownership of the fd */ FREE(ctx); } -- 2.18.0 _______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org https://lists.freedesktop.org/mailman/listinfo/mesa-dev