If done with popt context it should be free'd.
Signed-off-by: Swen Schillig <swen@linux.ibm.com>
Reviewed-by: Andrew Bartlett <abartlet@samba.org>
Reviewed-by: Ralph Böhme <slow@samba.org>
Autobuild-User(master): Andrew Bartlett <abartlet@samba.org>
Autobuild-Date(master): Thu Aug 22 01:41:37 UTC 2019 on sn-devel-184
{
int optidx = 0;
pid_t pid;
- poptContext pc;
+ poptContext pc = NULL;
const char *instruction_file;
const char **args;
const char *program;
if (argc == 1) {
poptPrintHelp(pc, stderr, 0);
- return 1;
+ goto out;
}
while ((optidx = poptGetNextOpt(pc)) != -1) {
args = poptGetArgs(pc);
if (args == NULL) {
poptPrintHelp(pc, stderr, 0);
- return 1;
+ goto out;
}
program_args = (char * const *)discard_const_p(char *, args);
err(1, "Failed to fork");
/* Never reached */
- return 1;
+ goto out;
case 0:
if(setsid()<0)
err(1, "Failed to exec: %s", program);
/* Never reached */
- return 1;
+ goto out;
default:
close(slave);
{
sigaction(SIGALRM, &sa, NULL);
}
+ poptFreeContext(pc);
return eval_parent(pid);
}
/* Never reached */
+
+out:
+ poptFreeContext(pc);
return 1;
}