charon-tkm: Remove unused get_other_esa_id() method
authorTobias Brunner <tobias@strongswan.org>
Fri, 4 Aug 2017 11:59:40 +0000 (13:59 +0200)
committerTobias Brunner <tobias@strongswan.org>
Mon, 7 Aug 2017 08:46:00 +0000 (10:46 +0200)
src/charon-tkm/src/tkm/tkm_kernel_sad.c
src/charon-tkm/src/tkm/tkm_kernel_sad.h
src/charon-tkm/tests/kernel_sad_tests.c

index c888f25..92ea25e 100644 (file)
@@ -131,26 +131,6 @@ CALLBACK(sad_entry_match_dst, bool,
                   entry->proto   == *proto;
 }
 
-CALLBACK(sad_entry_match_esa_id, bool,
-       sad_entry_t * const entry, va_list args)
-{
-       const esa_id_type *esa_id;
-
-       VA_ARGS_VGET(args, esa_id);
-       return entry->esa_id == *esa_id;
-}
-
-CALLBACK(sad_entry_match_other_esa, bool,
-       sad_entry_t * const entry, va_list args)
-{
-       const esa_id_type *esa_id;
-       const uint32_t *reqid;
-
-       VA_ARGS_VGET(args, esa_id, reqid);
-       return entry->reqid  == *reqid &&
-                  entry->esa_id != *esa_id;
-}
-
 CALLBACK(sad_entry_equal, bool,
        sad_entry_t * const left, va_list args)
 {
@@ -237,40 +217,6 @@ METHOD(tkm_kernel_sad_t, get_esa_id, esa_id_type,
        return id;
 }
 
-METHOD(tkm_kernel_sad_t, get_other_esa_id, esa_id_type,
-       private_tkm_kernel_sad_t * const this, const esa_id_type esa_id)
-{
-       esa_id_type id = 0;
-       sad_entry_t *entry = NULL;
-       uint32_t reqid;
-       bool res;
-
-       this->mutex->lock(this->mutex);
-       res = this->data->find_first(this->data, sad_entry_match_esa_id,
-                                                                (void**)&entry, &esa_id);
-       if (res && entry)
-       {
-               reqid = entry->reqid;
-       }
-       else
-       {
-               DBG3(DBG_KNL, "no SAD entry found for ESA id %llu", esa_id);
-               this->mutex->unlock(this->mutex);
-               return id;
-       }
-
-       res = this->data->find_first(this->data, sad_entry_match_other_esa,
-                                                                (void**)&entry, &esa_id, &reqid);
-       if (res && entry)
-       {
-               id = entry->esa_id;
-               DBG3(DBG_KNL, "returning ESA id %llu of other SAD entry with reqid %u",
-                        id, reqid);
-       }
-       this->mutex->unlock(this->mutex);
-       return id;
-}
-
 METHOD(tkm_kernel_sad_t, get_dst_host, host_t *,
        private_tkm_kernel_sad_t * const this, const uint32_t reqid,
        const uint32_t spi, const uint8_t proto)
@@ -350,7 +296,6 @@ tkm_kernel_sad_t *tkm_kernel_sad_create()
                .public = {
                        .insert = _insert,
                        .get_esa_id = _get_esa_id,
-                       .get_other_esa_id = _get_other_esa_id,
                        .get_dst_host = _get_dst_host,
                        .remove = __remove,
                        .destroy = _destroy,
index 63d02b7..bcb3b43 100644 (file)
@@ -64,16 +64,6 @@ struct tkm_kernel_sad_t {
                                 const uint32_t spi, const uint8_t proto);
 
        /**
-        * Get ESA id for entry associated with same security policy as the
-        * specified ESA.
-        *
-        * @param esa_id                id of ESA identifying the security policy
-        * @return                              ESA id of entry if found, 0 otherwise
-        */
-       esa_id_type (*get_other_esa_id)(tkm_kernel_sad_t * const this,
-                                                                       const esa_id_type esa_id);
-
-       /**
         * Get destination host for entry with given parameters.
         *
         * @param reqid                 reqid of CHILD SA
index 2a033d2..5dbe0ae 100644 (file)
@@ -81,37 +81,6 @@ START_TEST(test_get_esa_id_nonexistent)
 }
 END_TEST
 
-START_TEST(test_get_other_esa_id)
-{
-       host_t *addr = host_create_from_string("127.0.0.1", 1024);
-       tkm_kernel_sad_t *sad = tkm_kernel_sad_create();
-       fail_unless(sad->insert(sad, 23, 54, addr, addr, 27, 42, 50),
-                               "Error inserting SAD entry");
-       fail_unless(sad->insert(sad, 24, 54, addr, addr, 27, 42, 50),
-                               "Error inserting SAD entry");
-       fail_unless(sad->get_other_esa_id(sad, 23) == 24,
-                               "Error getting other esa id");
-       sad->destroy(sad);
-       addr->destroy(addr);
-}
-END_TEST
-
-START_TEST(test_get_other_esa_id_nonexistent)
-{
-       host_t *addr = host_create_from_string("127.0.0.1", 1024);
-       tkm_kernel_sad_t *sad = tkm_kernel_sad_create();
-       fail_unless(sad->get_other_esa_id(sad, 1) == 0,
-                               "Got other esa id for nonexistent SAD entry");
-       fail_unless(sad->insert(sad, 23, 54, addr, addr, 27, 42, 50),
-                               "Error inserting SAD entry");
-       fail_unless(sad->get_other_esa_id(sad, 23) == 0,
-                               "Got own esa id");
-
-       sad->destroy(sad);
-       addr->destroy(addr);
-}
-END_TEST
-
 START_TEST(test_get_dst_host)
 {
        host_t *addr = host_create_from_string("127.0.0.1", 1024);
@@ -182,11 +151,6 @@ Suite *make_kernel_sad_tests()
        tcase_add_test(tc, test_get_esa_id_nonexistent);
        suite_add_tcase(s, tc);
 
-       tc = tcase_create("get_other_esa_id");
-       tcase_add_test(tc, test_get_other_esa_id);
-       tcase_add_test(tc, test_get_other_esa_id_nonexistent);
-       suite_add_tcase(s, tc);
-
        tc = tcase_create("get_dst_host");
        tcase_add_test(tc, test_get_dst_host);
        tcase_add_test(tc, test_get_dst_host_nonexistent);