diff options
| author | Linus Nordberg <linus@nordu.net> | 2017-07-30 21:01:36 +0200 |
|---|---|---|
| committer | Linus Nordberg <linus@nordu.net> | 2017-08-01 18:10:20 +0200 |
| commit | 001cb9c6a5d88d012036f79baaa625543eb54bc0 (patch) | |
| tree | 8c03486b1d44428881a1990496f65fd9bda67ec4 | |
| parent | acb84e532506055e38be11ed000eb0e27d4aeb01 (diff) | |
Revert ed6f9b47.
Going to errexit doesn't free resconf as that commit claims. It does
free conf though, which is good.
coverity: 1449524
| -rw-r--r-- | radsecproxy.c | 3 |
1 files changed, 1 insertions, 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) { |