From: Arran Cudbard-Bell <a.cudbardb@freeradius.org>
Date: Wed, 18 Dec 2019 03:46:24 +0000 (+0700)
Subject: vb_date is microseconds since the epoch, i.e. an external time, and so SHOULD NOT... 
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=447b80a88213283d66bbfcc481c92e83f768f0d0;p=thirdparty%2Ffreeradius-server.git

vb_date is microseconds since the epoch, i.e. an external time, and so SHOULD NOT HAVE OUR INTERNAL TIME SUBTRACTED FROM IT
---

diff --git a/src/lib/util/value.c b/src/lib/util/value.c
index 01f0ff3ce6f..3453511e971 100644
--- a/src/lib/util/value.c
+++ b/src/lib/util/value.c
@@ -1468,7 +1468,7 @@ ssize_t fr_value_box_from_network(TALLOC_CTX *ctx,
 		switch (precision) {
 		default:
 		case FR_TIME_RES_SEC: /* external seconds, internal nanoseconds */
-			date *= 1000000000;
+			date *= NSEC;
 			break;
 
 		case FR_TIME_RES_MSEC:
@@ -1483,13 +1483,7 @@ ssize_t fr_value_box_from_network(TALLOC_CTX *ctx,
 			break;
 		}
 
-		/*
-		 *	Convert nanoseconds to struct timespec.
-		 */
-		ts.tv_sec = date / NSEC;
-		ts.tv_nsec = date % NSEC;
-
-		dst->vb_date = fr_time_from_timespec(&ts);
+		dst->vb_date = date;
 	}
 		break;