php.net |  support |  documentation |  report a bug |  advanced search |  search howto |  statistics |  random bug |  login

go to bug id or search bugs for

Request #66516 getimagesize() should cater to orientation flag
Submitted: 2014-01-19 16:01 UTC Modified: 2025-01-27 13:23 UTC
Votes:3
Avg. Score:3.7 ± 0.9
Reproduced:2 of 2 (100.0%)
Same Version:1 (50.0%)
Same OS:0 (0.0%)
From: milpas999 at gmail dot com Assigned: cmb (profile)
Status: Wont fix Package: GetImageSize related
PHP Version: Irrelevant OS: windows 8.1
Private report: No CVE-ID: None
View Developer Edit
Welcome back! If you're the original bug submitter, here's where you can edit the bug or add additional notes.
If you forgot your password, you can retrieve your password here.
Password:
Status:
Package:
Bug Type:
Summary:
From: milpas999 at gmail dot com
New email:
PHP Version: OS:

 

 [2014-01-19 16:01 UTC] milpas999 at gmail dot com 
Description:
------------
My objective is to upload an image and then to create a new image with some other 'width X height' resizing parameters. Original image has uploaded successfully, but when i am using getimagesize('local path of original uploaded image') it is returning wrong values.

And this function is not working only for images of dimension '1836 X 3264' 

original image dimension : 1836 X 3264
and when using getimagesize() for the image it returns

Array
(
    [0] => 3264
    [1] => 1836
    [2] => 2
    [3] => width="3264" height="1836"
    [bits] => 8
    [channels] => 3
    [mime] => image/jpeg
)

Same as  Bug #29424. but i am unsing PHP 5.4.3 and bug reporter was using PHP 5.0.0

Test script:
---------------
$source_image_path = '../sysimages/origimage/20131109_151445.jpg';

$arrrrr = getimagesize($source_image_path);

print("<pre>arrrrr :: ");
print_r($arrrrr);
print("</pre>");

Expected result:
----------------
Array
(
    [0] => 1836
    [1] => 3264
    [2] => 2
    [3] => width="1836" height="3264"
    [bits] => 8
    [channels] => 3
    [mime] => image/jpeg
)

Actual result:
--------------
Array
(
    [0] => 3264
    [1] => 1836
    [2] => 2
    [3] => width="3264" height="1836"
    [bits] => 8
    [channels] => 3
    [mime] => image/jpeg
)

Patches

Pull Requests

History

AllCommentsChangesGit/SVN commitsRelated reports
 [2014-01-19 16:11 UTC] pajoye@php.net
-Status: Open +Status: Feedback
 [2014-01-19 16:11 UTC] pajoye@php.net 
Please provide a link to the image you use.

There is certainly a rotation/orientation meta data which makes some application automatically rotate the image on load.
 [2014-12-30 10:42 UTC] php-bugs at lists dot php dot net 
No feedback was provided. The bug is being suspended because
we assume that you are no longer experiencing the problem.
If this is not the case and you are able to provide the
information that was requested earlier, please do so and
change the status of the bug back to "Re-Opened". Thank you.
 [2015-09-12 11:23 UTC] nicolas33thomas at gmail dot com 
I have the same issue with PHP 5.4.12.

It looks to mainly happen with pictures taken with a smartphone.


Here is the picture: https://drive.google.com/file/d/0B2PDHMmm8JwMbkVESkU5STZtVjA/view?usp=sharing

Note that this picture has been resampled.

Here is the original one:
https://drive.google.com/file/d/0B2PDHMmm8JwMYTRhZ0V4ZlA2MmM/view?usp=sharing

I tried with it, and I get inverted width and height value too.
 [2015-09-12 13:50 UTC] requinix@php.net
-Status: No Feedback +Status: Re-Opened -Type: Bug +Type: Feature/Change Request
 [2015-09-12 13:50 UTC] requinix@php.net 
> Here is the original one:
> https://drive.google.com/file/d/0B2PDHMmm8JwMYTRhZ0V4ZlA2MmM/view?usp=sharing
What @pajoye said is correct: the image itself is sideways but the camera added EXIF data that specifies the "correct" orientation. GIMP, for one, presents a dialog to that effect when you try to open the file.
Since getimagesize() is behaving the way it was intended to, I'm changing this to a feature request.

If it returned the re-oriented dimensions then that could cause problems if code relies on them while trying to process the image. Maybe it could continue returning the true dimensions but also include a new value in the array indicating the orientation?

However the exif extension is also available to handle this kind of problem. See some of the user comments in the exif_read_data() doc for examples:
http://php.net/exif-read-data#usernotes
 [2015-09-12 15:39 UTC] nicolas33thomas at gmail dot com 
Ok, I understand now.
It makes sense.

So if my picture has exif data, I will rely on them, otherwise, I will take the getimagesize result as it is.

Thank you for your quick reply.
 [2016-10-07 22:38 UTC] cmb@php.net
-Summary: getimagesize() function is returning swaped values of height and width. +Summary: getimagesize() should cater to orientation flag
 [2025-01-27 13:23 UTC] cmb@php.net 
I'm closing this as WONTFIX for pragmatic reasons.

First, getimagesize() is part of ext/standard, and as such
cannot (directly) rely on code of ext/exif (which already
implements reading of the Exif Orientation flag).  While
some restructuring might generally make sense, this is not
easily done, and would require the RFC process[1] anyway.

Second, libgd (the underlying library of ext/gd) does not support
the Orientation flag[2] (and is unlikely to support it in the
"near" future).  If getimagesize() would apply the Orientation
flag, the results would be inconsistent with ext/gd, what might
even be more confusing as it is now.

Third, this feature request is open for almost ten years without
further comments.

[1] <https://wiki.php.net/rfc>
[2] <https://github.com/libgd/libgd/issues/341>
 [2025-01-27 13:23 UTC] cmb@php.net
-Status: Re-Opened +Status: Wont fix -Assigned To: +Assigned To: cmb
 
PHP Copyright © 2001-2025 The PHP Group
All rights reserved. 		Last updated: Mon Oct 27 11:00:01 2025 UTC