Re: [PATCH i-g-t] lib/igt_kms: Fix passing invalid fd to openat(2).

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



On Fri, Jun 03, 2016 at 06:06:19PM +0300, Marius Vlad wrote:
> Introduced by 0e11befe442. openat(2) uses a relative path. Fix by
> passing the correct fd.
> 
> Signed-off-by: Marius Vlad <marius.c.vlad@xxxxxxxxx>
> CC: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>

Yup, At one point it was using the drm_fd to search for the right sysfs
every time,

> ---
>  lib/igt_kms.c | 3 ++-
>  1 file changed, 2 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_kms.c b/lib/igt_kms.c
> index b12a5b3..135c7b8 100644
> --- a/lib/igt_kms.c
> +++ b/lib/igt_kms.c
> @@ -641,7 +641,8 @@ bool kmstest_force_connector(int drm_fd, drmModeConnector *connector,
>  		return false;
>  	}
>  
> -	if (!igt_sysfs_set(drm_fd, path, value)) {
> +	/* use the fd returned by igt_sysfs_open() */

Just call it dir, don't tell people they can treat it as an fd.

> +	if (!igt_sysfs_set(dir, path, value)) {
>  		close(dir);
>  		return false;
>  	}

Preferrably without the comment,
Reviewed-by: Chris Wilson <chris@xxxxxxxxxxxxxxxxxx>
-Chris

-- 
Chris Wilson, Intel Open Source Technology Centre
_______________________________________________
Intel-gfx mailing list
Intel-gfx@xxxxxxxxxxxxxxxxxxxxx
https://lists.freedesktop.org/mailman/listinfo/intel-gfx




[Index of Archives]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]
  Powered by Linux