From: Emil Velikov <emil.velikov@collabora.com>
Date: Wed, 29 Aug 2018 17:13:13 +0000 (+0100)
Subject: st/nine: do not double-close the fd on teardown
X-Git-Url: https://git.libre-soc.org/?a=commitdiff_plain;h=7b8d1b313cd01bb916898d8bb92a566534e37677;p=mesa.git

st/nine: do not double-close the fd on teardown

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 <siro@das-labor.org>
Cc: Axel Davy <davyaxel0@gmail.com>
Cc: mesa-stable@lists.freedesktop.org>
Signed-off-by: Emil Velikov <emil.velikov@collabora.com>
Reviewed-by: Axel Davy <davyaxel0@gmail.com>
---

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);
 }