f8b58757d0
0d004db Revert "pppd: Include time.h before using time_t" e400854 pppdump: Eliminate printf format warning by using %zd 7f2f0de pppd: Refactor setjmp/longjmp with pipe pair in event wait loop 4e71317 make: Avoid using host include for cross-compiling 3202f89 pppoe: Remove the use of cdefs d8e8d7a pppd: Remove unused rcsid variables 486f854 pppd: Fix GLIBC version test for non-glibc toolchains b6cd558 pppd: Include time.h before using time_t ef8ec11 radius: Fix compiler warning f6330ec magic: Remove K&R style of arguments 347904e Add Submitting-patches.md Remove patches 130-no_cdefs_h.patch, 131-missing_prototype_macro.patch, 132-fix_linux_includes.patch as fixed upstream Refresh patches Signed-off-by: Hans Dedecker <dedeckeh@gmail.com>
94 lines
2.3 KiB
Diff
94 lines
2.3 KiB
Diff
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 <jo@mein.io>
|
|
|
|
--- a/pppd/main.c
|
|
+++ b/pppd/main.c
|
|
@@ -89,6 +89,7 @@
|
|
#include <sys/socket.h>
|
|
#include <netinet/in.h>
|
|
#include <arpa/inet.h>
|
|
+#include <sys/sysinfo.h>
|
|
|
|
#include "pppd.h"
|
|
#include "magic.h"
|
|
@@ -226,6 +227,7 @@ static struct subprocess *children;
|
|
|
|
/* 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));
|
|
@@ -525,6 +527,7 @@ main(argc, argv)
|
|
info("Starting link");
|
|
}
|
|
|
|
+ check_time();
|
|
gettimeofday(&start_time, NULL);
|
|
script_unsetenv("CONNECT_TIME");
|
|
script_unsetenv("BYTES_SENT");
|
|
@@ -1270,6 +1273,36 @@ struct callout {
|
|
|
|
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.
|
|
@@ -1340,6 +1373,8 @@ calltimeout()
|
|
{
|
|
struct callout *p;
|
|
|
|
+ check_time();
|
|
+
|
|
while (callout != NULL) {
|
|
p = callout;
|
|
|
|
@@ -1367,6 +1402,8 @@ timeleft(tvp)
|
|
{
|
|
if (callout == NULL)
|
|
return NULL;
|
|
+
|
|
+ check_time();
|
|
|
|
gettimeofday(&timenow, NULL);
|
|
tvp->tv_sec = callout->c_time.tv_sec - timenow.tv_sec;
|