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:25:09 +0200 |
| commit | cc8fe7ffb1433e1f3ccf7ac67eda091818e612b7 (patch) | |
| tree | 7070744570d703745efe5256b17ebf1b0a27fd7f | |
| parent | f9ea851f333ba325703d7e6e0da0a66f0ecd8163 (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 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) { |