From ef05983dc85f945fff7f5311eabbf79660377b79 Mon Sep 17 00:00:00 2001 From: Joachim Nilsson Date: Tue, 15 Sep 2015 11:49:47 +0200 Subject: [PATCH] Fix Coverity CID #149216: Buffer not NUL terminated We should of course always use `strlcpy()` instead of `strncpy()`. Signed-off-by: Joachim Nilsson --- plugins/netlink.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/plugins/netlink.c b/plugins/netlink.c index 14f5c9ba..f007457d 100644 --- a/plugins/netlink.c +++ b/plugins/netlink.c @@ -112,7 +112,7 @@ static void nl_link(struct nlmsghdr *nlmsg) if (a->rta_type == IFLA_IFNAME) { char msg[MAX_ARG_LEN]; - strncpy(ifname, RTA_DATA(a), sizeof(ifname)); + strlcpy(ifname, RTA_DATA(a), sizeof(ifname)); switch (nlmsg->nlmsg_type) { case RTM_NEWLINK: /*