[lttng-dev] [PATCH lttng-tools] Fix: getenv can return null

Jérémie Galarneau jeremie.galarneau at efficios.com
Wed May 1 10:37:56 EDT 2019


Merged in master and stable-2.11.

Thanks!
Jérémie

On Tue, Apr 30, 2019 at 07:09:24PM -0400, Jonathan Rajotte wrote:
> On system with LANG not defined getenv will return null.
> 
> An example of such system is the lava runner used by ci.lttng.org.
> 
> https://ci.lttng.org/view/System%20Tests/
> 
> Signed-off-by: Jonathan Rajotte <jonathan.rajotte-julien at efficios.com>
> ---
>  src/common/time.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/src/common/time.c b/src/common/time.c
> index 5519e3ab4..a01c16df5 100644
> --- a/src/common/time.c
> +++ b/src/common/time.c
> @@ -76,7 +76,7 @@ void __attribute__((constructor)) init_locale_utf8_support(void)
>  
>  	if (program_locale && strstr(program_locale, "utf8")) {
>  		utf8_output_supported = true;
> -	} else if (strstr(lang, "utf8")) {
> +	} else if (lang && strstr(lang, "utf8")) {
>  		utf8_output_supported = true;
>  	}
>  }
> -- 
> 2.17.1
> 


More information about the lttng-dev mailing list