--- libaitcfg/src/aitcfg.c	2014/01/29 23:48:34	1.10
+++ libaitcfg/src/aitcfg.c	2021/11/26 01:15:03	1.15
@@ -3,7 +3,7 @@
 *  by Michael Pounov <misho@openbsd-bg.org>
 *
 * $Author: misho $
-* $Id: aitcfg.c,v 1.10 2014/01/29 23:48:34 misho Exp $
+* $Id: aitcfg.c,v 1.15 2021/11/26 01:15:03 misho Exp $
 *
 **************************************************************************
 The ELWIX and AITNET software is distributed under the following
@@ -12,7 +12,7 @@ terms:
 All of the documentation and software included in the ELWIX and AITNET
 Releases is copyrighted by ELWIX - Sofia/Bulgaria <info@elwix.org>
 
-Copyright 2004 - 2014
+Copyright 2004 - 2021
 	by Michael Pounov <misho@elwix.org>.  All rights reserved.
 
 Redistribution and use in source and binary forms, with or without
@@ -71,8 +71,8 @@ cfg_tree_cmp(struct tagCfg *a, struct tagCfg *b)
 
 	assert(a && b);
 
-	ret = ((AIT_KEY(&a->cfg_sec) << 16) | AIT_KEY(&a->cfg_attr)) - 
-		((AIT_KEY(&b->cfg_sec) << 16) | AIT_KEY(&b->cfg_attr));
+	ret = ((AIT_KEY(&a->cfg_sec) << 15) | AIT_KEY(&a->cfg_attr)) - 
+		((AIT_KEY(&b->cfg_sec) << 15) | AIT_KEY(&b->cfg_attr));
 
 	if (ret < 0)
 		return -1;
@@ -118,23 +118,45 @@ cfg_SetErr(int eno, char *estr, ...)
 /*
  * cfgInitConfig() - Init config root
  *
- * @cfg = Config root
- * return: -1 error or 0 ok
+ * return: NULL error or !=NULL allocated config root
  */
-int
-cfgInitConfig(cfg_root_t * __restrict cfg)
+cfg_root_t *
+cfgInitConfig()
 {
-	if (!cfg)
-		return -1;
+	cfg_root_t *cfg = NULL;
 
+	cfg = e_malloc(sizeof(cfg_root_t));
+	if (!cfg) {
+		cfg_SetErr(elwix_GetErrno(), "%s", elwix_GetError());
+		return NULL;
+	} else
+		memset(cfg, 0, sizeof(cfg_root_t));
+
 	pthread_mutex_init(&cfg->rc_mtx, NULL);
 
-	SLIST_INIT(cfg);
+	TAILQ_INIT(cfg);
 	RB_INIT(cfg);
-	return 0;
+	return cfg;
 }
 
 /*
+ * cfgEndConfig() - Free resources & config root
+ *
+ * @pcfg = Config root
+ * return: none
+ */
+void
+cfgEndConfig(cfg_root_t **pcfg)
+{
+	if (pcfg && *pcfg) {
+		cfgClearConfig(*pcfg);
+		pthread_mutex_destroy(&(*pcfg)->rc_mtx);
+		e_free(*pcfg);
+		*pcfg = NULL;
+	}
+}
+
+/*
  * cfgLoadConfig() - Load config from file
  *
  * @cfgName = Config filename
@@ -150,9 +172,15 @@ cfgLoadConfig(const char *cfgName, cfg_root_t * __rest
 	if (!cfgName || !cfg) {
 		cfg_SetErr(EINVAL, "Invalid parameter(s)");
 		return -1;
-	} else
-		cfgInitConfig(cfg);
+	} else {
+		memset(cfg, 0, sizeof(cfg_root_t));
 
+		pthread_mutex_init(&cfg->rc_mtx, NULL);
+
+		TAILQ_INIT(cfg);
+		RB_INIT(cfg);
+	}
+
 	f = fopen(cfgName, "r");
 	if (!f) {
 		LOGERR;
@@ -180,8 +208,8 @@ cfgClearConfig(cfg_root_t * __restrict cfg)
 		return;
 
 	CFG_RC_LOCK(cfg);
-	while ((av = SLIST_FIRST(cfg))) {
-		SLIST_REMOVE_HEAD(cfg, cfg_next);
+	while ((av = TAILQ_FIRST(cfg))) {
+		TAILQ_REMOVE(cfg, av, cfg_next);
 
 		AIT_FREE_VAL(&av->cfg_val);
 		AIT_FREE_VAL(&av->cfg_attr);
@@ -240,15 +268,20 @@ cfgCreateConfig(const char *csConfigName, cfg_root_t *
 /*
  * cfgInitPasswd() - Init password root
  *
- * @pwd = Password root
- * return: -1 error or 0 ok
+ * return: NULL error or !=NULL allocated password root
  */
-int
-cfgInitPasswd(pwd_root_t * __restrict pwd)
+pwd_root_t *
+cfgInitPasswd()
 {
-	if (!pwd)
-		return -1;
+	pwd_root_t *pwd = NULL;
 
+	pwd = e_malloc(sizeof(pwd_root_t));
+	if (!pwd) {
+		cfg_SetErr(elwix_GetErrno(), "%s", elwix_GetError());
+		return NULL;
+	} else
+		memset(pwd, 0, sizeof(pwd_root_t));
+
 	pthread_mutex_init(&pwd->pwd_mtx, NULL);
 
 	SLIST_INIT(pwd);
@@ -257,6 +290,23 @@ cfgInitPasswd(pwd_root_t * __restrict pwd)
 }
 
 /*
+ * cfgEndPasswd() - Free resources & password root
+ *
+ * @ppwd = Password root
+ * return: none
+ */
+void
+cfgEndPasswd(pwd_root_t **ppwd)
+{
+	if (ppwd && *ppwd) {
+		cfgClearPasswd(*ppwd);
+		pthread_mutex_destroy(&(*ppwd)->pwd_mtx);
+		e_free(*ppwd);
+		*ppwd = NULL;
+	}
+}
+
+/*
  * cfgLoadPasswd() - Load passwords from file
  *
  * @pwdName = Passwords filename
@@ -272,8 +322,14 @@ cfgLoadPasswd(const char *pwdName, pwd_root_t * __rest
 	if (!pwdName || !pwd) {
 		cfg_SetErr(EINVAL, "Invalid parameter(s)");
 		return -1;
-	} else
-		cfgInitPasswd(pwd);
+	} else {
+		memset(pwd, 0, sizeof(pwd_root_t));
+
+		pthread_mutex_init(&pwd->pwd_mtx, NULL);
+
+		SLIST_INIT(pwd);
+		RB_INIT(pwd);
+	}
 
 	f = fopen(pwdName, "r");
 	if (!f) {