[prev in list] [next in list] [prev in thread] [next in thread]
List: freedesktop-igt-dev
Subject: Re: [igt-dev] [PATCH i-g-t] tests/kms_3d: fix uninitialized value
From: Petri Latvala <petri.latvala () intel ! com>
Date: 2019-08-29 11:03:33
Message-ID: 20190829110333.GD4019 () platvala-desk ! ger ! corp ! intel ! com
[Download RAW message or body]
On Thu, Aug 29, 2019 at 01:37:45PM +0300, Simon Ser wrote:
> If the device doesn't have any HDMI connectors, garbage is read from the stack.
>
> Signed-off-by: Simon Ser <simon.ser@intel.com>
Reviewed-by: Petri Latvala <petri.latvala@intel.com>
> ---
> tests/kms_3d.c | 6 +-----
> 1 file changed, 1 insertion(+), 5 deletions(-)
>
> diff --git a/tests/kms_3d.c b/tests/kms_3d.c
> index b970b3b5521c..2622b2d0a6f3 100644
> --- a/tests/kms_3d.c
> +++ b/tests/kms_3d.c
> @@ -30,7 +30,7 @@ igt_simple_main
> {
> int drm_fd;
> drmModeRes *res;
> - drmModeConnector *connector;
> + drmModeConnector *connector = NULL;
> const struct edid *edid;
> int mode_count, connector_id;
>
> @@ -43,14 +43,10 @@ igt_simple_main
>
> /* find an hdmi connector */
> for (int i = 0; i < res->count_connectors; i++) {
> -
> connector = drmModeGetConnectorCurrent(drm_fd, res->connectors[i]);
> -
> if (connector->connector_type == DRM_MODE_CONNECTOR_HDMIA)
> break;
> -
> drmModeFreeConnector(connector);
> -
> connector = NULL;
> }
> igt_require(connector);
> --
> 2.23.0
>
> _______________________________________________
> igt-dev mailing list
> igt-dev@lists.freedesktop.org
> https://lists.freedesktop.org/mailman/listinfo/igt-dev
_______________________________________________
igt-dev mailing list
igt-dev@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/igt-dev
[prev in list] [next in list] [prev in thread] [next in thread]
Configure |
About |
News |
Add a list |
Sponsored by KoreLogic