<div dir="auto"><div>Yes, I forgot to patch this part, will send a patch soon.</div><div dir="auto"><br><div class="gmail_quote" dir="auto"><div dir="ltr" class="gmail_attr">On Mon, Apr 12, 2021, 20:00 Tristan Matthews <<a href="mailto:tmatth@videolan.org">tmatth@videolan.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">On Mon, Apr 12, 2021 at 12:51 PM George Vaindirlis<br>
<<a href="mailto:george.vdl1@gmail.com" target="_blank" rel="noreferrer">george.vdl1@gmail.com</a>> wrote:<br>
><br>
> fix warning: ‘%u’ directive output may be truncated writing between 1 and 10 bytes into a region of size 5 [-Wformat-truncation=]<br>
> (Please reject the previous patch from me with the same description, it has a typo in the code)<br>
><br>
> ---<br>
>  modules/access/cdda.c | 2 +-<br>
>  1 file changed, 1 insertion(+), 1 deletion(-)<br>
><br>
> diff --git a/modules/access/cdda.c b/modules/access/cdda.c<br>
> index 9260b833ed..42ec944d12 100644<br>
> --- a/modules/access/cdda.c<br>
> +++ b/modules/access/cdda.c<br>
> @@ -635,7 +635,7 @@ static void AccessGetMeta(stream_t *access, vlc_meta_t *meta)<br>
>              vlc_meta_SetGenre(meta, str);<br>
><br>
>          const unsigned year = cddb_disc_get_year(sys->cddb);<br>
> -        if (year != 0)<br>
> +        if (year != 0 && year < 10000)<br>
>          {<br>
>              char yearbuf[5];<br>
><br>
> --<br>
> 2.30.2<br>
><br>
> _______________________________________________<br>
> vlc-devel mailing list<br>
> To unsubscribe or modify your subscription options:<br>
> <a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a><br>
<br>
Should the same check be done here?<br>
<a href="https://code.videolan.org/videolan/vlc/-/blob/master/modules/access/cdda.c#L829" rel="noreferrer noreferrer" target="_blank">https://code.videolan.org/videolan/vlc/-/blob/master/modules/access/cdda.c#L829</a><br>
<br>
Best,<br>
Tristan<br>
_______________________________________________<br>
vlc-devel mailing list<br>
To unsubscribe or modify your subscription options:<br>
<a href="https://mailman.videolan.org/listinfo/vlc-devel" rel="noreferrer noreferrer" target="_blank">https://mailman.videolan.org/listinfo/vlc-devel</a></blockquote></div></div></div>