On Wednesday, 2018-01-10 11:16:41 +0900, Seung-Woo Kim wrote:
> There is warning about ignoring return value of 'asprintf'. Fix to
> check return value of asprintf().
> 
> Signed-off-by: Seung-Woo Kim <sw0312....@samsung.com>

Reviewed-by: Eric Engestrom <eric.engest...@imgtec.com>

Do you have commit access or would you like me to push it for you?

> ---
>  tests/modetest/modetest.c |    5 ++++-
>  1 files changed, 4 insertions(+), 1 deletions(-)
> 
> diff --git a/tests/modetest/modetest.c b/tests/modetest/modetest.c
> index 62d9332..0773bd0 100644
> --- a/tests/modetest/modetest.c
> +++ b/tests/modetest/modetest.c
> @@ -656,10 +656,13 @@ static struct resources *get_resources(struct device 
> *dev)
>       for (i = 0; i < res->res->count_connectors; i++) {
>               struct connector *connector = &res->connectors[i];
>               drmModeConnector *conn = connector->connector;
> +             int num;
>  
> -             asprintf(&connector->name, "%s-%u",
> +             num = asprintf(&connector->name, "%s-%u",
>                        util_lookup_connector_type_name(conn->connector_type),
>                        conn->connector_type_id);
> +             if (num < 0)
> +                     goto error;
>       }
>  
>  #define get_properties(_res, __res, type, Type)                              
>         \
> -- 
> 1.7.4.1
> 
_______________________________________________
dri-devel mailing list
dri-devel@lists.freedesktop.org
https://lists.freedesktop.org/mailman/listinfo/dri-devel

Reply via email to