Revise implementation of jerry_port_get_local_time_zone_adjustment on Win32

Handle is_utc and not is_utc properly.
Handle the limitation of Win32 date API properly.

JerryScript-DCO-1.0-Signed-off-by: Yonggang Luo [email protected]

Author: lygstate

jerry-port/default/default-date.c

@@ -29,9 +29,21 @@
 #include "jerryscript-port-default.h"
 
 #ifdef _WIN32
-static const LONGLONG UnixEpochInTicks = 116444736000000000; /* difference between 1970 and 1601 */
+static const LONGLONG UnixEpochInTicks = 116444736000000000LL; /* difference between 1970 and 1601 */
 static const LONGLONG TicksPerMs = 10000; /* 1 tick is 100 nanoseconds */
 
+/**
+ * If you take the limit of SYSTEMTIME (last millisecond in 30827) then you end up with
+ * a FILETIME of 0x7fff35f4f06c58f0 by using SystemTimeToFileTime(). However, if you put
+ * 0x7fffffffffffffff into FileTimeToSystemTime() then you will end up in the year 30828,
+ * although this date is invalid for SYSTEMTIME. Any larger value (0x8000000000000000 and above)
+ * causes FileTimeToSystemTime() to fail.
+ * https://docs.microsoft.com/en-us/windows/win32/api/minwinbase/ns-minwinbase-systemtime
+ * https://docs.microsoft.com/en-us/windows/win32/api/minwinbase/ns-minwinbase-filetime
+ */
+static const long long UnixEpochOfDate_1601_01_02 = -11644387200000LL; /* unit: ms */
+static const long long UnixEpochOfDate_30827_12_29 = 9106702560000000LL; /* unit: ms */
+
 /* https://support.microsoft.com/en-us/help/167296/how-to-convert-a-unix-time-t-to-a-win32-filetime-or-systemtime */
 static void UnixTimeMsToFileTime (double t, LPFILETIME pft)
 {
@@ -57,30 +69,59 @@ static double FileTimeToUnixTimeMs (FILETIME ft)
  *         available. Otherwise, returns 0, assuming UTC time.
  */
 double jerry_port_get_local_time_zone_adjustment (double unix_ms,  /**< ms since unix epoch */
-                                                  bool is_utc)  /**< is the time above in UTC? */
+                                                  bool is_utc)  /**< is the time above in UTC, */
 {
 #ifdef _WIN32
-  FILETIME fileTime, localFileTime;
-  SYSTEMTIME systemTime, localSystemTime;
-  ULARGE_INTEGER time, localTime;
+  FILETIME utcFileTime, localFileTime;
+  SYSTEMTIME utcSystemTime, localSystemTime;
+  bool timeConverted = false;
+
+  /**
+   * If time is earlier than date 1601-01-02, then always using year 1601-01-02 to
+   * query time zone adjustment, this date(1601-01-02) will make sure both utc/local
+   * time both succeed with Win32 API, using date(1601-01-01) may leading win32 api
+   * failure, as after convert between local time/utc time, the date may earlier than
+   * 1601-01-01 in utc time, that's exceed the FILETIME representation range.
+   */
+  if (unix_ms < UnixEpochOfDate_1601_01_02)
+  {
+    unix_ms = UnixEpochOfDate_1601_01_02;
+  }
 
-  UnixTimeMsToFileTime (unix_ms, &fileTime);
-  /* If time is earlier than year 1601, then always using year 1601 to query time zone adjustment */
-  if (fileTime.dwHighDateTime >= 0x80000000)
+  /* Same reason as upper, do not use the latest support day */
+  if (unix_ms > UnixEpochOfDate_30827_12_29)
   {
-    fileTime.dwHighDateTime = 0;
-    fileTime.dwLowDateTime = 0;
+    unix_ms = UnixEpochOfDate_30827_12_29;
   }
 
-  if (FileTimeToSystemTime (&fileTime, &systemTime)
-      && SystemTimeToTzSpecificLocalTime (0, &systemTime, &localSystemTime)
-      && SystemTimeToFileTime (&localSystemTime, &localFileTime))
+  if (is_utc)
+  {
+    UnixTimeMsToFileTime (unix_ms, &utcFileTime);
+    if (FileTimeToSystemTime (&utcFileTime, &utcSystemTime)
+        && SystemTimeToTzSpecificLocalTime (0, &utcSystemTime, &localSystemTime)
+        && SystemTimeToFileTime (&localSystemTime, &localFileTime))
+    {
+      timeConverted = true;
+    }
+  }
+  else
+  {
+    UnixTimeMsToFileTime (unix_ms, &localFileTime);
+    if (FileTimeToSystemTime (&localFileTime, &localSystemTime)
+        && TzSpecificLocalTimeToSystemTime (0, &localSystemTime, &utcSystemTime)
+        && SystemTimeToFileTime (&utcSystemTime, &utcFileTime))
+    {
+      timeConverted = true;
+    }
+  }
+  if (timeConverted)
   {
-    time.LowPart = fileTime.dwLowDateTime;
-    time.HighPart = fileTime.dwHighDateTime;
+    ULARGE_INTEGER utcTime, localTime;
+    utcTime.LowPart = utcFileTime.dwLowDateTime;
+    utcTime.HighPart = utcFileTime.dwHighDateTime;
     localTime.LowPart = localFileTime.dwLowDateTime;
     localTime.HighPart = localFileTime.dwHighDateTime;
-    return (double) (((LONGLONG) localTime.QuadPart - (LONGLONG) time.QuadPart) / TicksPerMs);
+    return (double) (((LONGLONG) localTime.QuadPart - (LONGLONG) utcTime.QuadPart) / TicksPerMs);
   }
   return 0.0;
 #elif defined (__GNUC__) || defined (__clang__)

tests/jerry/date-getters.js

@@ -97,6 +97,7 @@ assert (isNaN (d.getTimezoneOffset()));
 /* 5. test case */
 assert (new Date(2013, -1).getMonth() === 11);
 assert (new Date(-2, -2).getFullYear() === -3);
+assert (new Date(30888, 2).getFullYear() === 30888);
 assert (new Date(-1, -1).getFullYear() === -2);
 assert (new Date(-1, -1, -1).getMonth() === 10);
 assert (new Date(-1, -1, -1, -1).getDate() === 28);