On 19 Feb 2016 16:05, Adhemerval Zanella wrote:
>  int
>  execl (const char *path, const char *arg, ...)
>  {
> ...
> +  char *argv[argc+1];

style nit: needs spaces around the +.  comes up in this patch more than once.

> +  argv[0] = (char*) arg;

space before the *

>  int
>  execle (const char *path, const char *arg, ...)
>  {
> ...
> +  va_start (ap, arg);
> +  argv[0] = (char*) arg;
> +  for (i = 1; i < argc; i++)
> +     argv[i] = va_arg (ap, char *);
> +  envp = va_arg (ap, char **);

is argv missing a NULL terminator ?
-mike