kernel-netlink: Pass protocol specific enum names to socket constructor

This avoid the hard dependency on enum names, and makes kernel_netlink_shared
independent of kernel_netlink_ipsec.

diff --git a/src/libhydra/plugins/kernel_netlink/kernel_netlink_ipsec.c b/src/libhydra/plugins/kernel_netlink/kernel_netlink_ipsec.c
index cd94168c438fbd28ab57f1aefc30d080e0748b54..dfd71f3bd58d562a3f90630862830e932779392b 100644 (file)
--- a/src/libhydra/plugins/kernel_netlink/kernel_netlink_ipsec.c
+++ b/src/libhydra/plugins/kernel_netlink/kernel_netlink_ipsec.c
@@ -2711,7 +2711,7 @@ kernel_netlink_ipsec_t *kernel_netlink_ipsec_create()
                fclose(f);
        }
 
-       this->socket_xfrm = netlink_socket_create(NETLINK_XFRM);
+       this->socket_xfrm = netlink_socket_create(NETLINK_XFRM, xfrm_msg_names);
        if (!this->socket_xfrm)
        {
                destroy(this);

diff --git a/src/libhydra/plugins/kernel_netlink/kernel_netlink_net.c b/src/libhydra/plugins/kernel_netlink/kernel_netlink_net.c
index 3d5538c35bd3cb73b4a85173d9125b01247fb129..e8d146c9184f5f59fa5c6b80c4a1beefca2bde0c 100644 (file)
--- a/src/libhydra/plugins/kernel_netlink/kernel_netlink_net.c
+++ b/src/libhydra/plugins/kernel_netlink/kernel_netlink_net.c
@@ -2469,7 +2469,7 @@ kernel_netlink_net_t *kernel_netlink_net_create()
                                .destroy = _destroy,
                        },
                },
-               .socket = netlink_socket_create(NETLINK_ROUTE),
+               .socket = netlink_socket_create(NETLINK_ROUTE, NULL),
                .rt_exclude = linked_list_create(),
                .routes = hashtable_create((hashtable_hash_t)route_entry_hash,
                                                                   (hashtable_equals_t)route_entry_equals, 16),

diff --git a/src/libhydra/plugins/kernel_netlink/kernel_netlink_shared.c b/src/libhydra/plugins/kernel_netlink/kernel_netlink_shared.c
index 74466e291124737831ee1bcd020a37c402263268..b4cece720f9f2f500d45c2374637fd796c6d58cb 100644 (file)
--- a/src/libhydra/plugins/kernel_netlink/kernel_netlink_shared.c
+++ b/src/libhydra/plugins/kernel_netlink/kernel_netlink_shared.c
@@ -46,14 +46,14 @@ struct private_netlink_socket_t {
        int seq;
 
        /**
-        * netlink socket protocol
+        * netlink socket
         */
-       int protocol;
+       int socket;
 
        /**
-        * netlink socket
+        * Enum names for Netlink messages
         */
-       int socket;
+       enum_name_t *names;
 };
 
 /**
@@ -83,10 +83,10 @@ METHOD(netlink_socket_t, netlink_send, status_t,
        addr.nl_pid = 0;
        addr.nl_groups = 0;
 
-       if (this->protocol == NETLINK_XFRM)
+       if (this->names)
        {
                DBG3(DBG_KNL, "sending %N: %b",
-                        xfrm_msg_names, in->nlmsg_type, in, in->nlmsg_len);
+                        this->names, in->nlmsg_type, in, in->nlmsg_len);
        }
        while (TRUE)
        {
@@ -230,7 +230,7 @@ METHOD(netlink_socket_t, destroy, void,
 /**
  * Described in header.
  */
-netlink_socket_t *netlink_socket_create(int protocol)
+netlink_socket_t *netlink_socket_create(int protocol, enum_name_t *names)
 {
        private_netlink_socket_t *this;
        struct sockaddr_nl addr = {
@@ -246,7 +246,7 @@ netlink_socket_t *netlink_socket_create(int protocol)
                .seq = 200,
                .mutex = mutex_create(MUTEX_TYPE_DEFAULT),
                .socket = socket(AF_NETLINK, SOCK_RAW, protocol),
-               .protocol = protocol,
+               .names = names,
        );
 
        if (this->socket == -1)

diff --git a/src/libhydra/plugins/kernel_netlink/kernel_netlink_shared.h b/src/libhydra/plugins/kernel_netlink/kernel_netlink_shared.h
index 1a03a89a8543784afa226de50167915f0f224a5d..069f746d10a70609d4486b938a5f7502866de038 100644 (file)
--- a/src/libhydra/plugins/kernel_netlink/kernel_netlink_shared.h
+++ b/src/libhydra/plugins/kernel_netlink/kernel_netlink_shared.h
@@ -64,9 +64,10 @@ struct netlink_socket_t {
 /**
  * Create a netlink_socket_t object.
  *
- * @param      protocol        protocol type (e.g. NETLINK_XFRM or NETLINK_ROUTE)
+ * @param protocol     protocol type (e.g. NETLINK_XFRM or NETLINK_ROUTE)
+ * @param names                optional enum names for Netlink messages
  */
-netlink_socket_t *netlink_socket_create(int protocol);
+netlink_socket_t *netlink_socket_create(int protocol, enum_name_t *names);
 
 /**
  * Creates an rtattr and adds it to the given netlink message.