From cc8fe7ffb1433e1f3ccf7ac67eda091818e612b7 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 ccedc0d..4abd8a7 100644 --- a/radsecproxy.c +++ b/radsecproxy.c @@ -3105,8 +3105,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