summaryrefslogtreecommitdiff
diff options
context:
space:
mode:
authorvenaas <venaas>2008-05-13 14:28:45 +0000
committervenaas <venaas@e88ac4ed-0b26-0410-9574-a7f39faa03bf>2008-05-13 14:28:45 +0000
commit748743942feb02681431be1ebc1ac4d04367a362 (patch)
tree3802c2262a23fb1ad4d582ecba7eeb927622310f
parent099ecc114be585f16a94dbc0989286b7306509c1 (diff)
fixed some bugs with free/alloc of strings and new gconfig
git-svn-id: https://svn.testnett.uninett.no/radsecproxy/trunk@252 e88ac4ed-0b26-0410-9574-a7f39faa03bf
-rw-r--r--catgconf.c4
-rw-r--r--gconfig.c40
2 files changed, 28 insertions, 16 deletions
diff --git a/catgconf.c b/catgconf.c
index dc0f312..f3d7b9c 100644
--- a/catgconf.c
+++ b/catgconf.c
@@ -6,10 +6,12 @@
#include "gconfig.h"
void listconfig(struct gconffile **cf, char *block, int compact) {
- char *opt, *val;
+ char *opt = NULL, *val = NULL;
int conftype;
for (;;) {
+ free(opt);
+ free(val);
getconfigline(cf, block, &opt, &val, &conftype);
if (!opt)
return;
diff --git a/gconfig.c b/gconfig.c
index 3e40139..7a367e7 100644
--- a/gconfig.c
+++ b/gconfig.c
@@ -18,6 +18,14 @@
#include "util.h"
#include "gconfig.h"
+char *mystringcopyx(const char *s) {
+ char *t;
+ t = stringcopy(s, 0);
+ if (!t)
+ debugx(1, DBG_ERR, "malloc failed");
+ return t;
+}
+
/* returns NULL on error, where to continue parsing if token and ok. E.g. "" will return token with empty string */
char *strtokenquote(char *s, char **token, char *del, char *quote, char *comment) {
char *t = s, *q, *r;
@@ -74,7 +82,7 @@ FILE *pushgconffile(struct gconffile **cf, const char *path) {
memmove(newcf + 1, newcf, sizeof(struct gconffile) * (i + 1));
}
newcf[0].file = f;
- newcf[0].path = stringcopy(path, 0);
+ newcf[0].path = mystringcopyx(path);
*cf = newcf;
return f;
}
@@ -90,9 +98,7 @@ FILE *pushgconffiles(struct gconffile **cf, const char *cfgpath) {
path = (char *)cfgpath;
else {
/* dirname may modify its argument */
- curfile = stringcopy((*cf)->path, 0);
- if (!curfile)
- debugx(1, DBG_ERR, "malloc failed");
+ curfile = mystringcopyx((*cf)->path);
dir = dirname(curfile);
path = malloc(strlen(dir) + strlen(cfgpath) + 2);
if (!path)
@@ -129,6 +135,7 @@ FILE *popgconffile(struct gconffile **cf) {
if (i && (*cf)[0].file) {
fclose((*cf)[0].file);
debug(DBG_DBG, "closing config file %s", (*cf)[0].path);
+ free((*cf)[0].path);
}
if (i < 2) {
free(*cf);
@@ -198,20 +205,20 @@ void getconfigline(struct gconffile **cf, char *block, char **opt, char **val, i
switch (tcount) {
case 2:
- *opt = tokens[0];
- *val = tokens[1];
+ *opt = mystringcopyx(tokens[0]);
+ *val = mystringcopyx(tokens[1]);
*conftype = CONF_STR;
break;
case 3:
if (tokens[1][0] == '=' && tokens[1][1] == '\0') {
- *opt = tokens[0];
- *val = tokens[2];
+ *opt = mystringcopyx(tokens[0]);
+ *val = mystringcopyx(tokens[2]);
*conftype = CONF_STR;
break;
}
if (tokens[2][0] == '{' && tokens[2][1] == '\0') {
- *opt = tokens[0];
- *val = tokens[1];
+ *opt = mystringcopyx(tokens[0]);
+ *val = mystringcopyx(tokens[1]);
*conftype = CONF_CBK;
break;
}
@@ -229,12 +236,13 @@ void getconfigline(struct gconffile **cf, char *block, char **opt, char **val, i
void getgenericconfig(struct gconffile **cf, char *block, ...) {
va_list ap;
- char *opt, *val, *word, *optval, **str = NULL, ***mstr = NULL;
+ char *opt = NULL, *val, *word, *optval, **str = NULL, ***mstr = NULL;
uint8_t *bln;
int type = 0, conftype = 0, n;
void (*cbk)(struct gconffile **, char *, char *, char *) = NULL;
for (;;) {
+ free(opt);
getconfigline(cf, block, &opt, &val, &conftype);
if (!opt)
return;
@@ -242,6 +250,7 @@ void getgenericconfig(struct gconffile **cf, char *block, ...) {
if (conftype == CONF_STR && !strcasecmp(opt, "include")) {
if (!pushgconffiles(cf, val))
debugx(1, DBG_ERR, "failed to include config file %s", val);
+ free(val);
continue;
}
@@ -292,9 +301,7 @@ void getgenericconfig(struct gconffile **cf, char *block, ...) {
case CONF_STR:
if (*str)
debugx(1, DBG_ERR, "configuration error, option %s already set to %s", opt, *str);
- *str = stringcopy(val, 0);
- if (!*str)
- debugx(1, DBG_ERR, "malloc failed");
+ *str = val;
break;
case CONF_MSTR:
if (*mstr)
@@ -304,7 +311,7 @@ void getgenericconfig(struct gconffile **cf, char *block, ...) {
*mstr = realloc(*mstr, sizeof(char *) * (n + 2));
if (!*mstr)
debugx(1, DBG_ERR, "malloc failed");
- (*mstr)[n] = stringcopy(val, 0);
+ (*mstr)[n] = val;
(*mstr)[n + 1] = NULL;
break;
case CONF_BLN:
@@ -323,6 +330,7 @@ void getgenericconfig(struct gconffile **cf, char *block, ...) {
debugx(1, DBG_ERR, "malloc failed");
sprintf(optval, "%s %s", opt, val);
cbk(cf, optval, opt, val);
+ free(val);
free(optval);
continue;
default:
@@ -332,5 +340,7 @@ void getgenericconfig(struct gconffile **cf, char *block, ...) {
debug(DBG_DBG, "getgenericconfig: block %s: %s = %s", block, opt, val);
else
debug(DBG_DBG, "getgenericconfig: %s = %s", opt, val);
+ if (type == CONF_BLN)
+ free(val);
}
}