From 6758bf4952922c028dab2a926848554291d8ad14 Mon Sep 17 00:00:00 2001 From: tedu <> Date: Mon, 5 Jan 2015 13:10:10 +0000 Subject: [PATCH] convert clock() to clock_gettime() for improved precision (and accuracy?) guenther suggested using thread time, which actually may improve accuracy if somebody puts this in a threaded program. --- src/lib/libc/crypt/bcrypt.c | 16 +++++++++------- 1 file changed, 9 insertions(+), 7 deletions(-) diff --git a/src/lib/libc/crypt/bcrypt.c b/src/lib/libc/crypt/bcrypt.c index abcbe138..1114eae4 100644 --- a/src/lib/libc/crypt/bcrypt.c +++ b/src/lib/libc/crypt/bcrypt.c @@ -1,4 +1,4 @@ -/* $OpenBSD: bcrypt.c,v 1.47 2014/12/30 10:27:24 tedu Exp $ */ +/* $OpenBSD: bcrypt.c,v 1.48 2015/01/05 13:10:10 tedu Exp $ */ /* * Copyright (c) 2014 Ted Unangst @@ -231,24 +231,26 @@ bcrypt_checkpass(const char *pass, const char *goodhash) int bcrypt_autorounds(void) { - clock_t before, after; + struct timespec before, after; int r = 8; char buf[_PASSWORD_LEN]; int duration; - before = clock(); + clock_gettime(CLOCK_THREAD_CPUTIME_ID, &before); bcrypt_newhash("testpassword", r, buf, sizeof(buf)); - after = clock(); + clock_gettime(CLOCK_THREAD_CPUTIME_ID, &after); - duration = after - before; + duration = after.tv_sec - before.tv_sec; + duration *= 1000000; + duration += (after.tv_nsec - before.tv_nsec) / 1000; /* too quick? slow it down. */ - while (r < 16 && duration <= CLOCKS_PER_SEC / 4) { + while (r < 16 && duration <= 1000000 / 4) { r += 1; duration *= 2; } /* too slow? speed it up. */ - while (r > 4 && duration > CLOCKS_PER_SEC / 2) { + while (r > 4 && duration > 1000000 / 2) { r -= 1; duration /= 2; }