Skip to content

Always consume meridian for correct error reporting#711

Closed
DaveRandom wants to merge 1 commit intophp:PHP-5.6from
DaveRandom:meridianErrorMessages
Closed

Always consume meridian for correct error reporting#711
DaveRandom wants to merge 1 commit intophp:PHP-5.6from
DaveRandom:meridianErrorMessages

Conversation

@DaveRandom
Copy link
Contributor

Always consume meridian data from input string to ensure the correct error is reported in the errors array.

Fix for bug #64975.

Re-submit of #668 because I screwed up my branching.

@smalyshev
Copy link
Contributor

This seems to break a unit test:
Bug #67253 (timelib_meridian_with_check out-of-bounds read) [ext/date/tests/bug67253.phpt]

Please look into it.

@DaveRandom
Copy link
Contributor Author

In retrospect, this is not the best way to deal with the underlying problem.

I'm closing this for now.

@DaveRandom DaveRandom closed this Aug 18, 2014
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants