2012-06-19 17:03:24 +00:00
|
|
|
pppd: Watch out for time warps
|
|
|
|
|
|
|
|
On many embedded systems there is no battery backed RTC and a proper system
|
|
|
|
time only becomes available through NTP after establishing a connection.
|
|
|
|
|
|
|
|
When the clock suddenly jumps forward, the internal accounting (connect time)
|
|
|
|
is confused resulting in unreliable data.
|
|
|
|
|
|
|
|
This patch implements periodic clock checking to look for time warps, if one
|
|
|
|
is detected, the internal counters are adjusted accordingly.
|
|
|
|
|
|
|
|
Signed-off-by: Jo-Philipp Wich <jow@openwrt.org>
|
|
|
|
|
2009-05-21 10:49:21 +00:00
|
|
|
--- a/pppd/main.c
|
|
|
|
+++ b/pppd/main.c
|
2009-05-20 22:36:38 +00:00
|
|
|
@@ -90,6 +90,7 @@
|
|
|
|
#include <sys/socket.h>
|
|
|
|
#include <netinet/in.h>
|
|
|
|
#include <arpa/inet.h>
|
|
|
|
+#include <sys/sysinfo.h>
|
|
|
|
|
|
|
|
#include "pppd.h"
|
|
|
|
#include "magic.h"
|
2011-07-28 16:51:12 +00:00
|
|
|
@@ -228,6 +229,7 @@ static struct subprocess *children;
|
2009-05-20 22:36:38 +00:00
|
|
|
|
|
|
|
/* Prototypes for procedures local to this file. */
|
|
|
|
|
|
|
|
+static void check_time(void);
|
|
|
|
static void setup_signals __P((void));
|
|
|
|
static void create_pidfile __P((int pid));
|
|
|
|
static void create_linkpidfile __P((int pid));
|
2011-10-20 21:42:07 +00:00
|
|
|
@@ -535,6 +537,7 @@ main(argc, argv)
|
2009-05-20 22:36:38 +00:00
|
|
|
info("Starting link");
|
|
|
|
}
|
|
|
|
|
|
|
|
+ check_time();
|
|
|
|
gettimeofday(&start_time, NULL);
|
|
|
|
script_unsetenv("CONNECT_TIME");
|
|
|
|
script_unsetenv("BYTES_SENT");
|
2011-10-20 21:42:07 +00:00
|
|
|
@@ -1267,6 +1270,36 @@ struct callout {
|
2009-05-20 22:36:38 +00:00
|
|
|
|
|
|
|
static struct callout *callout = NULL; /* Callout list */
|
|
|
|
static struct timeval timenow; /* Current time */
|
|
|
|
+static long uptime_diff = 0;
|
|
|
|
+static int uptime_diff_set = 0;
|
|
|
|
+
|
|
|
|
+static void check_time(void)
|
|
|
|
+{
|
|
|
|
+ long new_diff;
|
|
|
|
+ struct timeval t;
|
|
|
|
+ struct sysinfo i;
|
|
|
|
+ struct callout *p;
|
|
|
|
+
|
|
|
|
+ gettimeofday(&t, NULL);
|
|
|
|
+ sysinfo(&i);
|
|
|
|
+ new_diff = t.tv_sec - i.uptime;
|
|
|
|
+
|
|
|
|
+ if (!uptime_diff_set) {
|
|
|
|
+ uptime_diff = new_diff;
|
|
|
|
+ uptime_diff_set = 1;
|
|
|
|
+ return;
|
|
|
|
+ }
|
|
|
|
+
|
|
|
|
+ if ((new_diff - 5 > uptime_diff) || (new_diff + 5 < uptime_diff)) {
|
|
|
|
+ /* system time has changed, update counters and timeouts */
|
|
|
|
+ info("System time change detected.");
|
|
|
|
+ start_time.tv_sec += new_diff - uptime_diff;
|
|
|
|
+
|
|
|
|
+ for (p = callout; p != NULL; p = p->c_next)
|
|
|
|
+ p->c_time.tv_sec += new_diff - uptime_diff;
|
|
|
|
+ }
|
|
|
|
+ uptime_diff = new_diff;
|
|
|
|
+}
|
|
|
|
|
|
|
|
/*
|
|
|
|
* timeout - Schedule a timeout.
|
2011-10-20 21:42:07 +00:00
|
|
|
@@ -1337,6 +1370,8 @@ calltimeout()
|
2009-05-20 22:36:38 +00:00
|
|
|
{
|
|
|
|
struct callout *p;
|
|
|
|
|
|
|
|
+ check_time();
|
|
|
|
+
|
|
|
|
while (callout != NULL) {
|
|
|
|
p = callout;
|
|
|
|
|
2011-10-20 21:42:07 +00:00
|
|
|
@@ -1364,6 +1399,8 @@ timeleft(tvp)
|
2009-05-20 22:36:38 +00:00
|
|
|
{
|
|
|
|
if (callout == NULL)
|
|
|
|
return NULL;
|
|
|
|
+
|
|
|
|
+ check_time();
|
|
|
|
|
|
|
|
gettimeofday(&timenow, NULL);
|
|
|
|
tvp->tv_sec = callout->c_time.tv_sec - timenow.tv_sec;
|