Comments (24)
Hmm, if there is an OPcache bug in PHP 7.3, it won't be resolved anyway. Does this issue also happen with PHP 8.0?
from imagine.
There is also to note that not all 7.3 builds seem to affected by this (see https://3v4l.org/AYCSU for example).
Note, though, that 3v4l.org runs with OPcache disabled.
from imagine.
Hi, I've created a repo that allowed me to reproduce this issue:
https://github.com/mnocon/imagine-segfault-reproducer
Please let me know if anything is unclear
You can also see that if you downgrade to imagine 1.2.x then the segmentation fault does not occur.
from imagine.
It seems an opcache bug in particular: by disabling it we don't have the segfault anymore
from imagine.
#828 seems to work just fine
from imagine.
Without more details it's rather hard to see what's going wrong...
from imagine.
Approve, have same issue
from imagine.
Confirm a problem
from imagine.
Without more details have the same issue
from imagine.
Apache error: End of script output before headers: index.php
after this:
$this->getImagine()->open($filePath)
from imagine.
I have the same issue with ver 1.3.1, php ver 7.3, using gd.
Doing local development on Windows with symfony cli server.
Server crashes when it reaches this:
$imagine - new Imagine\Gd\Imagine();
$image = $imagine->open($filePath);
I'm not very skilled at debugging, but the crash seems to occur where \Imagine\Factory\ClassFactoryInterface::createImage() creates a new \Imagine\Gd\Image.
Server does not crash with imagine/imagine ver 1.2.4.
Also, the code fragment containing Imagine\Gd\Imagine::open seems to run without error in a phpunit test.
from imagine.
In order to fix this issue, we need to know what's going wrong.
And in order to know what's going wrong, we need:
- the image that's causing the problem
- the exact version of PHP
- the exact version of the imagick PHP module
- the exact version of the imagemagick system library
To get the info for points 2., 3., and 4.:
if the problem occurs in a web application
Simply create a sample php file on the server with the following contents:
<?php phpinfo(INFO_GENERAL); phpinfo(INFO_MODULES);
By visiting the page with the browser:
- The PHP version is at the top of the page
- The imagick version is in the "imagick" section, row "imagick module version"
- The imagemagick version is in the "imagick" section, rows "Imagick compiled with ImageMagick version" and "Imagick using ImageMagick library version"
if the problem occurs in a CLI environment:
- The PHP version can be obtained by running
php -v
- The imagick and the imagemagick versions can be obtained by running
php -ri imagick
from imagine.
Crash occurs opening any one of several different jpg image files.
php version is 7.3.5
phpinfo has no "imagick" section.
$ php -ri imagick
PHP Parse error: syntax error, unexpected end of file in Command line code on line 1
``
Parse error: syntax error, unexpected end of file in Command line code on line 1
php -m confirms it contains no imagick module
from imagine.
Same problem for me. Imagine open causes segfault. Tried nginx and apache, both have problem.
Using PHP 7.3. GD library, no imagick installed.
from imagine.
Same answer: in order to fix the issue we need to be able to reproduce it
from imagine.
Same answer: in order to fix the issue we need to be able to reproduce it
@mlocati
Are you saying that the information I supplied is insufficient to reproduce the issue?
Have you successfully run (new Imagine\Gd\Imagine())->open($filepath) on an image file with a copy of php 7.3 compiled without an imagick module?
from imagine.
@ovgray the CI tests should already do that
from imagine.
@ovgray the CI tests should already do that
Have you considered the possibility that your CI tests are using a version of php 7.3 that has an imagick module?
When I look at e.g. https://github.com/php-imagine/Imagine/runs/3935559862?check_suite_focus=true, the two php 7.3 versions are labelled "gd-gimagick (Docker)" and "gd-imagick (Docker)". Does that mean the php versions have gimagick and imagick modules, respectively?
from imagine.
This seems to be a PHP bug to me. I was able to reduce the reproducer down to a single file that still causes the segmentation fault:
<?php
function y($x, $y, $z)
{
switch ($a->a()) {
default:
$s = '' . $b;
}
$c = array($b);
switch ($a->a()) {
case A::B:
$e = 0;
$f = 0;
if (!empty($d[''])) {
$e = 0;
} else {
if (!isset($d[''])) {
if (isset($d[''])) {
$d[''] = $d[''];
}
}
if (isset($d[''])) {
$e = a(0, a(0, $d['']));
}
}
case A::B:
if (!isset($d[''])) {
if (isset($d[''])) {
$d[''] = round((0 - $d['']) * 0 * 0);
}
}
if (isset($d[''])) {
if ($d[''] < 0 || $d[''] > 0) {
new Z('');
}
}
if (!isset($d[''])) {
if (isset($d[''])) {
$d[''] = $d[''];
}
}
if (isset($d[''])) {
$g[] = $d[''];
}
break;
case A::B:
if (isset($d[''])) {
$g[] = $d[''];
}
break;
case A::B:
if (!isset($d[''])) {
if (isset($d[''])) {
$d[''] = $d[''];
}
}
if (isset($d[''])) {
if ($d[''] < 0) {
new Z('');
}
$g[] = $d[''];
}
break;
}
}
echo "Success!";
The source of this weird looking function is here:
Lines 638 to 735 in 303cf35
PHP Version of the reproducer from @mnocon :
PHP 7.3.29 (cli) (built: Aug 17 2021 14:00:32) ( NTS )
Copyright (c) 1997-2018 The PHP Group
Zend Engine v3.3.29, Copyright (c) 1998-2018 Zend Technologies
with Zend OPcache v7.3.29, Copyright (c) 1999-2018, by Zend Technologies
with blackfire v1.74.1~linux-x64-non_zts73, https://blackfire.io, by Blackfire
from imagine.
Thanks for investigating!
What would be the options here? Two things come to my mind:
- Refactor this function to work around the PHP bug IF the 1.3.x series should support PHP 7.3 and if it's even possible
- Bump the minimum required version to PHP 7.4 for the 1.3.x series (including the already released 1.3.0 and 1.3.1 tags IMHO)
What do you think?
from imagine.
I guess we'd need some help from @cmb69 here...
from imagine.
It does not happen on PHP 8.0 and 8.1 (on the images that I use):
PHP 8.0.16 (cli) (built: Feb 18 2022 21:20:58) ( NTS )
Copyright (c) The PHP Group
Zend Engine v4.0.16, Copyright (c) Zend Technologies
with Zend OPcache v8.0.16, Copyright (c), by Zend Technologies
with blackfire v1.74.1~linux-x64-non_zts80, https://blackfire.io, by Blackfire
PHP 8.1.3 (cli) (built: Feb 18 2022 19:37:16) (NTS)
Copyright (c) The PHP Group
Zend Engine v4.1.3, Copyright (c) Zend Technologies
with Zend OPcache v8.1.3, Copyright (c), by Zend Technologies
with blackfire v1.74.1~linux-x64-non_zts81, https://blackfire.io, by Blackfire
I've added them to the reproducer, just in case
from imagine.
What would be the options here?
PHP 7.3 reached its end of life so you need to get your distro to patch this in their PHP 7.3 package I think. There is also to note that not all 7.3 builds seem to affected by this (see https://3v4l.org/AYCSU for example).
Refactoring this function to avoid the error should be possible, as many changes in this method I tested stopped the segmentation fault from appearing. I’m not sure if it is the responsibility of the Imagine project, @mlocati has to decide this I think. If desired, I can provide a pull request that does such a “workaround”. already happend in the meantime in #828
from imagine.
If someone else can confirm that #828 works, I'll merge it and publish a new release
from imagine.
Related Issues (20)
- Check why Imagine\Test\Imagick\DrawerTest tests randomly on Windows with PHP 7.4 HOT 1
- Deprecation messages in symfony 5.4 HOT 13
- Split getSupportedFormats into getSupportedInputFormats and getSupportedOutputFormats HOT 2
- Release a PHP 8.1 version HOT 3
- Support for CMYK ? HOT 11
- curl error: 1409442E:SSL routines:ssl3_read_bytes:tlsv1 alert protocol version HOT 9
- Unable to open remote jpg image HOT 1
- Issue reading GDImage object using \Imagine\Gd\Imagine read() method HOT 3
- Is there an inverse to getGdResource() HOT 7
- Gd\Drawer -> ellipse
- Why does Point class reject negative values...? HOT 3
- Hello, is this image library still being updated? I want him to be a replacement for intervention, because intervention is no longer updated. HOT 1
- `getImageAlphaChannel` returns incorrect value HOT 3
- Incorrect formula to calculate the linnear position. HOT 1
- Update min required PHP version and fix deprecation warnings HOT 3
- Error when file is coming from temp directory: Creating an image in "C:\Users\...\Temp\phpA6B2.tmp" format is not supported, please use one of the following formats: HOT 7
- Feature request: Add support for "setOption('jpeg:size', '800x800');"
- vulnerable libraries (jQuery) HOT 4
- Memory limit not throw new RuntimeException
Recommend Projects
-
React
A declarative, efficient, and flexible JavaScript library for building user interfaces.
-
Vue.js
🖖 Vue.js is a progressive, incrementally-adoptable JavaScript framework for building UI on the web.
-
Typescript
TypeScript is a superset of JavaScript that compiles to clean JavaScript output.
-
TensorFlow
An Open Source Machine Learning Framework for Everyone
-
Django
The Web framework for perfectionists with deadlines.
-
Laravel
A PHP framework for web artisans
-
D3
Bring data to life with SVG, Canvas and HTML. 📊📈🎉
-
Recommend Topics
-
javascript
JavaScript (JS) is a lightweight interpreted programming language with first-class functions.
-
web
Some thing interesting about web. New door for the world.
-
server
A server is a program made to process requests and deliver data to clients.
-
Machine learning
Machine learning is a way of modeling and interpreting data that allows a piece of software to respond intelligently.
-
Visualization
Some thing interesting about visualization, use data art
-
Game
Some thing interesting about game, make everyone happy.
Recommend Org
-
Facebook
We are working to build community through open source technology. NB: members must have two-factor auth.
-
Microsoft
Open source projects and samples from Microsoft.
-
Google
Google ❤️ Open Source for everyone.
-
Alibaba
Alibaba Open Source for everyone
-
D3
Data-Driven Documents codes.
-
Tencent
China tencent open source team.
from imagine.