Skip to content

Commit

Permalink
Revert ed6f9b4.
Browse files Browse the repository at this point in the history
Going to errexit doesn't free resconf as that commit claims. It does
free conf though, which is good.

coverity: 1449524
  • Loading branch information
Linus Nordberg committed Aug 1, 2017
1 parent f9ea851 commit cc8fe7f
Showing 1 changed file with 1 addition and 2 deletions.
3 changes: 1 addition & 2 deletions radsecproxy.c
Original file line number Diff line number Diff line change
Expand Up @@ -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) {
Expand Down

0 comments on commit cc8fe7f

Please sign in to comment.