On Thu, May 18 2017, "Laurent Bercot" <ska-skaw...@skarnet.org> 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 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.