
25 May
2011
25 May
'11
11:53 a.m.
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. -- Eric Blake eblake@redhat.com +1-801-349-2682 Libvirt virtualization library http://libvirt.org