?
Referenced as Pull Request for: # 7814
avatar ryandemmer
ryandemmer
4 Sep 2015

Line 441 to 444 of libraries/joomla/filter/JInputFilter.php extracts possible extensions in the file name to check against a list of invalid extensions.

https://github.com/joomla/joomla-cms/blob/staging/libraries/joomla/filter/input.php

On line 442, array_reverse is used to re-arrange the array created so as to move the file name to the end, removing it with array_pop on line 443, but array_reverse returns the re-ordered array, it does not change the original array - http://php.net/manual/en/function.array-reverse.php

Therefore line 442 should be:

$explodedName = array_reverse($explodedName);

avatar ryandemmer ryandemmer - open - 4 Sep 2015
avatar ryandemmer
ryandemmer - comment - 4 Sep 2015

Pull request submitted - #7814

avatar zero-24
zero-24 - comment - 4 Sep 2015

Thanks @ryandemmer

FYI: AS we now use issues.joomla.org / Github as Issue management we don't require a issue if you have a PR.

If you have code (like here) you only need to open the Pull Request with all the data and testinstructions. :smile:

So i will close here and copy the Infos to the PR. Thanks.

avatar zero-24 zero-24 - change - 4 Sep 2015
The description was changed
Status New Closed
Closed_Date 0000-00-00 00:00:00 2015-09-04 09:29:44
Closed_By zero-24
avatar zero-24 zero-24 - close - 4 Sep 2015
avatar zero-24 zero-24 - close - 4 Sep 2015
avatar zero-24 zero-24 - change - 4 Sep 2015
Labels Added: ?

Add a Comment

Login with GitHub to post a comment