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