Nmap Development mailing list archives

Re: time_t changes in OpenBSD-current


From: Daniel Miller <bonsaiviking () gmail com>
Date: Mon, 6 Jan 2014 21:52:40 -0600

On Mon, Jan 6, 2014 at 9:35 PM, David Fifield <david () bamsoftware com> wrote:
On Sat, Jan 04, 2014 at 05:18:41PM +0100, Giovanni Bechis wrote:
in september OpenBSD changed time_t to long long on all platforms,
this change broke nmap on 32 bits platforms.
IMHO the main reason is that timing.cc class stores data in a double
and some precision is lost.
The following patch seems to fix the problem, maybe a bit more work is
needed.

Thanks for this patch. Do we need to also add a cast to (long long) so
it will work on other platforms where time_t is long or unsigned long?


Note that the %ll specifier is C99 and may not be supported on all
compilers (check VC++ perhaps?)

Dan

Ref: http://www.cplusplus.com/reference/cstdio/printf/
_______________________________________________
Sent through the dev mailing list
http://nmap.org/mailman/listinfo/dev
Archived at http://seclists.org/nmap-dev/


Current thread: