Add missing check for nl_socket_alloc failure

This can happen if the memory alloc fails. Patch V2: add goto error Patch V3: return -ENOMEM instead of going to error Change-Id: Iee66caa794d267ac5f8bee584633352893047171 Signed-off-by: Arne Schwabe <[email protected]> Acked-by: Antonio Quartulli <[email protected]> Message-Id: <[email protected]> URL: https://www.mail-archive.com/[email protected]/msg27541.html Signed-off-by: Gert Doering <[email protected]>
OpenVPN · Dec 2, 2023 · d1c31e4 · d1c31e4
1 parent 59551b9
commit d1c31e4
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/src/openvpn/dco_linux.c b/src/openvpn/dco_linux.c
@@ -81,6 +81,12 @@ resolve_ovpn_netlink_id(int msglevel)
     int ret;
     struct nl_sock *nl_sock = nl_socket_alloc();
 
+    if (!nl_sock)
+    {
+        msg(msglevel, "Allocating net link socket failed");
+        return -ENOMEM;
+    }
+
     ret = genl_connect(nl_sock);
     if (ret)
     {