# [PATCH 1/1] Fix time offset calculation in case of unsigned time_t

1 messages from 2013-02-25 to 2013-02-25. Participants: Mike Gorchak.
Thread: https://gitlist.dev/t/33013

## Mike Gorchak, 2013-02-25 21:53

Subject: [PATCH 1/1] Fix time offset calculation in case of unsigned time_t
Message-ID: <CAHXAxrO3nSVmALnefXWbvjR-=dOgVHDDVhyhJCeu9fsco=vjOw@mail.gmail.com>
URL: https://gitlist.dev/e/CAHXAxrO3nSVmALnefXWbvjR-%3DdOgVHDDVhyhJCeu9fsco%3DvjOw%40mail.gmail.com

```
Fix time offset calculation expression in case if time_t
is unsigned. This code works fine for signed and
unsigned time_t.

Signed-off-by: Mike Gorchak <mike.gorchak.qnx@gmail.com>
---
 date.c | 11 +++++++++--
 1 file changed, 9 insertions(+), 2 deletions(-)

diff --git a/date.c b/date.c
index 1ac28e5..11ee7b4 100644
--- a/date.c
+++ b/date.c
@@ -648,6 +648,7 @@ int parse_date_basic(const char *date, unsigned
long *timestamp, int *offset)
 	struct tm tm;
 	int tm_gmt;
 	unsigned long dummy_timestamp;
+	time_t temp_time;
 	int dummy_offset;

 	if (!timestamp)
@@ -694,8 +695,14 @@ int parse_date_basic(const char *date, unsigned
long *timestamp, int *offset)

 	/* mktime uses local timezone */
 	*timestamp = tm_to_time_t(&tm);
-	if (*offset == -1)
-		*offset = ((time_t)*timestamp - mktime(&tm)) / 60;
+	if (*offset == -1) {
+		temp_time = mktime(&tm);
+		if ((time_t)*timestamp > temp_time) {
+			*offset = ((time_t)*timestamp - temp_time) / 60;
+		} else {
+			*offset = -(int)((temp_time - (time_t)*timestamp) / 60);
+		}
+	}

 	if (*timestamp == -1)
 		return -1;
-- 
1.8.2-rc0

```
