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

Nicolas George george at nsup.org
Mon Oct 30 16:16:27 EET 2023


Leo Izen (12023-10-30):
> Also, fwiw, info->assoc_language is the name of a language. It shouldn't
> actually ever be 63 characters long for real streams in the wild, and
> truncating it is just a memory-safety precaution. It's not a bug.


It SHOULD not, but it CAN be, and properly reporting the error to the
user is important.

It is just about adding an if and return error.

Regards,

-- 
  Nicolas George


More information about the ffmpeg-devel mailing list