Prevent timetz2tm() from scribbling on its input in HAVE_INT64_TIMESTAMP case.
authorTom Lane
Thu, 13 Feb 2003 17:04:19 +0000 (17:04 +0000)
committerTom Lane
Thu, 13 Feb 2003 17:04:19 +0000 (17:04 +0000)
src/backend/utils/adt/date.c

index 09d0e9aa25603062d5e210c3b4e06cb1147203e6..a31cce92073a6425a08ff6a40d778cee221f385c 100644 (file)
@@ -8,7 +8,7 @@
  *
  *
  * IDENTIFICATION
- *   $Header: /cvsroot/pgsql/src/backend/utils/adt/date.c,v 1.78 2003/01/31 01:08:08 tgl Exp $
+ *   $Header: /cvsroot/pgsql/src/backend/utils/adt/date.c,v 1.79 2003/02/13 17:04:19 tgl Exp $
  *
  *-------------------------------------------------------------------------
  */
@@ -1349,23 +1349,22 @@ timetz_out(PG_FUNCTION_ARGS)
 
 /* timetz2tm()
  * Convert TIME WITH TIME ZONE data type to POSIX time structure.
- * For dates within the system-supported time_t range, convert to the
- * local time zone. If out of this range, leave as GMT. - tgl 97/05/27
  */
 static int
 timetz2tm(TimeTzADT *time, struct tm * tm, fsec_t *fsec, int *tzp)
 {
 #ifdef HAVE_INT64_TIMESTAMP
-   tm->tm_hour = (time->time / INT64CONST(3600000000));
-   time->time -= (tm->tm_hour * INT64CONST(3600000000));
-   tm->tm_min = (time->time / INT64CONST(60000000));
-   time->time -= (tm->tm_min * INT64CONST(60000000));
-   tm->tm_sec = (time->time / INT64CONST(1000000));
-   *fsec = (time->time - (tm->tm_sec * INT64CONST(1000000)));
+   int64       trem = time->time;
+
+   tm->tm_hour = (trem / INT64CONST(3600000000));
+   trem -= (tm->tm_hour * INT64CONST(3600000000));
+   tm->tm_min = (trem / INT64CONST(60000000));
+   trem -= (tm->tm_min * INT64CONST(60000000));
+   tm->tm_sec = (trem / INT64CONST(1000000));
+   *fsec = (trem - (tm->tm_sec * INT64CONST(1000000)));
 #else
-   double      trem;
+   double      trem = time->time;
 
-   trem = time->time;
    TMODULO(trem, tm->tm_hour, 3600e0);
    TMODULO(trem, tm->tm_min, 60e0);
    TMODULO(trem, tm->tm_sec, 1e0);