>> - setenv("SHELL", sh, 1);
>> + setenv("SHELL", args[0], 1);
>
> I'm not sure whether this is a good idea? The user asked to execute
> args[0] instead of the shell, but this doesn't imply that args[0] is
> his shell.
Yes, you are right, it should be setenv("SHELL", sh, 1);
Can you send a patch for it?
Regards,
