On 05/10/2012 11:23 PM, Peter Hutterer wrote:
> On Mon, Apr 16, 2012 at 11:14:22AM -0700, Chase Douglas wrote:
[snip]
>> void
>> +LogWriteSigSafe(int verb, const char *f, log_param_t *args, int num_args)
>> +{
>> + static char tmpBuffer[1024];
>> + int len = 0;
>> + char number[20];
>> + static Bool newline = TRUE;
>> +
>> + if (newline) {
>> + CARD32 ms = GetTimeInMillis();
>
> GetTimeInMillis calls gettimeofday() which isn't signal-safe, afaict.
Thanks for catching that. We could switch to clock_gettime(), but I'm
not sure what differences, if any, there would be between it and
gettimeofday(). For now, I've removed the timestamp printing. Someone
else can come along and add the timestamp back in if they want.
-- Chase
_______________________________________________
[email protected]: X.Org development
Archives: http://lists.x.org/archives/xorg-devel
Info: http://lists.x.org/mailman/listinfo/xorg-devel
