date_parse_from_format error when parsing 'Gis' format

[assertio-dani]

Description

While '60101' is a valid date in 'Gis' format (06:01:01), parsing it with date_parse_from_format will result in an error.

<?php
var_export(date_parse_from_format('Gis', '60101'));

Resulted in this output:

array (
  'year' => false,
  'month' => false,
  'day' => false,
  'hour' => 60,
  'minute' => 10,
  'second' => 0,
  'fraction' => 0.0,
  'warning_count' => 1,
  'warnings' => 
  array (
    5 => 'The parsed time was invalid',
  ),
  'error_count' => 1,
  'errors' => 
  array (
    4 => 'A two digit second could not be found',
  ),
  'is_localtime' => false,
)

PHP Version

PHP 8.1.1 (cli) (built: Dec 21 2021 19:35:25)

Operating System

docker image php:8.1-cli

[damianwadley]

See https://bugs.php.net/bug.php?id=79685, which I had changed to a doc bug but I'll let someone else make that decision here.

[derickr]

On 27 December 2021 07:49:06 GMT, Damian Wadley ***@***.***> wrote: See https://bugs.php.net/bug.php?id=79685, which I had changed to a doc bug but I'll let someone else make that decision here.

It should be a doc bug here too. The format parser doesn't backtrack and is greedy, so for the G it tries to read at most two characters. And then there isn't enough left for i and s.