On Thu, Jul 17, 2003 at 02:25:16PM +0100, Paul Makepeace wrote:
> 
> /* buncha #ifndef/#defines for HOME, etc */
> 
> char * new_env[] = {
>     HOME,               /* 0 */
>     PATH,               /* 1 */
>     SHELL,              /* 2 */
> #ifdef MAJORDOMO_CF
>     MAJORDOMO_CF,       /* 3 */
> #endif
>     0,          /* possibly for USER or LOGNAME */
>     0,          /* possible for LOGNAME */
>     0,          /* possibly for timezone */
>     0
> };
> 
> /* later... */
> 
>     for (i = 0 ; env[i] != NULL && e <= new_env_size; i++) {
>         if ((strncmp(env[i], "USER=", 5) == 0) ||
>             (strncmp(env[i], "TZ=", 3) == 0) ||
>             (strncmp(env[i], "LOGNAME=", 8) == 0)) {
>             new_env[e++] = env[i];
>         }
>     }

Quick, tell me that new_env_size is smaller than 

   sizeof(new_env)/sizeof(char *) - 2;

--
Nick

Reply via email to