traffic-selector: Don't end printf'ed list of traffic selectors with a space

diff --git a/src/libcharon/config/peer_cfg.c b/src/libcharon/config/peer_cfg.c
index ce9301006ff3e4d836660c1a8a6e7ea5cbb85d30..aa2a39ce59d256f544de2c78d51255d7f31959a5 100644 (file)
--- a/src/libcharon/config/peer_cfg.c
+++ b/src/libcharon/config/peer_cfg.c
@@ -302,7 +302,7 @@ METHOD(peer_cfg_t, select_child_cfg, child_cfg_t*,
        enumerator_t *enumerator;
        int best = 0;
 
-       DBG2(DBG_CFG, "looking for a child config for %#R=== %#R", my_ts, other_ts);
+       DBG2(DBG_CFG, "looking for a child config for %#R === %#R", my_ts, other_ts);
        enumerator = create_child_cfg_enumerator(this);
        while (enumerator->enumerate(enumerator, &current))
        {

diff --git a/src/libcharon/plugins/error_notify/error_notify_listener.c b/src/libcharon/plugins/error_notify/error_notify_listener.c
index f7a1f49ec46685ed7cd8d70bfdcd8549a566fc55..ce577c62c34590377135afafb59f324ed467187f 100644 (file)
--- a/src/libcharon/plugins/error_notify/error_notify_listener.c
+++ b/src/libcharon/plugins/error_notify/error_notify_listener.c
@@ -110,7 +110,7 @@ METHOD(listener_t, alert, bool,
                        list = va_arg(args, linked_list_t*);
                        list2 = va_arg(args, linked_list_t*);
                        snprintf(msg.str, sizeof(msg.str), "the received traffic selectors "
-                                        "did not match: %#R=== %#R", list, list2);
+                                        "did not match: %#R === %#R", list, list2);
                        break;
                case ALERT_INSTALL_CHILD_SA_FAILED:
                        msg.type = htonl(ERROR_NOTIFY_INSTALL_CHILD_SA_FAILED);

diff --git a/src/libcharon/plugins/ha/ha_child.c b/src/libcharon/plugins/ha/ha_child.c
index 17f2d50d13e753dd013c505abac4f6f22edff638..dbb6adc8fb13ad058092f4e86f772d7e8c34fc15 100644 (file)
--- a/src/libcharon/plugins/ha/ha_child.c
+++ b/src/libcharon/plugins/ha/ha_child.c
@@ -126,7 +126,7 @@ METHOD(listener_t, child_keys, bool,
                        ike_sa->get_my_host(ike_sa), child_sa->get_spi(child_sa, TRUE));
        seg_o = this->kernel->get_segment_spi(this->kernel,
                        ike_sa->get_other_host(ike_sa), child_sa->get_spi(child_sa, FALSE));
-       DBG1(DBG_CFG, "handling HA CHILD_SA %s{%d} %#R=== %#R "
+       DBG1(DBG_CFG, "handling HA CHILD_SA %s{%d} %#R === %#R "
                "(segment in: %d%s, out: %d%s)", child_sa->get_name(child_sa),
                child_sa->get_unique_id(child_sa), local_ts, remote_ts,
                seg_i, this->segments->is_active(this->segments, seg_i) ? "*" : "",

diff --git a/src/libcharon/plugins/ha/ha_dispatcher.c b/src/libcharon/plugins/ha/ha_dispatcher.c
index afa0993095a0ca89d818e1116a46949efc771a3d..07ef607c684ee3af31d0023f27428f16f37a0dd4 100644 (file)
--- a/src/libcharon/plugins/ha/ha_dispatcher.c
+++ b/src/libcharon/plugins/ha/ha_dispatcher.c
@@ -848,7 +848,7 @@ static void process_child_add(private_ha_dispatcher_t *this,
        seg_o = this->kernel->get_segment_spi(this->kernel,
                                                                ike_sa->get_other_host(ike_sa), outbound_spi);
 
-       DBG1(DBG_CFG, "installed HA CHILD_SA %s{%d} %#R=== %#R "
+       DBG1(DBG_CFG, "installed HA CHILD_SA %s{%d} %#R === %#R "
                "(segment in: %d%s, out: %d%s)", child_sa->get_name(child_sa),
                child_sa->get_unique_id(child_sa), local_ts, remote_ts,
                seg_i, this->segments->is_active(this->segments, seg_i) ? "*" : "",

diff --git a/src/libcharon/plugins/stroke/stroke_list.c b/src/libcharon/plugins/stroke/stroke_list.c
index c7e4c9c65b5ef3a49ce38ca79a08213d7225e807..c0192b5c0c2f58e6c4a19f3be13ab86d86d33b39 100644 (file)
--- a/src/libcharon/plugins/stroke/stroke_list.c
+++ b/src/libcharon/plugins/stroke/stroke_list.c
@@ -334,7 +334,7 @@ static void log_child_sa(FILE *out, child_sa_t *child_sa, bool all)
                                                        child_sa->create_ts_enumerator(child_sa, TRUE));
        other_ts = linked_list_create_from_enumerator(
                                                        child_sa->create_ts_enumerator(child_sa, FALSE));
-       fprintf(out, "\n%12s{%d}:   %#R=== %#R\n",
+       fprintf(out, "\n%12s{%d}:   %#R === %#R\n",
                        child_sa->get_name(child_sa), child_sa->get_unique_id(child_sa),
                        my_ts, other_ts);
        my_ts->destroy(my_ts);
@@ -586,7 +586,7 @@ METHOD(stroke_list_t, status, void,
                        {
                                my_ts = child_cfg->get_traffic_selectors(child_cfg, TRUE, NULL, NULL);
                                other_ts = child_cfg->get_traffic_selectors(child_cfg, FALSE, NULL, NULL);
-                               fprintf(out, "%12s:   child:  %#R=== %#R%N",
+                               fprintf(out, "%12s:   child:  %#R === %#R %N",
                                                child_cfg->get_name(child_cfg), my_ts, other_ts,
                                                ipsec_mode_names, child_cfg->get_mode(child_cfg));
                                my_ts->destroy_offset(my_ts, offsetof(traffic_selector_t, destroy));
@@ -620,7 +620,7 @@ METHOD(stroke_list_t, status, void,
                }
                my_ts = child_cfg->get_traffic_selectors(child_cfg, TRUE, NULL, NULL);
                other_ts = child_cfg->get_traffic_selectors(child_cfg, FALSE, NULL, NULL);
-               fprintf(out, "%12s:  %#R=== %#R%N\n",
+               fprintf(out, "%12s:  %#R === %#R %N\n",
                                child_cfg->get_name(child_cfg), my_ts, other_ts,
                                ipsec_mode_names, child_cfg->get_mode(child_cfg));
                my_ts->destroy_offset(my_ts, offsetof(traffic_selector_t, destroy));

diff --git a/src/libcharon/sa/ikev1/tasks/quick_delete.c b/src/libcharon/sa/ikev1/tasks/quick_delete.c
index 1b95a8b11abd81429ec16db3e6f2248cb723765d..ade59a2ddb6f82b388d0856454eba2be91e9fb13 100644 (file)
--- a/src/libcharon/sa/ikev1/tasks/quick_delete.c
+++ b/src/libcharon/sa/ikev1/tasks/quick_delete.c
@@ -115,7 +115,7 @@ static bool delete_child(private_quick_delete_t *this, protocol_id_t protocol,
        if (this->expired)
        {
                DBG0(DBG_IKE, "closing expired CHILD_SA %s{%d} "
-                        "with SPIs %.8x_i %.8x_o and TS %#R=== %#R",
+                        "with SPIs %.8x_i %.8x_o and TS %#R === %#R",
                         child_sa->get_name(child_sa), child_sa->get_unique_id(child_sa),
                         ntohl(child_sa->get_spi(child_sa, TRUE)),
                         ntohl(child_sa->get_spi(child_sa, FALSE)), my_ts, other_ts);
@@ -126,7 +126,7 @@ static bool delete_child(private_quick_delete_t *this, protocol_id_t protocol,
                child_sa->get_usestats(child_sa, FALSE, NULL, &bytes_out, NULL);
 
                DBG0(DBG_IKE, "closing CHILD_SA %s{%d} with SPIs "
-                        "%.8x_i (%llu bytes) %.8x_o (%llu bytes) and TS %#R=== %#R",
+                        "%.8x_i (%llu bytes) %.8x_o (%llu bytes) and TS %#R === %#R",
                         child_sa->get_name(child_sa), child_sa->get_unique_id(child_sa),
                         ntohl(child_sa->get_spi(child_sa, TRUE)), bytes_in,
                         ntohl(child_sa->get_spi(child_sa, FALSE)), bytes_out,

diff --git a/src/libcharon/sa/ikev1/tasks/quick_mode.c b/src/libcharon/sa/ikev1/tasks/quick_mode.c
index 45eb7f0d6d78c1a68a3f4041b160be36f75b30cb..e7d26443b02babfd62078a8ac6487ca85a260035 100644 (file)
--- a/src/libcharon/sa/ikev1/tasks/quick_mode.c
+++ b/src/libcharon/sa/ikev1/tasks/quick_mode.c
@@ -388,7 +388,7 @@ static bool install(private_quick_mode_t *this)
                                this->child_sa->create_ts_enumerator(this->child_sa, FALSE));
 
        DBG0(DBG_IKE, "CHILD_SA %s{%d} established "
-                "with SPIs %.8x_i %.8x_o and TS %#R=== %#R",
+                "with SPIs %.8x_i %.8x_o and TS %#R === %#R",
                 this->child_sa->get_name(this->child_sa),
                 this->child_sa->get_unique_id(this->child_sa),
                 ntohl(this->child_sa->get_spi(this->child_sa, TRUE)),

diff --git a/src/libcharon/sa/ikev2/tasks/child_create.c b/src/libcharon/sa/ikev2/tasks/child_create.c
index e08f3dab13cd3f7b0a6ff3a8ed27336c2ddcc438..97f73d851d9d346cf75db86f493ff5b0fd0ec686 100644 (file)
--- a/src/libcharon/sa/ikev2/tasks/child_create.c
+++ b/src/libcharon/sa/ikev2/tasks/child_create.c
@@ -712,7 +712,7 @@ static status_t select_and_install(private_child_create_t *this,
                                this->child_sa->create_ts_enumerator(this->child_sa, FALSE));
 
        DBG0(DBG_IKE, "CHILD_SA %s{%d} established "
-                "with SPIs %.8x_i %.8x_o and TS %#R=== %#R",
+                "with SPIs %.8x_i %.8x_o and TS %#R === %#R",
                 this->child_sa->get_name(this->child_sa),
                 this->child_sa->get_unique_id(this->child_sa),
                 ntohl(this->child_sa->get_spi(this->child_sa, TRUE)),
@@ -1245,7 +1245,7 @@ METHOD(task_t, build_r, status_t,
        }
        if (this->config == NULL)
        {
-               DBG1(DBG_IKE, "traffic selectors %#R=== %#R inacceptable",
+               DBG1(DBG_IKE, "traffic selectors %#R === %#R inacceptable",
                         this->tsr, this->tsi);
                charon->bus->alert(charon->bus, ALERT_TS_MISMATCH, this->tsi, this->tsr);
                message->add_notify(message, FALSE, TS_UNACCEPTABLE, chunk_empty);

diff --git a/src/libcharon/sa/ikev2/tasks/child_delete.c b/src/libcharon/sa/ikev2/tasks/child_delete.c
index f0b11e2910a8551e51727c6dc0c5777d10e51377..877ae053162fdf10e667b965525a20bc373d5515 100644 (file)
--- a/src/libcharon/sa/ikev2/tasks/child_delete.c
+++ b/src/libcharon/sa/ikev2/tasks/child_delete.c
@@ -266,7 +266,7 @@ static void log_children(private_child_delete_t *this)
                if (this->expired)
                {
                        DBG0(DBG_IKE, "closing expired CHILD_SA %s{%d} "
-                                "with SPIs %.8x_i %.8x_o and TS %#R=== %#R",
+                                "with SPIs %.8x_i %.8x_o and TS %#R === %#R",
                                 child_sa->get_name(child_sa), child_sa->get_unique_id(child_sa),
                                 ntohl(child_sa->get_spi(child_sa, TRUE)),
                                 ntohl(child_sa->get_spi(child_sa, FALSE)), my_ts, other_ts);
@@ -277,7 +277,7 @@ static void log_children(private_child_delete_t *this)
                        child_sa->get_usestats(child_sa, FALSE, NULL, &bytes_out, NULL);
 
                        DBG0(DBG_IKE, "closing CHILD_SA %s{%d} with SPIs %.8x_i "
-                                "(%llu bytes) %.8x_o (%llu bytes) and TS %#R=== %#R",
+                                "(%llu bytes) %.8x_o (%llu bytes) and TS %#R === %#R",
                                 child_sa->get_name(child_sa), child_sa->get_unique_id(child_sa),
                                 ntohl(child_sa->get_spi(child_sa, TRUE)), bytes_in,
                                 ntohl(child_sa->get_spi(child_sa, FALSE)), bytes_out,

diff --git a/src/libstrongswan/selectors/traffic_selector.c b/src/libstrongswan/selectors/traffic_selector.c
index 66863245902e84a83c508f21acfa76dd973753a4..a6298b394801bb5d7f055847357431e128d9f053 100644 (file)
--- a/src/libstrongswan/selectors/traffic_selector.c
+++ b/src/libstrongswan/selectors/traffic_selector.c
@@ -219,9 +219,8 @@ int traffic_selector_printf_hook(printf_hook_data_t *data,
        enumerator_t *enumerator;
        char from_str[INET6_ADDRSTRLEN] = "";
        char to_str[INET6_ADDRSTRLEN] = "";
-       char *serv_proto = NULL;
-       bool has_proto;
-       bool has_ports;
+       char *serv_proto = NULL, *sep = "";
+       bool has_proto, has_ports;
        size_t written = 0;
        u_int32_t from[4], to[4];
 
@@ -235,8 +234,8 @@ int traffic_selector_printf_hook(printf_hook_data_t *data,
                enumerator = list->create_enumerator(list);
                while (enumerator->enumerate(enumerator, (void**)&this))
                {
-                       /* call recursivly */
-                       written += print_in_hook(data, "%R ", this);
+                       written += print_in_hook(data, "%s%R", sep, this);
+                       sep = " ";
                }
                enumerator->destroy(enumerator);
                return written;

diff --git a/src/libstrongswan/tests/suites/test_traffic_selector.c b/src/libstrongswan/tests/suites/test_traffic_selector.c
index bec32d2d8ccde3c5eea12e2a5dc1a7304fac523d..5c0fb754d7a80cead8908a0d1d835eab49f10948 100644 (file)
--- a/src/libstrongswan/tests/suites/test_traffic_selector.c
+++ b/src/libstrongswan/tests/suites/test_traffic_selector.c
@@ -770,17 +770,17 @@ START_TEST(test_printf_hook_hash)
        list = linked_list_create_with_items(
                                traffic_selector_create_from_cidr("10.1.0.0/16", 0, 0, 65535),
                                NULL);
-       verify_list("10.1.0.0/16 ", NULL, list);
+       verify_list("10.1.0.0/16", NULL, list);
        list = linked_list_create_with_items(
                                traffic_selector_create_from_cidr("10.1.0.0/16", 0, 0, 65535),
                                traffic_selector_create_from_cidr("10.1.0.1/32", IPPROTO_UDP, 1234, 1235),
                                NULL);
-       verify_list("10.1.0.0/16 10.1.0.1/32[udp/1234-1235] ", "10.1.0.0/16 10.1.0.1/32[17/1234-1235] ", list);
+       verify_list("10.1.0.0/16 10.1.0.1/32[udp/1234-1235]", "10.1.0.0/16 10.1.0.1/32[17/1234-1235]", list);
        list = linked_list_create_with_items(
                                traffic_selector_create_from_cidr("10.1.0.0/16", 0, 0, 65535),
                                traffic_selector_create_from_string(IPPROTO_UDP, TS_IPV4_ADDR_RANGE, "10.1.0.1", 1234, "10.1.0.99", 1235),
                                NULL);
-       verify_list("10.1.0.0/16 10.1.0.1..10.1.0.99[udp/1234-1235] ", "10.1.0.0/16 10.1.0.1..10.1.0.99[17/1234-1235] ", list);
+       verify_list("10.1.0.0/16 10.1.0.1..10.1.0.99[udp/1234-1235]", "10.1.0.0/16 10.1.0.1..10.1.0.99[17/1234-1235]", list);
 }
 END_TEST