Commit cd9e5db0 authored by Poul-Henning Kamp's avatar Poul-Henning Kamp

GCC doesn't think NULL is a valid terminator of execl(2).

I don't think GCC is a valid compiler for sensible code.
parent 77e6d3ae
......@@ -339,7 +339,8 @@ process_start(struct process *p)
assert(dup2(fd2[1], STDERR_FILENO) == STDERR_FILENO);
VSUB_closefrom(STDERR_FILENO + 1);
AZ(setpgid(0, 0));
AZ(execl("/bin/sh", "/bin/sh", "-c", VSB_data(cl), NULL));
// Not using NULL because GCC is now even more demented...
AZ(execl("/bin/sh", "/bin/sh", "-c", VSB_data(cl), (char*)0));
exit(1);
}
vtc_log(p->vl, 3, "PID: %ld", (long)p->pid);
......
Markdown is supported
0% or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment