time-util: mktime_or_timegm are changing the struct tm

after that wm_day etc. seems to be changed. Moving the check infront of
the mktime_or_timegm fixes that.

diff --git a/src/basic/time-util.c b/src/basic/time-util.c
index bcfa42714d24c9f641f155b5ac0210441839dd4d..f67467c5bde3251da232bee7a29a0e2c584185ab 100644 (file)
--- a/src/basic/time-util.c
+++ b/src/basic/time-util.c
@@ -849,11 +849,11 @@ parse_usec:
         }
 
 from_tm:
-        x = mktime_or_timegm(&tm, utc);
-        if (x < 0)
+        if (weekday >= 0 && tm.tm_wday != weekday)
                 return -EINVAL;
 
-        if (weekday >= 0 && tm.tm_wday != weekday)
+        x = mktime_or_timegm(&tm, utc);
+        if (x < 0)
                 return -EINVAL;
 
         ret = (usec_t) x * USEC_PER_SEC + x_usec;