aboutsummaryrefslogtreecommitdiffstats
path: root/src/charon/threads/kernel_interface.c
diff options
context:
space:
mode:
authorMartin Willi <martin@strongswan.org>2006-06-09 08:41:41 +0000
committerMartin Willi <martin@strongswan.org>2006-06-09 08:41:41 +0000
commitb543bef50ce0371c0be59c60e70419ac04c8cc87 (patch)
tree8350d73be38a114279dd4e3569ba2439befd32a3 /src/charon/threads/kernel_interface.c
parent0bb32cb5f3dc03c9e754d02f19527a7e221b4e7b (diff)
downloadstrongswan-b543bef50ce0371c0be59c60e70419ac04c8cc87.tar.bz2
strongswan-b543bef50ce0371c0be59c60e70419ac04c8cc87.tar.xz
improved kernel interface logging
Diffstat (limited to 'src/charon/threads/kernel_interface.c')
-rw-r--r--src/charon/threads/kernel_interface.c85
1 files changed, 38 insertions, 47 deletions
diff --git a/src/charon/threads/kernel_interface.c b/src/charon/threads/kernel_interface.c
index b78f67449..617e324bb 100644
--- a/src/charon/threads/kernel_interface.c
+++ b/src/charon/threads/kernel_interface.c
@@ -73,7 +73,7 @@ struct xfrm_data_t {
/** algorithm */
struct xfrm_algo algo;
/** policy tmpl */
- struct xfrm_user_tmpl tmpl[2];
+ struct xfrm_user_tmpl tmpl;
};
};
@@ -263,8 +263,7 @@ static status_t get_spi(private_kernel_interface_t *this,
netlink_message_t request, *response;
status_t status = SUCCESS;
-
- this->logger->log(this->logger, CONTROL|LEVEL2, "getting spi");
+ this->logger->log(this->logger, CONTROL|LEVEL1, "Getting SPI for reqid %d", reqid);
memset(&request, 0, sizeof(request));
request.hdr.nlmsg_len = NLMSG_ALIGN(NLMSG_LENGTH(sizeof(request.spi)));
@@ -316,7 +315,7 @@ static status_t add_sa( private_kernel_interface_t *this,
host_t *me,
host_t *other,
u_int32_t spi,
- int protocol,
+ protocol_id_t protocol,
u_int32_t reqid,
u_int64_t expire_soft,
u_int64_t expire_hard,
@@ -332,7 +331,8 @@ static status_t add_sa( private_kernel_interface_t *this,
memset(&request, 0, sizeof(request));
- this->logger->log(this->logger, CONTROL|LEVEL2, "adding SA");
+ this->logger->log(this->logger, CONTROL|LEVEL1, "Adding %s SA with SPI 0x%x, reqid %d to kernel",
+ mapping_find(protocol_id_m, protocol), htonl(spi), reqid);
request.hdr.nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK;
request.hdr.nlmsg_type = replace ? XFRM_MSG_UPDSA : XFRM_MSG_NEWSA;
@@ -371,7 +371,8 @@ static status_t add_sa( private_kernel_interface_t *this,
mapping_find(encryption_algorithm_m, enc_alg->algorithm));
return FAILED;
}
- this->logger->log(this->logger, CONTROL|LEVEL2, "using key size %d", key_size);
+ this->logger->log(this->logger, CONTROL|LEVEL2, " using encryption algorithm %s with key size %d",
+ mapping_find(encryption_algorithm_m, enc_alg->algorithm), key_size);
data->length = 4 + sizeof(data->algo) + key_size;
data->algo.alg_key_len = key_size;
request.hdr.nlmsg_len += data->length;
@@ -395,7 +396,8 @@ static status_t add_sa( private_kernel_interface_t *this,
mapping_find(integrity_algorithm_m, int_alg->algorithm));
return FAILED;
}
- this->logger->log(this->logger, CONTROL|LEVEL2, "using key size %d", key_size);
+ this->logger->log(this->logger, CONTROL|LEVEL2, " using integrity algorithm %s with key size %d",
+ mapping_find(integrity_algorithm_m, int_alg->algorithm), key_size);
data->length = 4 + sizeof(data->algo) + key_size;
data->algo.alg_key_len = key_size;
request.hdr.nlmsg_len += data->length;
@@ -439,7 +441,8 @@ static status_t del_sa( private_kernel_interface_t *this,
memset(&request, 0, sizeof(request));
status_t status = SUCCESS;
- this->logger->log(this->logger, CONTROL|LEVEL2, "deleting SA");
+ this->logger->log(this->logger, CONTROL|LEVEL1, "Deleting %s SA with SPI 0x%x from kernel",
+ mapping_find(protocol_id_m, protocol), htonl(spi));
request.hdr.nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK;
request.hdr.nlmsg_type = XFRM_MSG_DELSA;
@@ -454,14 +457,18 @@ static status_t del_sa( private_kernel_interface_t *this,
if (this->send_message(this, &request, &response) != SUCCESS)
{
+ this->logger->log(this->logger, ERROR, "netlink communication failed");
return FAILED;
}
else if (response->hdr.nlmsg_type != NLMSG_ERROR)
{
+ this->logger->log(this->logger, ERROR, "netlink request XFRM_MSG_DELSA not acknowledged");
status = FAILED;
}
else if (response->e.error)
{
+ this->logger->log(this->logger, ERROR|LEVEL1, "netlink request XFRM_MSG_DELSA received error: %s",
+ strerror(-response->e.error));
status = FAILED;
}
@@ -476,14 +483,16 @@ static status_t add_policy(private_kernel_interface_t *this,
host_t *me, host_t *other,
host_t *src, host_t *dst,
u_int8_t src_hostbits, u_int8_t dst_hostbits,
- int direction, int upper_proto,
- bool ah, bool esp,
+ int direction, int upper_proto,
+ protocol_id_t protocol,
u_int32_t reqid)
{
netlink_message_t request, *response;
status_t status = SUCCESS;
+ xfrm_data_t *data;
- this->logger->log(this->logger, CONTROL|LEVEL2, "adding policy");
+ this->logger->log(this->logger, CONTROL|LEVEL1, "Adding %s policy with reqid %d to kernel",
+ mapping_find(protocol_id_m, protocol), reqid);
memset(&request, 0, sizeof(request));
request.hdr.nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK;
@@ -516,39 +525,18 @@ static status_t add_policy(private_kernel_interface_t *this,
request.sa.lft.soft_use_expires_seconds = 0;
request.sa.lft.hard_use_expires_seconds = 0;
- if (esp || ah)
- {
- xfrm_data_t *data;
- int tmpl_pos = 0;
- data = (xfrm_data_t*)(((u_int8_t*)&request) + request.hdr.nlmsg_len);
- data->type = XFRMA_TMPL;
- if (esp)
- {
- data->tmpl[tmpl_pos].reqid = reqid;
- data->tmpl[tmpl_pos].id.proto = KERNEL_ESP;
- data->tmpl[tmpl_pos].aalgos = data->tmpl[tmpl_pos].ealgos = data->tmpl[tmpl_pos].calgos = ~0;
- data->tmpl[tmpl_pos].mode = TRUE;
-
- data->tmpl[tmpl_pos].saddr = me->get_xfrm_addr(me);
- data->tmpl[tmpl_pos].id.daddr = me->get_xfrm_addr(other);
-
- tmpl_pos++;
- }
- if (ah)
- {
- data->tmpl[tmpl_pos].reqid = reqid;
- data->tmpl[tmpl_pos].id.proto = KERNEL_AH;
- data->tmpl[tmpl_pos].aalgos = data->tmpl[tmpl_pos].ealgos = data->tmpl[tmpl_pos].calgos = ~0;
- data->tmpl[tmpl_pos].mode = TRUE;
-
- data->tmpl[tmpl_pos].saddr = me->get_xfrm_addr(me);
- data->tmpl[tmpl_pos].id.daddr = other->get_xfrm_addr(other);
-
- tmpl_pos++;
- }
- data->length = 4 + sizeof(struct xfrm_user_tmpl) * tmpl_pos;
- request.hdr.nlmsg_len += data->length;
- }
+ data = (xfrm_data_t*)(((u_int8_t*)&request) + request.hdr.nlmsg_len);
+ data->type = XFRMA_TMPL;
+
+ data->tmpl.reqid = reqid;
+ data->tmpl.id.proto = protocol == PROTO_AH ? KERNEL_AH : KERNEL_ESP;
+ data->tmpl.aalgos = data->tmpl.ealgos = data->tmpl.calgos = ~0;
+ data->tmpl.mode = TRUE;
+ data->tmpl.saddr = me->get_xfrm_addr(me);
+ data->tmpl.id.daddr = me->get_xfrm_addr(other);
+
+ data->length = 4 + sizeof(struct xfrm_user_tmpl);
+ request.hdr.nlmsg_len += data->length;
if (this->send_message(this, &request, &response) != SUCCESS)
{
@@ -583,8 +571,7 @@ static status_t del_policy(private_kernel_interface_t *this,
netlink_message_t request, *response;
status_t status = SUCCESS;
-
- this->logger->log(this->logger, CONTROL|LEVEL2, "deleting policy");
+ this->logger->log(this->logger, CONTROL|LEVEL1, "Removing policy from kernel");
memset(&request, 0, sizeof(request));
request.hdr.nlmsg_flags = NLM_F_REQUEST | NLM_F_ACK;
@@ -607,14 +594,18 @@ static status_t del_policy(private_kernel_interface_t *this,
if (this->send_message(this, &request, &response) != SUCCESS)
{
+ this->logger->log(this->logger, ERROR, "netlink communication failed");
return FAILED;
}
else if (response->hdr.nlmsg_type != NLMSG_ERROR)
{
+ this->logger->log(this->logger, ERROR, "netlink request XFRM_MSG_DELPOLICY not acknowledged");
status = FAILED;
}
else if (response->e.error)
{
+ this->logger->log(this->logger, ERROR, "netlink request XFRM_MSG_DELPOLICY received error: %s",
+ strerror(-response->e.error));
status = FAILED;
}
@@ -801,7 +792,7 @@ kernel_interface_t *kernel_interface_create()
/* public functions */
this->public.get_spi = (status_t(*)(kernel_interface_t*,host_t*,host_t*,protocol_id_t,u_int32_t,u_int32_t*))get_spi;
this->public.add_sa = (status_t(*)(kernel_interface_t *,host_t*,host_t*,u_int32_t,protocol_id_t,u_int32_t,u_int64_t,u_int64_t,algorithm_t*,algorithm_t*,prf_plus_t*,bool))add_sa;
- this->public.add_policy = (status_t(*)(kernel_interface_t*,host_t*, host_t*,host_t*,host_t*,u_int8_t,u_int8_t,int,int,bool,bool,u_int32_t))add_policy;
+ this->public.add_policy = (status_t(*)(kernel_interface_t*,host_t*, host_t*,host_t*,host_t*,u_int8_t,u_int8_t,int,int,protocol_id_t,u_int32_t))add_policy;
this->public.del_sa = (status_t(*)(kernel_interface_t*,host_t*,u_int32_t,protocol_id_t))del_sa;
this->public.del_policy = (status_t(*)(kernel_interface_t*,host_t*,host_t*,host_t*,host_t*,u_int8_t,u_int8_t,int,int))del_policy;