CID603224: missing break in parsedate.
Could lead to wrongly setting the TYPE_MINUTE flag for an invalid (>59) number of minutes. Harmless, as that flag is never used. For completeness, also set the flag for seconds (also never used). Fixes #11552.
This commit is contained in:
parent
82e1194224
commit
57f933d348
|
@ -1014,11 +1014,16 @@ parsedate_etc(const char* dateString, time_t now, int* _flags)
|
||||||
dateMask.Set(TYPE_HOUR);
|
dateMask.Set(TYPE_HOUR);
|
||||||
break;
|
break;
|
||||||
case 'M':
|
case 'M':
|
||||||
|
if (element->value > 59)
|
||||||
|
goto next_format;
|
||||||
|
|
||||||
dateMask.Set(TYPE_MINUTE);
|
dateMask.Set(TYPE_MINUTE);
|
||||||
|
break;
|
||||||
case 'S':
|
case 'S':
|
||||||
if (element->value > 59)
|
if (element->value > 59)
|
||||||
goto next_format;
|
goto next_format;
|
||||||
|
|
||||||
|
dateMask.Set(TYPE_SECOND);
|
||||||
break;
|
break;
|
||||||
case 'y':
|
case 'y':
|
||||||
case 'Y':
|
case 'Y':
|
||||||
|
|
Loading…
Reference in New Issue