imvalid timezones are checked in generalized time format validator

_changelog.txt

@@ -12,6 +12,7 @@
     - updated tests for OpenLDAP
     - fixed error when in add/remove extend operation for case mismatch in user or group dn
     - integer validator now automatically convert valid string numbers to int
+    - invalid timezone are checked when validating Generalized Time Format
     - added test cases for validators
 
 - 2.2.1 2017.02.12

ldap3/protocol/formatters/formatters.py

@@ -201,6 +201,9 @@ def format_time(raw_value):
                 raise ValueError
         except ValueError:
             return raw_value
+        if timezone_hour > 23 or timezone_minute > 59:  # invalid timezone
+            return raw_value
+
         if str is not bytes:  # Python 3
             timezone = OffsetTzInfo((timezone_hour * 60 + timezone_minute) * (1 if sep == b'+' else -1), 'UTC' + str(sep + offset, encoding='utf-8'))
         else:  # Python 2

test/testValidators.py

@@ -149,6 +149,14 @@ def test_validate_time_valid_str_with_timezone(self):
         validated = validate_time('20170317094232+0100')
         self.assertTrue(validated)
 
-    def test_validate_time_invalid_str(self):
+    def test_validate_time_invalid_str_1(self):
         validated = validate_time('abc')
-        self.assertFalse(validated)
+        self.assertFalse(validated)
+
+    def test_validate_time_invalid_str_2(self):
+        validated = validate_time('20170317254201Z')
+        self.assertFalse(validated)
+
+    def test_validate_time_invalid_str_with_timezone(self):
+        validated = validate_time('20170317094232+24')
+        self.assertFalse(validated)