[FFmpeg-devel] [PATCH] avformat/hls: use av_strlcopy instead of strncpy

[Andreas Rheinhardt]

Leo Izen:
> Avoids a -Wstringop-truncation warning by using av_strlcopy instead of
> strncpy.
> 
> Signed-off-by: Leo Izen <leo.izen at gmail.com>
> ---
>  libavformat/hls.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
> 
> diff --git a/libavformat/hls.c b/libavformat/hls.c
> index f5f549b24d..076f92ecfb 100644
> --- a/libavformat/hls.c
> +++ b/libavformat/hls.c
> @@ -543,7 +543,7 @@ static struct rendition *new_rendition(HLSContext *c, struct rendition_info *inf
>          int langlen = strlen(rend->language);
>          if (langlen < sizeof(rend->language) - 3) {
>              rend->language[langlen] = ',';
> -            strncpy(rend->language + langlen + 1, info->assoc_language,
> +            av_strlcpy(rend->language + langlen + 1, info->assoc_language,
>                      sizeof(rend->language) - langlen - 2);
>          }
>      }

Doesn't this just silence the warning instead of fixing the potential
truncation?

- Andreas