Home page logo
/

nmap-dev logo Nmap Development mailing list archives

Re: time_t changes in OpenBSD-current
From: David Fifield <david () bamsoftware com>
Date: Tue, 7 Jan 2014 08:23:27 -0800

On Mon, Jan 06, 2014 at 09:52:40PM -0600, Daniel Miller wrote:
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?)

I think we already use %ll in a few places, so it should be okay.

David Fifield
_______________________________________________
Sent through the dev mailing list
http://nmap.org/mailman/listinfo/dev
Archived at http://seclists.org/nmap-dev/


  By Date           By Thread  

Current thread:
[ Nmap | Sec Tools | Mailing Lists | Site News | About/Contact | Advertising | Privacy ]