From 1afa7394f74134285baea24c23ef858b4ab78eff Mon Sep 17 00:00:00 2001 From: Linus Nordberg Date: Sun, 30 Jul 2017 21:01:36 +0200 Subject: Revert ed6f9b47. Going to errexit doesn't free resconf as that commit claims. It does free conf though, which is good. coverity: 1449524 --- radsecproxy.c | 3 +-- 1 file changed, 1 insertion(+), 2 deletions(-) diff --git a/radsecproxy.c b/radsecproxy.c index 0b3f52e..45508a8 100644 --- a/radsecproxy.c +++ b/radsecproxy.c @@ -3114,8 +3114,7 @@ int confserver_cb(struct gconffile **cf, void *arg, char *block, char *opt, char if (resconf || !conf->dynamiclookupcommand) { if (!compileserverconfig(conf, block)) - return 0; /* Don't goto errexit and free resconf -- it's - * not ours to free. */ + goto errexit; } if (!conf->secret) { -- cgit v1.1