On Monday, September 01, 2014 07:45:26 PM Dave Airlie wrote:
> Possible resource leak reported by coverity.
>
> Reported-by: Coverity scanner.
> Signed-off-by: Dave Airlie <airl...@redhat.com>
> ---
> 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;
> Reviewed-by: Kenneth Graunke <kenn...@whitecape.org>
signature.asc
Description: This is a digitally signed message part.
_______________________________________________ mesa-dev mailing list mesa-dev@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/mesa-dev
