From: Emil Velikov <[email protected]> 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 <[email protected]> Cc: Axel Davy <[email protected]> Cc: [email protected]> Signed-off-by: Emil Velikov <[email protected]> --- 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 [email protected] https://lists.freedesktop.org/mailman/listinfo/mesa-dev
