[Intel-gfx] [PATCH i-g-t] lib/igt_core: Handle glib errors correctly to avoid stderr spew

Paul Kocialkowski paul.kocialkowski at linux.intel.com
Thu Jul 20 07:47:18 UTC 2017


On Thu, 2017-07-20 at 10:45 +0300, Paul Kocialkowski wrote:
> This adds the required error clean/free calls after calling
> configuration parsing functions. In addition to properly handling
> memory,
> this avoids glib spewing out error messages on stderr, which breaks
> the
> whole CI.

Fixes: ee31e0b5 ("Introduce common frame dumping configuration...")

> Signed-off-by: Paul Kocialkowski <paul.kocialkowski at linux.intel.com>
> ---
>  lib/igt_core.c | 11 ++++++++++-
>  1 file changed, 10 insertions(+), 1 deletion(-)
> 
> diff --git a/lib/igt_core.c b/lib/igt_core.c
> index 028ef6bd..03d9a5bf 100644
> --- a/lib/igt_core.c
> +++ b/lib/igt_core.c
> @@ -637,10 +637,16 @@ static int config_parse(void)
>  							"FrameDumpPat
> h",
>  							&error);
>  
> +	g_clear_error(&error);
> +
>  	rc = g_key_file_get_integer(igt_key_file, "DUT",
> "SuspendResumeDelay",
>  				    &error);
> -	if (error && error->code == G_KEY_FILE_ERROR_INVALID_VALUE)
> +	if (error && error->code == G_KEY_FILE_ERROR_INVALID_VALUE) {
> +		g_error_free(error);
>  		return -2;
> +	}
> +
> +	g_clear_error(&error);
>  
>  	if (rc != 0)
>  		igt_set_autoresume_delay(rc);
> @@ -809,6 +815,7 @@ static int common_init(int *argc, char **argv,
>  	ret = g_key_file_load_from_file(igt_key_file, key_file_loc,
>  					G_KEY_FILE_NONE, &error);
>  	if (error && error->code == G_KEY_FILE_ERROR) {
> +		g_error_free(error);
>  		g_key_file_free(igt_key_file);
>  		igt_key_file = NULL;
>  		ret = -2;
> @@ -816,6 +823,8 @@ static int common_init(int *argc, char **argv,
>  		goto out;
>  	}
>  
> +	g_clear_error(&error);
> +
>  	ret = config_parse();
>  #endif
>  
-- 
Paul Kocialkowski <paul.kocialkowski at linux.intel.com>
Intel Finland Oy - BIC 0357606-4 - Westendinkatu 7, 02160 Espoo, Finland


More information about the Intel-gfx mailing list