From 569b8f3bc3c2e4f076f5ddf8ff4045b52775b8f9 Mon Sep 17 00:00:00 2001
From: Christian Grothoff <christian@grothoff.org>
Date: Fri, 10 Jun 2011 15:51:48 +0000
Subject: [PATCH] add API to remove section

---
 src/include/gnunet_configuration_lib.h | 10 ++++
 src/util/configuration.c               | 65 ++++++++++++++++++++------
 2 files changed, 60 insertions(+), 15 deletions(-)

diff --git a/src/include/gnunet_configuration_lib.h b/src/include/gnunet_configuration_lib.h
index 87aa8eacc..3cdd54c8f 100644
--- a/src/include/gnunet_configuration_lib.h
+++ b/src/include/gnunet_configuration_lib.h
@@ -174,6 +174,16 @@ void GNUNET_CONFIGURATION_iterate_sections (const struct GNUNET_CONFIGURATION_Ha
                                             GNUNET_CONFIGURATION_Section_Iterator iter,
                                             void *iter_cls);
 
+
+/**
+ * Remove the given section and all options in it.
+ *
+ * @param cfg configuration to inspect
+ * @param section name of the section to remove
+ */
+void GNUNET_CONFIGURATION_remove_section (struct GNUNET_CONFIGURATION_Handle *cfg,
+					  const char *section);
+
 /**
  * Get a configuration value that should be a number.
  *
diff --git a/src/util/configuration.c b/src/util/configuration.c
index 10f2d7575..4aa5fed5a 100644
--- a/src/util/configuration.c
+++ b/src/util/configuration.c
@@ -132,21 +132,9 @@ void
 GNUNET_CONFIGURATION_destroy (struct GNUNET_CONFIGURATION_Handle *cfg)
 {
   struct ConfigSection *sec;
-  struct ConfigEntry *ent;
 
   while (NULL != (sec = cfg->sections))
-    {
-      cfg->sections = sec->next;
-      while (NULL != (ent = sec->entries))
-        {
-          sec->entries = ent->next;
-          GNUNET_free (ent->key);
-          GNUNET_free_non_null (ent->val);
-          GNUNET_free (ent);
-        }
-      GNUNET_free (sec->name);
-      GNUNET_free (sec);
-    }
+    GNUNET_CONFIGURATION_remove_section (cfg, sec->name);
   GNUNET_free (cfg);
 }
 
@@ -414,9 +402,56 @@ GNUNET_CONFIGURATION_iterate_sections (const struct GNUNET_CONFIGURATION_Handle
                                        void *iter_cls)
 {
   struct ConfigSection *spos;
+  struct ConfigSection *next;
 
-  for (spos = cfg->sections; spos != NULL; spos = spos->next)
-    iter (iter_cls, spos->name);
+  next = cfg->sections; 
+  while (next != NULL)
+    {
+      spos = next;
+      next = spos->next;
+      iter (iter_cls, spos->name);
+    }
+}
+
+/**
+ * Remove the given section and all options in it.
+ *
+ * @param cfg configuration to inspect
+ * @param section name of the section to remove
+ */
+void GNUNET_CONFIGURATION_remove_section (struct GNUNET_CONFIGURATION_Handle *cfg,
+					  const char *section)
+{
+  struct ConfigSection *spos;
+  struct ConfigSection *prev;
+  struct ConfigEntry *ent;
+
+  prev = NULL;
+  spos = cfg->sections; 
+  while (spos != NULL) 	 
+    {
+      if (0 == strcmp (section,
+		       spos->name))
+	{
+	  if (prev == NULL)
+	    cfg->sections = spos->next;
+	  else
+	    prev->next = spos->next;
+	  while (NULL != (ent = spos->entries))
+	    {
+	      spos->entries = ent->next;
+	      GNUNET_free (ent->key);
+	      GNUNET_free_non_null (ent->val);
+	      GNUNET_free (ent);
+	      cfg->dirty = GNUNET_YES;
+	    }
+	  GNUNET_free (spos->name);
+	  GNUNET_free (spos);
+	  return;
+	}
+      prev = spos;
+      spos = spos->next;
+    }
 }
 
 
-- 
2.25.1