Bug #77546 [Com]: iptcembed broken function
Edit report at https://bugs.php.net/bug.php?id=77546&edit=1
ID: 77546
Comment by: imagevuex at gmail dot com
Reported by: php at degoulet dot net
Summary: iptcembed broken function
Status: Closed
Type: Bug
Package: *Graphics related
Operating System: linux
PHP Version: 7.3.1
Assigned To: nikic
Block user comment: N
Private report: N
New Comment:
Thanks for the fix in PHP 7.3.3.
Previous Comments:
------------------------------------------------------------------------
[2019-02-08 09:40:17] nikic@php.net
This change will be part of PHP 7.3.3.
------------------------------------------------------------------------
[2019-02-08 09:32:06] php at degoulet dot net
commit f27f9022660d28c13c6209f4679768dcbd961b43 not integrated on php 7.3.2 ??
https://github.com/php/php-src/commit/f27f9022660d28c13c6209f4679768dcbd961b43
------------------------------------------------------------------------
[2019-02-08 05:06:13] imagevuex at gmail dot com
Temporary solution, check if the image stream is valid with getimagesizefromstring() before writing
to file:
$content = iptcembed($iptc, $file,0);
if($content && @getimagesizefromstring($content)) // is valid
------------------------------------------------------------------------
[2019-02-06 07:48:36] imagevuex at gmail dot com
+1
PHP 7.3 breaks our gallery app as iptcembed() is corrupting JPG images (getimagesize error). Looking
forward to see this fixed.
------------------------------------------------------------------------
[2019-01-30 16:11:23] krakjoe@php.net
Automatic comment on behalf of g.degoulet@of2m.fr
Revision: http://git.php.net/?p=php-src.git;a=commit;h=f27f9022660d28c13c6209f4679768dcbd961b43
Log: Fix #77546 iptcembed broken function
------------------------------------------------------------------------
The remainder of the comments for this report are too long. To view
the rest of the comments, please view the bug report online at
https://bugs.php.net/bug.php?id=77546
--
Edit this bug report at https://bugs.php.net/bug.php?id=77546&edit=1
Thread (7 messages)