You are viewing a plain text version of this content. The canonical link for it is here.
Posted to commits@nuttx.apache.org by xi...@apache.org on 2022/04/12 13:16:16 UTC

[incubator-nuttx] branch master updated: libc/net:use strlcpy instead of strncpy

This is an automated email from the ASF dual-hosted git repository.

xiaoxiang pushed a commit to branch master
in repository https://gitbox.apache.org/repos/asf/incubator-nuttx.git


The following commit(s) were added to refs/heads/master by this push:
     new c843cb8a52 libc/net:use strlcpy instead of strncpy
c843cb8a52 is described below

commit c843cb8a5270805a365e9f094943f7d95c99aaa8
Author: anjiahao <an...@xiaomi.com>
AuthorDate: Fri Apr 8 15:34:46 2022 +0800

    libc/net:use strlcpy instead of strncpy
    
    Signed-off-by: anjiahao <an...@xiaomi.com>
---
 libs/libc/net/lib_getifaddrs.c | 2 +-
 libs/libc/netdb/lib_dnscache.c | 2 +-
 2 files changed, 2 insertions(+), 2 deletions(-)

diff --git a/libs/libc/net/lib_getifaddrs.c b/libs/libc/net/lib_getifaddrs.c
index 0e17cd4fa6..82630e8df6 100644
--- a/libs/libc/net/lib_getifaddrs.c
+++ b/libs/libc/net/lib_getifaddrs.c
@@ -166,7 +166,7 @@ int getifaddrs(FAR struct ifaddrs **addrs)
         }
 
       myaddrs->addrs.ifa_name = myaddrs->name;
-      strncpy(myaddrs->name, req.lifr_name, IF_NAMESIZE);
+      strlcpy(myaddrs->name, req.lifr_name, IF_NAMESIZE);
 
       myaddrs->addrs.ifa_flags = flags;
 
diff --git a/libs/libc/netdb/lib_dnscache.c b/libs/libc/netdb/lib_dnscache.c
index c5a18af510..09b1d718c2 100644
--- a/libs/libc/netdb/lib_dnscache.c
+++ b/libs/libc/netdb/lib_dnscache.c
@@ -142,7 +142,7 @@ void dns_save_answer(FAR const char *hostname,
   entry->ctime = (time_t)now.tv_sec;
 #endif
 
-  strncpy(entry->name, hostname, CONFIG_NETDB_DNSCLIENT_NAMESIZE);
+  strlcpy(entry->name, hostname, CONFIG_NETDB_DNSCLIENT_NAMESIZE);
   memcpy(&entry->addr, addr, naddr * sizeof(*addr));
   entry->naddr = naddr;