From 001cb9c6a5d88d012036f79baaa625543eb54bc0 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 9289354..9b3d5d2 100644 --- a/radsecproxy.c +++ b/radsecproxy.c @@ -2998,8 +2998,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