httpd-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Chuck Murcko <ch...@telebase.com>
Subject Re: problems w/patch 40a under IRIX
Date Tue, 21 Nov 1995 17:03:17 GMT
Chuck Murcko liltingly intones:
> 
> Ben Laurie liltingly intones:
> > 
> > > !#if !defined(IRIX) && !defined(LINUX)
> > > !     tz = lt->tm_zone;
> > > !#endif
> > > !     j = 0;
> > > !     if (tz != NULL) env[j++] = pstrcat(p, "TZ=", tz, NULL);
> > 
> > Oops! tz is uninitialized at this point under Irix/Linux. Core dumps loom.
> 
> Modern compilers will init tz to NULL. Were this an actual patch I would
> do it explicitly in the tz declaration, to be safe.

Oops. tz is not static or extern, so not in BSS. Must be initialized to NULL
to be safe.

chuck
Chuck Murcko	Telebase Systems, Inc.	Wayne PA	chuck@telebase.com
And now, on a lighter note:
Don't tell any big lies today.  Small ones can be just as effective.

Mime
View raw message