]> git.sesse.net Git - ffmpeg/commitdiff
cmdutils: replace strncpy() with direct assignment
authorAnton Khirnov <anton@khirnov.net>
Wed, 24 Feb 2021 10:19:27 +0000 (11:19 +0100)
committerAnton Khirnov <anton@khirnov.net>
Tue, 16 Mar 2021 09:41:01 +0000 (10:41 +0100)
Only one character is actually rewritten.

Fixes truncation warnings, such as
warning: ‘strncpy’ output truncated before terminating nul copying 3 bytes from a string of the same length [-Wstringop-truncation]
in gcc 10.2.0

fftools/cmdutils.c

index 5e2d3c174b49fc7ffa3cf84c0d1caf68cc118dfb..fe424b6a4c8583fe54345f48c81bf06b5ccbcdfb 100644 (file)
@@ -1163,13 +1163,13 @@ static void print_buildconf(int flags, int level)
     // Change all the ' --' strings to '~--' so that
     // they can be identified as tokens.
     while ((conflist = strstr(str, " --")) != NULL) {
-        strncpy(conflist, "~--", 3);
+        conflist[0] = '~';
     }
 
     // Compensate for the weirdness this would cause
     // when passing 'pkg-config --static'.
     while ((remove_tilde = strstr(str, "pkg-config~")) != NULL) {
-        strncpy(remove_tilde, "pkg-config ", 11);
+        remove_tilde[sizeof("pkg-config~") - 2] = ' ';
     }
 
     splitconf = strtok(str, "~");