[vlc-devel] [PATCH] kms: avoid printing NULL

Thomas Guillem thomas at gllm.fr
Fri Aug 23 14:34:59 CEST 2019


LGTM

On Fri, Aug 23, 2019, at 12:38, Alexandre Janniaux wrote:
> It fixes the following warning:
> ā€˜%4sā€™ directive argument is null [-Wformat-overflow=]
> ---
>  modules/video_output/kms.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/modules/video_output/kms.c b/modules/video_output/kms.c
> index 025a4a2f79..7626254b65 100644
> --- a/modules/video_output/kms.c
> +++ b/modules/video_output/kms.c
> @@ -732,7 +732,7 @@ static int Open(vout_display_t *vd, const 
> vout_display_cfg_t *cfg,
>          chroma = NULL;
>      } else {
>          sys->vlc_fourcc = fmtp->i_chroma;
> -        msg_Dbg(vd, "Chroma %4s invalid, using default", chroma);
> +        msg_Dbg(vd, "Chroma not defined, using default");
>      }
> 
>      chroma = var_InheritString(vd, "kms-drm-chroma");
> --
> 2.23.0
> _______________________________________________
> vlc-devel mailing list
> To unsubscribe or modify your subscription options:
> https://mailman.videolan.org/listinfo/vlc-devel


More information about the vlc-devel mailing list