Abort on init with duplicate options (i.e. options with the same name).

Reported by Thomas Adam.

Signed-off-by: Peter Rosin <peda@lysator.liu.se>

src/confuse.c

@@ -535,6 +535,20 @@ static void cfg_init_defaults(cfg_t *cfg)
 	int i;
 
 	for (i = 0; cfg->opts[i].name; i++) {
+		int j;
+		for (j = 0; j < i; ++j) {
+			if (strcmp(cfg->opts[i].name, cfg->opts[j].name))
+				continue;
+			/*
+			 * There are two definitions of the same option name.
+			 * What to do? It's a programming error and not an end
+			 * user input error. Lets print a message and abort...
+			 */
+			fprintf(stderr, "duplicate option '%s' not allowed\n",
+				cfg->opts[i].name);
+			abort();
+		}
+
 		/* libConfuse doesn't handle default values for "simple" options */
 		if (cfg->opts[i].simple_value.ptr || is_set(CFGF_NODEFAULT, cfg->opts[i].flags))
 			continue;