On Thu, May 18 2017, "Laurent Bercot" <[email protected]> wrote:
>>In exec.c, we already win a little by doing a single call of >>pathexec_run instead of having each branch do their own > > This change is invalid. When you exit the if (dash) block, the > dashed[] array goes out of scope, and is deallocated; but argv[0] is > still pointing to it. The duplication of the call to pathexec_run() > is necessary. > Right, sorry about that. Not a big fan of VLAs, maybe that's why I misread the code.
