Project

General

Profile

Actions

Bug #68484

closed

Cropmask causes php to run out of memory (for some users)

Added by Philipp Wrann almost 9 years ago. Updated over 6 years ago.

Status:
Closed
Priority:
Must have
Assignee:
-
Category:
Image Cropping
Target version:
Start date:
2015-07-23
Due date:
% Done:

100%

Estimated time:
TYPO3 Version:
7
PHP Version:
5.6
Tags:
Complexity:
Is Regression:
No
Sprint Focus:

Description

A realy weird problem i can not track down...

In one application i use TYPO3 7 (7.4-dev at the moment [89139e81a49259ec03c5ea6b0c35734f9528f5f3])
First of all - i looked at the logs and there was no change regarding crop or permissions, so i think it still exists in master. I did not checkout the current master so far because i really lack time.

The situation is as follows:
A redactional user (sys_file_reference, crop, tt_content and all permissions set in the group) creates a new file reference. The reference points to a image - a very large one. Everything works fine so far. Then the new crop feature is used and suddenly the record can not be opened anymore (after saving). The error comes from PHP (memory limit - limit is at 512M)

BUT: When i open the record with an admin user - it works.
Even if i open it with a different backend user (with same group) - it works.

Only for one user it does not - neither if i simulate the user. So it has definitly to do with permissions.

The cropmask contained negative floatingpoint numbers - it was like: {"x":56.23968627450987,"y":-0.03011764705879614,"width":910.7576470588235,"height":683.0682352941177,"rotate":0}
Or {"x":-0.24181706943997588,"y":0,"width":6843.423065157541,"height":5132.567298868156,"rotate":0}

If the very large crop is the problem i think the imageprocessing order in the backend is wrong, only the ratio should be extracted and be applied on the size that is needed for the backend.


Files

SiggisNatursaibling003.jpg (9.71 MB) SiggisNatursaibling003.jpg Philipp Wrann, 2015-07-27 10:11
Actions #1

Updated by Susanne Moog almost 9 years ago

  • Status changed from New to Needs Feedback

Hi,

can you maybe add the image that created the problem here or add some more information (how large was it, format...)? I just tried to reproduce this with an 6MB png but all went well.

Thanks,

Susi

Actions #2

Updated by Philipp Wrann almost 9 years ago

It must have been this one, but i am not completely sure (allready removed the crop and there are a few images related to that record..

Actions #3

Updated by Susanne Moog almost 9 years ago

  • Target version changed from 7.4 (Backend) to 7.5
Actions #4

Updated by Gerrit Code Review over 8 years ago

  • Status changed from Needs Feedback to Under Review

Patch set 1 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43389

Actions #5

Updated by Gerrit Code Review over 8 years ago

Patch set 2 for branch master of project Packages/TYPO3.CMS has been pushed to the review server.
It is available at http://review.typo3.org/43389

Actions #6

Updated by Thomas Maroschik over 8 years ago

  • Status changed from Under Review to Resolved
  • % Done changed from 0 to 100
Actions #7

Updated by Riccardo De Contardi over 6 years ago

  • Status changed from Resolved to Closed
Actions

Also available in: Atom PDF