Segment manipulation in HA sync is thread save

src/charon/plugins/ha_sync/ha_sync_segments.c

@@ -15,6 +15,7 @@
 
 #include "ha_sync_segments.h"
 
+#include <utils/mutex.h>
 #include <utils/linked_list.h>
 
 typedef u_int32_t u32;
@@ -36,6 +37,11 @@ struct private_ha_sync_segments_t {
 	 */
 	ha_sync_segments_t public;
 
+	/**
+	 * read/write lock for segment manipulation
+	 */
+	rwlock_t *lock;
+
 	/**
 	 * Init value for jhash
 	 */
@@ -121,6 +127,8 @@ static void enable_disable(private_ha_sync_segments_t *this, u_int segment,
 	enumerator_t *enumerator;
 	u_int i, limit;
 
+	this->lock->write_lock(this->lock);
+
 	if (segment == 0 || segment <= this->segment_count)
 	{
 		if (segment)
@@ -160,6 +168,8 @@ static void enable_disable(private_ha_sync_segments_t *this, u_int segment,
 
 		log_segments(this, enable, segment);
 	}
+
+	this->lock->unlock(this->lock);
 }
 
 /**
@@ -213,10 +223,12 @@ static void resync(private_ha_sync_segments_t *this, u_int segment)
 	ike_sa_id_t *id;
 	u_int16_t mask = bit_of(segment);
 
+	list = linked_list_create();
+	this->lock->read_lock(this->lock);
+
 	if (segment > 0 && segment <= this->segment_count && (this->active & mask))
 	{
 		this->active &= ~mask;
-		list = linked_list_create();
 
 		DBG1(DBG_CFG, "resyncing HA sync segment %d", segment);
 
@@ -234,6 +246,8 @@ static void resync(private_ha_sync_segments_t *this, u_int segment)
 			}
 		}
 		enumerator->destroy(enumerator);
+	}
+	this->lock->unlock(this->lock);
 
 	while (list->remove_last(list, (void**)&id) == SUCCESS)
 	{
@@ -257,13 +271,13 @@ static void resync(private_ha_sync_segments_t *this, u_int segment)
 	}
 	list->destroy(list);
 }
-}
 
 /**
  * Implementation of ha_sync_segments_t.destroy.
  */
 static void destroy(private_ha_sync_segments_t *this)
 {
+	this->lock->destroy(this->lock);
 	free(this);
 }
 
@@ -282,6 +296,7 @@ ha_sync_segments_t *ha_sync_segments_create()
 	this->public.resync = (void(*)(ha_sync_segments_t*, u_int segment))resync;
 	this->public.destroy = (void(*)(ha_sync_segments_t*))destroy;
 
+	this->lock = rwlock_create(RWLOCK_TYPE_DEFAULT);
 	this->initval = 0;
 	this->active = 0;
 	this->segment_count = lib->settings->get_int(lib->settings,