Moved packet_t to libstrongswan

diff --git a/src/libcharon/Android.mk b/src/libcharon/Android.mk
index 5e93e235f88445674aef75fe58625462247b6147..a4ac87182214b4887aa1137747370e4c7593495e 100644 (file)
--- a/src/libcharon/Android.mk
+++ b/src/libcharon/Android.mk
@@ -44,7 +44,7 @@ encoding/payloads/vendor_id_payload.c encoding/payloads/vendor_id_payload.h \
 encoding/payloads/hash_payload.c encoding/payloads/hash_payload.h \
 kernel/kernel_handler.c kernel/kernel_handler.h \
 network/receiver.c network/receiver.h network/sender.c network/sender.h \
-network/packet.c network/packet.h network/socket.c network/socket.h \
+network/socket.c network/socket.h \
 network/socket_manager.c network/socket_manager.h \
 processing/jobs/acquire_job.c processing/jobs/acquire_job.h \
 processing/jobs/delete_child_sa_job.c processing/jobs/delete_child_sa_job.h \

diff --git a/src/libcharon/Makefile.am b/src/libcharon/Makefile.am
index df23e22f6e0ebb0b5c263f181e8a2f7772c17bb2..cd2c9de14702bae81cfbb25ff26590d3adb7db99 100644 (file)
--- a/src/libcharon/Makefile.am
+++ b/src/libcharon/Makefile.am
@@ -42,7 +42,7 @@ encoding/payloads/vendor_id_payload.c encoding/payloads/vendor_id_payload.h \
 encoding/payloads/hash_payload.c encoding/payloads/hash_payload.h \
 kernel/kernel_handler.c kernel/kernel_handler.h \
 network/receiver.c network/receiver.h network/sender.c network/sender.h \
-network/packet.c network/packet.h network/socket.c network/socket.h \
+network/socket.c network/socket.h \
 network/socket_manager.c network/socket_manager.h \
 processing/jobs/acquire_job.c processing/jobs/acquire_job.h \
 processing/jobs/delete_child_sa_job.c processing/jobs/delete_child_sa_job.h \

diff --git a/src/libcharon/encoding/message.h b/src/libcharon/encoding/message.h
index 6f3c7967f5ff6a3387e2df9ee5bbab1d6c460758..6d558daf6a5b85ab38e1d3261393cd8cd9b47e9d 100644 (file)
--- a/src/libcharon/encoding/message.h
+++ b/src/libcharon/encoding/message.h
@@ -27,11 +27,11 @@
 typedef struct message_t message_t;
 
 #include <library.h>
-#include <network/packet.h>
 #include <encoding/payloads/ike_header.h>
 #include <encoding/payloads/notify_payload.h>
 #include <sa/keymat.h>
 #include <sa/ike_sa_id.h>
+#include <utils/packet.h>
 #include <utils/linked_list.h>
 
 /**

diff --git a/src/libcharon/network/receiver.c b/src/libcharon/network/receiver.c
index 3a52f8dc375acf71cb8e0ec74f4dac6326059b57..b270d65dfaab66577a338f696fb17aa2d57fa38c 100644 (file)
--- a/src/libcharon/network/receiver.c
+++ b/src/libcharon/network/receiver.c
@@ -22,12 +22,12 @@
 
 #include <daemon.h>
 #include <network/socket.h>
-#include <network/packet.h>
 #include <processing/jobs/job.h>
 #include <processing/jobs/process_message_job.h>
 #include <processing/jobs/callback_job.h>
 #include <crypto/hashers/hasher.h>
 #include <threading/mutex.h>
+#include <utils/packet.h>
 
 /** lifetime of a cookie, in seconds */
 #define COOKIE_LIFETIME 10

diff --git a/src/libcharon/network/receiver.h b/src/libcharon/network/receiver.h
index 93b3d3c0c932ee40e1020de3bb13ecbc7918dc6f..9e8edee459ad51b628cef9f0a2910dd8ebd5f2f7 100644 (file)
--- a/src/libcharon/network/receiver.h
+++ b/src/libcharon/network/receiver.h
@@ -26,8 +26,8 @@
 typedef struct receiver_t receiver_t;
 
 #include <library.h>
-#include <network/packet.h>
 #include <utils/host.h>
+#include <utils/packet.h>
 
 /**
  * Callback called for any received UDP encapsulated ESP packet.

diff --git a/src/libcharon/network/sender.h b/src/libcharon/network/sender.h
index c4f18d73b1cad9f5871b0ef88b351bdfdac38721..9b5c325cc76e567cafc9687e02a564173619a74c 100644 (file)
--- a/src/libcharon/network/sender.h
+++ b/src/libcharon/network/sender.h
@@ -26,7 +26,7 @@
 typedef struct sender_t sender_t;
 
 #include <library.h>
-#include <network/packet.h>
+#include <utils/packet.h>
 
 /**
  * Callback job responsible for sending IKE packets over the socket.

diff --git a/src/libcharon/network/socket.h b/src/libcharon/network/socket.h
index 4a4ef52e6aa2c8b88a0f1446fbce0ea8fd05e51e..b8850c6edd0f117e9d337d4f42fdbf5e49bf385b 100644 (file)
--- a/src/libcharon/network/socket.h
+++ b/src/libcharon/network/socket.h
@@ -27,7 +27,7 @@
 typedef struct socket_t socket_t;
 
 #include <library.h>
-#include <network/packet.h>
+#include <utils/packet.h>
 #include <utils/enumerator.h>
 #include <plugins/plugin.h>
 

diff --git a/src/libcharon/sa/ike_sa.h b/src/libcharon/sa/ike_sa.h
index e52355962c78ca694d6f7e7283dcb2bc4c1a7c27..de9e0ede4829dd854242aefb30068fe39d846345 100644 (file)
--- a/src/libcharon/sa/ike_sa.h
+++ b/src/libcharon/sa/ike_sa.h
@@ -43,6 +43,7 @@ typedef struct ike_sa_t ike_sa_t;
 #include <config/peer_cfg.h>
 #include <config/ike_cfg.h>
 #include <credentials/auth_cfg.h>
+#include <utils/packet.h>
 
 /**
  * Timeout in seconds after that a half open IKE_SA gets deleted.

diff --git a/src/libcharon/sa/ikev2/tasks/ike_mobike.h b/src/libcharon/sa/ikev2/tasks/ike_mobike.h
index a7e3fe7e3273803dc3c80a21a9c24a6afbd6a12f..3b447af516448f2ee90244474269219be513ed6a 100644 (file)
--- a/src/libcharon/sa/ikev2/tasks/ike_mobike.h
+++ b/src/libcharon/sa/ikev2/tasks/ike_mobike.h
@@ -26,7 +26,7 @@ typedef struct ike_mobike_t ike_mobike_t;
 #include <library.h>
 #include <sa/ike_sa.h>
 #include <sa/task.h>
-#include <network/packet.h>
+#include <utils/packet.h>
 
 /**
  * Task of type ike_mobike, detects and handles MOBIKE extension.

diff --git a/src/libstrongswan/Android.mk b/src/libstrongswan/Android.mk
index 8cba58816a663aac773abcbc9c1e4d9c9ca907fd..b6abf156f1457e33464126525818bf9ee5d5151a 100644 (file)
--- a/src/libstrongswan/Android.mk
+++ b/src/libstrongswan/Android.mk
@@ -24,7 +24,7 @@ pen/pen.c plugins/plugin_loader.c plugins/plugin_feature.c processing/jobs/job.c
 processing/jobs/callback_job.c processing/processor.c processing/scheduler.c \
 selectors/traffic_selector.c threading/thread.c threading/thread_value.c \
 threading/mutex.c threading/semaphore.c threading/rwlock.c threading/spinlock.c \
-utils.c utils/host.c utils/identification.c utils/lexparser.c \
+utils.c utils/host.c utils/packet.c utils/identification.c utils/lexparser.c \
 utils/linked_list.c utils/hashtable.c utils/enumerator.c utils/optionsfrom.c \
 utils/capabilities.c utils/backtrace.c
 

diff --git a/src/libstrongswan/Makefile.am b/src/libstrongswan/Makefile.am
index dc849d6f275ea76e3bb2b7c67936028c721839e5..b21f86f1ef329518d62e32568410004a53feb249 100644 (file)
--- a/src/libstrongswan/Makefile.am
+++ b/src/libstrongswan/Makefile.am
@@ -22,7 +22,7 @@ pen/pen.c plugins/plugin_loader.c plugins/plugin_feature.c processing/jobs/job.c
 processing/jobs/callback_job.c processing/processor.c processing/scheduler.c \
 selectors/traffic_selector.c threading/thread.c threading/thread_value.c \
 threading/mutex.c threading/semaphore.c threading/rwlock.c threading/spinlock.c \
-utils.c utils/host.c utils/identification.c utils/lexparser.c \
+utils.c utils/host.c utils/packet.c utils/identification.c utils/lexparser.c \
 utils/linked_list.c utils/hashtable.c utils/enumerator.c utils/optionsfrom.c \
 utils/capabilities.c utils/backtrace.c
 
@@ -57,8 +57,9 @@ processing/jobs/callback_job.h processing/processor.h processing/scheduler.h \
 selectors/traffic_selector.h threading/thread.h threading/thread_value.h \
 threading/mutex.h threading/condvar.h threading/spinlock.h threading/semaphore.h \
 threading/rwlock.h threading/lock_profiler.h utils.h utils/host.h \
-utils/identification.h utils/lexparser.h utils/linked_list.h utils/hashtable.h \
-utils/enumerator.h utils/optionsfrom.h utils/capabilities.h utils/backtrace.h
+utils/packet.h utils/identification.h utils/lexparser.h utils/linked_list.h \
+utils/hashtable.h utils/enumerator.h utils/optionsfrom.h utils/capabilities.h \
+utils/backtrace.h
 endif
 
 library.lo :   $(top_builddir)/config.status

diff --git a/src/libcharon/network/packet.c b/src/libstrongswan/utils/packet.c
similarity index 93%
rename from src/libcharon/network/packet.c
rename to src/libstrongswan/utils/packet.c
index c817e00fb24f6a7d2c4d4183c087b25d0b54fd11..b5716fc4bc63c7cc34f81c422b9a16dbaef59bfc 100644 (file)
--- a/src/libcharon/network/packet.c
+++ b/src/libstrongswan/utils/packet.c
@@ -110,15 +110,16 @@ METHOD(packet_t, clone_, packet_t*,
        packet_t *other;
 
        other = packet_create();
-       if (this->destination != NULL)
+       if (this->destination)
        {
-               other->set_destination(other, this->destination->clone(this->destination));
+               other->set_destination(other,
+                                                          this->destination->clone(this->destination));
        }
-       if (this->source != NULL)
+       if (this->source)
        {
                other->set_source(other, this->source->clone(this->source));
        }
-       if (this->data.ptr != NULL)
+       if (this->data.ptr)
        {
                other->set_data(other, chunk_clone(this->adjusted_data));
        }
@@ -128,7 +129,7 @@ METHOD(packet_t, clone_, packet_t*,
 /*
  * Documented in header
  */
-packet_t *packet_create(void)
+packet_t *packet_create()
 {
        private_packet_t *this;
 

diff --git a/src/libcharon/network/packet.h b/src/libstrongswan/utils/packet.h
similarity index 55%
rename from src/libcharon/network/packet.h
rename to src/libstrongswan/utils/packet.h
index c53364104aea9ae96c0ea0112bc742980584775f..9106bbef50e96b5d83bbae56dc6c77c9fd18be74 100644 (file)
--- a/src/libcharon/network/packet.h
+++ b/src/libstrongswan/utils/packet.h
@@ -17,7 +17,7 @@
 
 /**
  * @defgroup packet packet
- * @{ @ingroup network
+ * @{ @ingroup utils
  */
 
 #ifndef PACKET_H_
@@ -29,97 +29,83 @@ typedef struct packet_t packet_t;
 #include <utils/host.h>
 
 /**
- * Abstraction of an UDP-Packet, contains data, sender and receiver.
+ * Abstraction of an IP/UDP-Packet, contains data, sender and receiver.
  */
 struct packet_t {
 
        /**
         * Set the source address.
         *
-        * Set host_t is now owned by packet_t, it will destroy
-        * it if necessary.
-        *
-        * @param source        address to set as source
+        * @param source        address to set as source (gets owned)
         */
-       void (*set_source) (packet_t *packet, host_t *source);
+       void (*set_source)(packet_t *packet, host_t *source);
 
        /**
         * Set the destination address.
         *
-        * Set host_t is now owned by packet_t, it will destroy
-        * it if necessary.
-        *
-        * @param source        address to set as destination
+        * @param source        address to set as destination (gets owned)
         */
-       void (*set_destination) (packet_t *packet, host_t *destination);
+       void (*set_destination)(packet_t *packet, host_t *destination);
 
        /**
         * Get the source address.
         *
-        * Set host_t is still owned by packet_t, clone it
-        * if needed.
-        *
-        * @return                      source address
+        * @return                      source address (internal data)
         */
-       host_t *(*get_source) (packet_t *packet);
+       host_t *(*get_source)(packet_t *packet);
 
        /**
         * Get the destination address.
         *
-        * Set host_t is still owned by packet_t, clone it
-        * if needed.
-        *
-        * @return                      destination address
+        * @return                      destination address (internal data)
         */
-       host_t *(*get_destination) (packet_t *packet);
+       host_t *(*get_destination)(packet_t *packet);
 
        /**
         * Get the data from the packet.
         *
-        * The data pointed by the chunk is still owned
-        * by the packet. Clone it if needed.
-        *
-        * @return                      chunk containing the data
+        * @return                      chunk containing the data (internal data)
         */
-       chunk_t (*get_data) (packet_t *packet);
+       chunk_t (*get_data)(packet_t *packet);
 
        /**
         * Set the data in the packet.
         *
-        * Supplied chunk data is now owned by the
-        * packet. It will free it.
-        *
-        * @param data          chunk with data to set
+        * @param data          chunk with data to set (gets owned)
         */
-       void (*set_data) (packet_t *packet, chunk_t data);
+       void (*set_data)(packet_t *packet, chunk_t data);
 
        /**
         * Increase the offset where the actual packet data starts.
         *
+        * The total offset applies to future calls of get_data() and clone().
+        *
         * @note The offset is reset to 0 when set_data() is called.
         *
         * @param bytes         the number of additional bytes to skip
         */
-       void (*skip_bytes) (packet_t *packet, size_t bytes);
+       void (*skip_bytes)(packet_t *packet, size_t bytes);
 
        /**
         * Clones a packet_t object.
         *
+        * @note Data is cloned without skipped bytes.
+        *
         * @param clone         clone of the packet
         */
-       packet_t* (*clone) (packet_t *packet);
+       packet_t* (*clone)(packet_t *packet);
 
        /**
         * Destroy the packet, freeing contained data.
         */
-       void (*destroy) (packet_t *packet);
+       void (*destroy)(packet_t *packet);
 };
 
 /**
- * create an empty packet
+ * Create an empty packet
  *
  * @return packet_t object
  */
-packet_t *packet_create(void);
+packet_t *packet_create();
 
 #endif /** PACKET_H_ @}*/