tm2sec: clear new fields in tm

Old users of the time APIs would hand-craft
time structs without first zeroing all the
members. When this got into tmnorm(), we
would try to access the new members, and
things would go off the rails.

This makes tm2sec() clear the new fields
before passing them to the new APIs, so
that the hand-crafted structs remain
valid.

1 files changed, 9 insertions, 0 deletions

diff --git a/sys/src/libc/9sys/tm2sec.c b/sys/src/libc/9sys/tm2sec.c
index 8b3a9d967..1959e7980 100644
--- a/sys/src/libc/9sys/tm2sec.c
+++ b/sys/src/libc/9sys/tm2sec.c
@@ -7,5 +7,14 @@ tm2sec(Tm *tm)
 	Tm tt;
 
 	tt = *tm;
+	/*
+	 * The zone offset should be calculated,
+	 * but old code may not init tz member.
+	 * nil it out so we don't access junk.
+	 * while we're at it, old code probably
+	 * leaves junk in nsec.
+	 */
+	tt.nsec = 0;
+	tt.tz = nil;
 	return tmnorm(&tt);
 }