This is the mail archive of the libc-alpha@sourceware.org mailing list for the glibc project.
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |
Other format: | [Raw text] |
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
Attachment:
signature.asc
Description: Digital signature
Index Nav: | [Date Index] [Subject Index] [Author Index] [Thread Index] | |
---|---|---|
Message Nav: | [Date Prev] [Date Next] | [Thread Prev] [Thread Next] |