[FFmpeg-cvslog] avformat/aadec: Replace strncpy by av_strlcpy

Andreas Rheinhardt git at videolan.org
Fri Mar 5 10:24:16 EET 2021


ffmpeg | branch: master | Andreas Rheinhardt <andreas.rheinhardt at gmail.com> | Tue Mar  2 10:00:18 2021 +0100| [491a30c79cf7e1fa7e32aeb4968a8c63af6e0534] | committer: Andreas Rheinhardt

avformat/aadec: Replace strncpy by av_strlcpy

While this usage of strncpy is correct, said function nevertheless has
the disadvantage of not automatically ensuring that the destination
string is zero-terminated. So av_strlcpy should be preferred.
This also removes a -Wstringop-truncation warning from GCC (it doesn't
matter whether the buffer is truncated, as long as it can fit all
the names of the supported codecs).

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=491a30c79cf7e1fa7e32aeb4968a8c63af6e0534
---

 libavformat/aadec.c | 2 +-
 1 file changed, 1 insertion(+), 1 deletion(-)

diff --git a/libavformat/aadec.c b/libavformat/aadec.c
index b1e93ea072..e88cdb89df 100644
--- a/libavformat/aadec.c
+++ b/libavformat/aadec.c
@@ -114,7 +114,7 @@ static int aa_read_header(AVFormatContext *s)
         avio_get_str(pb, nval, val, sizeof(val));
         if (!strcmp(key, "codec")) {
             av_log(s, AV_LOG_DEBUG, "Codec is <%s>\n", val);
-            strncpy(codec_name, val, sizeof(codec_name) - 1);
+            av_strlcpy(codec_name, val, sizeof(codec_name));
         } else if (!strcmp(key, "HeaderSeed")) {
             av_log(s, AV_LOG_DEBUG, "HeaderSeed is <%s>\n", val);
             header_seed = atoi(val);



More information about the ffmpeg-cvslog mailing list