Module: Mesa
Branch: master
Commit: 291ae622fd723d5ee2e08bbb8256c51bb5bce008
URL:    
http://cgit.freedesktop.org/mesa/mesa/commit/?id=291ae622fd723d5ee2e08bbb8256c51bb5bce008

Author: Dave Airlie <[email protected]>
Date:   Mon Sep  1 19:44:19 2014 +1000

loader: fds can be 0

Possible resource leak reported by coverity.

Reported-by: Coverity scanner.
Reviewed-by: Kenneth Graunke <[email protected]>
Signed-off-by: Dave Airlie <[email protected]>

---

 src/loader/loader.c |    2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/src/loader/loader.c b/src/loader/loader.c
index 47e1f58..bdd3906 100644
--- a/src/loader/loader.c
+++ b/src/loader/loader.c
@@ -402,7 +402,7 @@ int loader_get_user_preferred_fd(int default_fd, int 
*different_device)
    }
 
    fd = drm_open_device(device_name);
-   if (fd > 0) {
+   if (fd >= 0) {
       close(default_fd);
    } else {
       fd = default_fd;

_______________________________________________
mesa-commit mailing list
[email protected]
http://lists.freedesktop.org/mailman/listinfo/mesa-commit

Reply via email to