Zbigniew Jędrzejewski-Szmek 930e1b
From a0598047e4bfd308a25b9a516529ab49074fa527 Mon Sep 17 00:00:00 2001
Zbigniew Jędrzejewski-Szmek 930e1b
From: Lennart Poettering <lennart@poettering.net>
Zbigniew Jędrzejewski-Szmek 930e1b
Date: Mon, 11 Nov 2013 03:03:17 +0100
Zbigniew Jędrzejewski-Szmek 930e1b
Subject: [PATCH] timer: consider (usec_t) -1 an invalid timestamp
Zbigniew Jędrzejewski-Szmek 930e1b
Zbigniew Jędrzejewski-Szmek 930e1b
---
Zbigniew Jędrzejewski-Szmek 930e1b
 src/shared/time-util.c | 4 ++--
Zbigniew Jędrzejewski-Szmek 930e1b
 src/shared/time-util.h | 5 ++++-
Zbigniew Jędrzejewski-Szmek 930e1b
 2 files changed, 6 insertions(+), 3 deletions(-)
Zbigniew Jędrzejewski-Szmek 930e1b
Zbigniew Jędrzejewski-Szmek 930e1b
diff --git a/src/shared/time-util.c b/src/shared/time-util.c
Zbigniew Jędrzejewski-Szmek 930e1b
index b8a6bd7..81d4ede 100644
Zbigniew Jędrzejewski-Szmek 930e1b
--- a/src/shared/time-util.c
Zbigniew Jędrzejewski-Szmek 930e1b
+++ b/src/shared/time-util.c
Zbigniew Jędrzejewski-Szmek 930e1b
@@ -157,7 +157,7 @@ char *format_timestamp(char *buf, size_t l, usec_t t) {
Zbigniew Jędrzejewski-Szmek 930e1b
         assert(buf);
Zbigniew Jędrzejewski-Szmek 930e1b
         assert(l > 0);
Zbigniew Jędrzejewski-Szmek 930e1b
 
Zbigniew Jędrzejewski-Szmek 930e1b
-        if (t <= 0)
Zbigniew Jędrzejewski-Szmek 930e1b
+        if (t <= 0 || t == (usec_t) -1)
Zbigniew Jędrzejewski-Szmek 930e1b
                 return NULL;
Zbigniew Jędrzejewski-Szmek 930e1b
 
Zbigniew Jędrzejewski-Szmek 930e1b
         sec = (time_t) (t / USEC_PER_SEC);
Zbigniew Jędrzejewski-Szmek 930e1b
@@ -175,7 +175,7 @@ char *format_timestamp_us(char *buf, size_t l, usec_t t) {
Zbigniew Jędrzejewski-Szmek 930e1b
         assert(buf);
Zbigniew Jędrzejewski-Szmek 930e1b
         assert(l > 0);
Zbigniew Jędrzejewski-Szmek 930e1b
 
Zbigniew Jędrzejewski-Szmek 930e1b
-        if (t <= 0)
Zbigniew Jędrzejewski-Szmek 930e1b
+        if (t <= 0 || t == (usec_t) -1)
Zbigniew Jędrzejewski-Szmek 930e1b
                 return NULL;
Zbigniew Jędrzejewski-Szmek 930e1b
 
Zbigniew Jędrzejewski-Szmek 930e1b
         sec = (time_t) (t / USEC_PER_SEC);
Zbigniew Jędrzejewski-Szmek 930e1b
diff --git a/src/shared/time-util.h b/src/shared/time-util.h
Zbigniew Jędrzejewski-Szmek 930e1b
index 7660fe1..bc10d22 100644
Zbigniew Jędrzejewski-Szmek 930e1b
--- a/src/shared/time-util.h
Zbigniew Jędrzejewski-Szmek 930e1b
+++ b/src/shared/time-util.h
Zbigniew Jędrzejewski-Szmek 930e1b
@@ -64,7 +64,10 @@ dual_timestamp* dual_timestamp_get(dual_timestamp *ts);
Zbigniew Jędrzejewski-Szmek 930e1b
 dual_timestamp* dual_timestamp_from_realtime(dual_timestamp *ts, usec_t u);
Zbigniew Jędrzejewski-Szmek 930e1b
 dual_timestamp* dual_timestamp_from_monotonic(dual_timestamp *ts, usec_t u);
Zbigniew Jędrzejewski-Szmek 930e1b
 
Zbigniew Jędrzejewski-Szmek 930e1b
-#define dual_timestamp_is_set(ts) ((ts)->realtime > 0)
Zbigniew Jędrzejewski-Szmek 930e1b
+static inline bool dual_timestamp_is_set(dual_timestamp *ts) {
Zbigniew Jędrzejewski-Szmek 930e1b
+        return ((ts->realtime > 0 && ts->realtime != (usec_t) -1) ||
Zbigniew Jędrzejewski-Szmek 930e1b
+                (ts->monotonic > 0 && ts->monotonic != (usec_t) -1));
Zbigniew Jędrzejewski-Szmek 930e1b
+}
Zbigniew Jędrzejewski-Szmek 930e1b
 
Zbigniew Jędrzejewski-Szmek 930e1b
 usec_t timespec_load(const struct timespec *ts) _pure_;
Zbigniew Jędrzejewski-Szmek 930e1b
 struct timespec *timespec_store(struct timespec *ts, usec_t u);