[libvirt] [PATCH v2] virsh: time_t is not a long on FreeBSD
Matthias Bolte
matthias.bolte at googlemail.com
Wed May 25 16:52:48 UTC 2011
2011/5/25 Eric Blake <eblake at redhat.com>:
> On 05/25/2011 09:06 AM, Matthias Bolte wrote:
>> localtime_r expects time_t.
>> ---
>> v2: add overflow check
>
> ACK.
>
>> - localtime_r(&creation, &time_info);
>> + creation_time_t = creation_longlong;
>> + if (creation_time_t != creation_longlong) {
>> + vshError(ctl, "%s", _("time_t overflow"));
>
> We shouldn't hit this error until 2038, and by then, hopefully more
> 32-bit platforms (if any are still running) will have switched to 64-bit
> time_t. :) But I agree with keeping this error in the code, since
> silent wraparound is always worse than explicit mention of overflow.
I like to be on the safe side :)
Thanks, pushed.
Matthias
More information about the libvir-list
mailing list