[Bug 11382] New: elapsed time calculation does not account for time function possibly returning -1
samba-bugs at samba.org
samba-bugs at samba.org
Sat Jul 4 04:38:29 UTC 2015
https://bugzilla.samba.org/show_bug.cgi?id=11382
Bug ID: 11382
Summary: elapsed time calculation does not account for time
function possibly returning -1
Product: rsync
Version: 3.1.1
Hardware: All
OS: All
Status: NEW
Severity: normal
Priority: P5
Component: core
Assignee: wayned at samba.org
Reporter: brant at gurganus.name
QA Contact: rsync-qa at samba.org
starttime currently initialized on line 1529 of main.c and endtime currently
initialized on line 222 of main.c, both of which are used on line 344 are used
in a manner that does not handle that time(NULL) can return -1.
Out of curiosity, I did try reading through the glibc source code for that
function to find it currently does return -1, and at the glibc level, that's
all that the time function does unless appropriate bridging to a kernel is
implemented. I did not take the time to try looking at other libc
implementations or to see under what conditions the linux kernel might return
-1.
As I only see one usage on line 344, if one of those two values comes back as
-1 for some reason, perhaps that information line should be ommitted or
replaced since the statistic trying to be printed would not be a valid
statistic if one of the two time() calls returned -1.
Glibc documents the -1 possibility at
http://www.gnu.org/software/libc/manual/html_node/Simple-Calendar-Time.html#Simple-Calendar-Time
Microsoft documents the -1 possibility for their libc implementations as well
at
https://msdn.microsoft.com/en-us/library/1f4c8f33.aspx?f=255&MSPPError=-2147217396
--
You are receiving this mail because:
You are the QA Contact for the bug.
More information about the rsync
mailing list