philipp / rpms / dhcp

Forked from rpms/dhcp 4 years ago
Clone

Blame SOURCES/dhcp-4.2.4-P1-interval.patch

54343e
diff -up dhcp-4.2.4/common/dispatch.c.foo dhcp-4.2.4/common/dispatch.c
54343e
--- dhcp-4.2.4/common/dispatch.c.foo	2012-07-26 21:31:43.875349675 -0500
54343e
+++ dhcp-4.2.4/common/dispatch.c	2012-07-26 21:39:14.961710319 -0500
54343e
@@ -324,7 +324,20 @@ void add_timeout (when, where, what, ref
54343e
 	q->next  = timeouts;
54343e
 	timeouts = q;
54343e
 
54343e
-	isc_interval_set(&interval, sec & DHCP_SEC_MAX, usec * 1000);
54343e
+	/* isc_time_nowplusinterval() is not safe with 64-bit time_t and will
54343e
+	 * return an error for sufficiently large intervals.  We have to limit
54343e
+	 * the interval to INT_MAX or less to ensure the interval doesn't
54343e
+	 * overflow 32 bits, since the returned isc_time_t fields are
54343e
+	 * 32-bit unsigned ints.
54343e
+	 *
54343e
+	 * HACK: The 9 is a magic number of seconds, since some time may have
54343e
+	 * gone by since the last call to gettimeofday() and the one in
54343e
+	 * isc_time_nowplusinterval().
54343e
+	 */
54343e
+	if (sec > TIME_MAX)
54343e
+		sec = TIME_MAX - 9;
54343e
+
54343e
+	isc_interval_set(&interval, sec, usec * 1000);
54343e
 	status = isc_time_nowplusinterval(&expires, &interval);
54343e
 	if (status != ISC_R_SUCCESS) {
54343e
 		/*