On 05/12/2016 11:10 AM, Emil Velikov wrote:
On 12 May 2016 at 15:10, Leo Liu <[email protected]> wrote:
fd should be set to -1 only if it got closed by pipe_loader_release.
Signed-off-by: Leo Liu <[email protected]>
---
src/gallium/auxiliary/vl/vl_winsys_dri.c | 5 +++--
1 file changed, 3 insertions(+), 2 deletions(-)
diff --git a/src/gallium/auxiliary/vl/vl_winsys_dri.c
b/src/gallium/auxiliary/vl/vl_winsys_dri.c
index 0136526..4636feb 100644
--- a/src/gallium/auxiliary/vl/vl_winsys_dri.c
+++ b/src/gallium/auxiliary/vl/vl_winsys_dri.c
@@ -427,9 +427,10 @@ vl_dri2_screen_create(Display *display, int screen)
return &scrn->base;
release_pipe:
- if (scrn->base.dev)
+ if (scrn->base.dev) {
pipe_loader_release(&scrn->base.dev, 1);
- fd = -1;
+ fd = -1;
+ }
free_authenticate:
free(authenticate);
close_fd:
+if (fd != -1)
.... close(fd)
Please add a -1 check before the close.
Sure I will add it, commit it to the repo later today.
Thanks,
Leo
Thanks
Emil
_______________________________________________
mesa-dev mailing list
[email protected]
https://lists.freedesktop.org/mailman/listinfo/mesa-dev
