Bug #78984 [Com]: DateTimeZone accepting invalid UTC timezones
| From: | carusogabriel@php.net | Date: | Tue, 17 Dec 2019 23:12:24 +0000 |
| Subject: | Bug #78984 [Com]: DateTimeZone accepting invalid UTC timezones | ||
| References: | 1 | Groups: | php.bugs |
| Request: | Send a blank email to php-bugs+get-224372@lists.php.net to get a copy of this message | ||
Edit report at https://bugs.php.net/bug.php?id=78984&edit=1
ID: 78984
Comment by: carusogabriel@php.net
Reported by: eu+php at redrat dot com dot br
Summary: DateTimeZone accepting invalid UTC timezones
Status: Open
Type: Bug
Package: Date/time related
Operating System: Linux
PHP Version: 7.4.0
Block user comment: N
Private report: N
New Comment:
Looks like this was a feature request: #45528
Introduced via https://github.com/php/php-src/commit/d676396435.
Previous Comments:
------------------------------------------------------------------------
[2019-12-17 19:14:11] eu+php at redrat dot com dot br
Description:
------------
Today, object \DateTimeZone accepts various timezone format in your construct. One option to
instantiate this is using UTC formats like "-3" or "+5", but today UTC have only
timezone between -12 and +12 and construct is accepting any number, like in test script.
Expected result if you define invalid UTC timezone is fatal error, like when you set other unknown
or bad timezone, as described in php_date.c#L3484, but today this object is accepting any integer
value and causes mistakes in date time operations.
Test script:
---------------
<?php
var_dump(
new \DateTimeZone('-3'),
new \DateTimeZone('+3'),
new \DateTimeZone('+30157')
);
Expected result:
----------------
Fatal error: Uncaught Exception: DateTimeZone::__construct(): Unknown or bad timezone (+30157) in
%script%
Actual result:
--------------
object(DateTimeZone)#1 (2) {
["timezone_type"]=>
int(1)
["timezone"]=>
string(6) "-03:00"
}
object(DateTimeZone)#2 (2) {
["timezone_type"]=>
int(1)
["timezone"]=>
string(6) "+03:00"
}
object(DateTimeZone)#3 (2) {
["timezone_type"]=>
int(1)
["timezone"]=>
string(9) "+30157eZ"
}
------------------------------------------------------------------------
--
Edit this bug report at https://bugs.php.net/bug.php?id=78984&edit=1