Properly copy interface name if unknown.

We use a static string if the interface name is unknown, so using memcpy
with IFNAMSIZ is incorrect as that would overrun the static string.

diff --git a/src/libhydra/plugins/kernel_netlink/kernel_netlink_net.c b/src/libhydra/plugins/kernel_netlink/kernel_netlink_net.c
index 6d0a567809410383c0a57840737231fca90a083d..8315ed31091b4c71c73d8b0ea92950dc1a5efea6 100644 (file)
--- a/src/libhydra/plugins/kernel_netlink/kernel_netlink_net.c
+++ b/src/libhydra/plugins/kernel_netlink/kernel_netlink_net.c
@@ -350,7 +350,7 @@ static void process_link(private_kernel_netlink_net_t *this,
                                entry->addrs = linked_list_create();
                                this->ifaces->insert_last(this->ifaces, entry);
                        }
-                       memcpy(entry->ifname, name, IFNAMSIZ);
+                       strncpy(entry->ifname, name, IFNAMSIZ);
                        entry->ifname[IFNAMSIZ-1] = '\0';
                        if (event)
                        {