RawParseUtils: Default to UTC in invalid timezones

PersonIdent used to translate invalid timezones to UTC [1],
but the new java.time code just throws an exception. Also the parsing
used happen on demand, but now is done in the constructor, so the
exception is thrown even if the timezone is not used at all.

Check the parsed timezone and default to UTC if it is out of range.

[1] https://docs.oracle.com/javase/8/docs/api/java/util/TimeZone.html#getTimeZone-java.lang.String-

Change-Id: I90dd7d842ac8f44caef3b76d57375dead76bebde

1 files changed, 8 insertions, 2 deletions

diff --git a/org.eclipse.jgit/src/org/eclipse/jgit/util/RawParseUtils.java b/org.eclipse.jgit/src/org/eclipse/jgit/util/RawParseUtils.java
index 1fc695959a..3ed72516c7 100644
--- a/org.eclipse.jgit/src/org/eclipse/jgit/util/RawParseUtils.java
+++ b/org.eclipse.jgit/src/org/eclipse/jgit/util/RawParseUtils.java
@@ -32,6 +32,7 @@ import java.nio.charset.CharsetDecoder;
 import java.nio.charset.CodingErrorAction;
 import java.nio.charset.IllegalCharsetNameException;
 import java.nio.charset.UnsupportedCharsetException;
+import java.time.DateTimeException;
 import java.time.Instant;
 import java.time.ZoneId;
 import java.time.ZoneOffset;
@@ -473,12 +474,17 @@ public final class RawParseUtils {
 	 * @param ptrResult
 	 *            optional location to return the new ptr value through. If null
 	 *            the ptr value will be discarded.
-	 * @return the ZoneOffset represention of the timezone offset string
+	 * @return the ZoneOffset represention of the timezone offset string.
+	 *         Invalid offsets default to UTC.
 	 */
 	private static ZoneId parseZoneOffset(final byte[] b, int ptr,
 			MutableInteger ptrResult) {
 		int hhmm = parseBase10(b, ptr, ptrResult);
-		return ZoneOffset.ofHoursMinutes(hhmm / 100, hhmm % 100);
+		try {
+			return ZoneOffset.ofHoursMinutes(hhmm / 100, hhmm % 100);
+		} catch (DateTimeException e) {
+			return UTC;
+		}
 	}
 
 	/**