commit e666bdfe9b7e67ab3f393ae4389baf5db0e77279
author Szymon Janc <szymon.janc@codecoup.pl> Mon Jun 27 17:54:20 2022 +0200
committer Szymon Janc <szymon.janc@codecoup.pl> Tue Jun 28 10:57:09 2022 +0200
tree dd1de6740efc37a0928146e5e24f117234f013b6
parent b84388bc54eb4376c7d01e1e6a7d3823b5e584ee

net/ip/native_socks: Fix build on with -Werror=stringop-overflow

Werror=stringop-overflow= seems to be now enabled by default on GCC 11.

repos/apache-mynewt-core/net/ip/native_sockets/src/native_sock.c: In
    function ‘native_sock_mn_addr_to_addr’:
repos/apache-mynewt-core/net/ip/native_sockets/src/native_sock.c:199:9:
    error: ‘strncat’ specified bound 108 equals destination size
    [-Werror=stringop-overflow=]
  199 |         strncat(sun->sun_path, msun->msun_path,
                        sizeof sun->sun_path);
      |         ^~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
cc1: all warnings being treated as errors

net/ip/native_sockets/src/native_sock.c

diff --git a/net/ip/native_sockets/src/native_sock.c b/net/ip/native_sockets/src/native_sock.c
index 5fbf0d1..71ad048 100644
--- a/net/ip/native_sockets/src/native_sock.c
+++ b/net/ip/native_sockets/src/native_sock.c
@@ -196,7 +196,7 @@
         sun->sun_len = sizeof(*sun);
 #endif
         sun->sun_path[0] = '\0';
-        strncat(sun->sun_path, msun->msun_path, sizeof sun->sun_path);
+        strncat(sun->sun_path, msun->msun_path, sizeof(sun->sun_path) - 1);
         if (strcmp(sun->sun_path, msun->msun_path) != 0) {
             /* Path too long. */
             return MN_EINVAL;