ID: 25581
Updated by: [EMAIL PROTECTED]
Reported By: willertan1980 at yahoo dot com
-Status: Wont fix
+Status: Open
-Bug Type: GetImageSize related
+Bug Type: Feature/Change Request
-Operating System: WinXP
+Operating System: *
PHP Version: 4.3.3
Assigned To: helly
New Comment:
Well at least you have reached i looked again and think i could do the
change. I'll look into it during weekend.
Previous Comments:
------------------------------------------------------------------------
[2003-09-22 01:41:12] willertan1980 at yahoo dot com
Sorry !! im too violence , :P
not to offend !!
JUST BAD IN MOOD
------------------------------------------------------------------------
[2003-09-22 00:41:47] willertan1980 at yahoo dot com
Bitmap was original developed by both microsoft and IBM for both
windows and OS2.
if php dont wish to make it support for both, just announce it "Only
for windows Bitmap" and not "bitmap"
------------------------------------------------------------------------
[2003-09-22 00:35:43] willertan1980 at yahoo dot com
I wish to make it clear. I did not find this bug from somewhere else.
Ok, maybe I was to blame by not making a good explanation
I just installed the latest version (php 4.3.3) as told
And the problem still exists
The problem file � Bitmap file saved by using OS2/1.x or Win 2.x
header
(You can generate this file by using Photoshop, just draw something and
save it with
Save As: bmp-> file format -> choose OS/2 instead of windows)
This time, I do a check on PHP source code (php 4.3.3)
And I found this bug at
\ext\standard\image.c (lines 154)
result->width =
(((unsigned int)dim[ 3]) << 24) + (((unsigned int)dim[ 2]) << 16) +
(((unsigned int)dim[ 1]) << 8) + ((unsigned int) dim[ 0]);
result->height =
(((unsigned int)dim[ 7]) << 24) + (((unsigned int)dim[ 6]) << 16) +
(((unsigned int)dim[ 5]) << 8) + ((unsigned int) dim[ 4]);
PHP was reading 4 bytes for each width and height, that is true if for
readings >win3.x and >os2 2.x, because it is LONG integer, but for os2
1.x and win 2.x it is SHORT int, should be read as 2 bytes SHORT
OLDER VERSION - os2 1.x and win 2.x
DWORD biSize; //
SHORT biWidth;
SHORT biHeight;
WORD biPlanes;
WORD biBitCount;
NEWER VERSION >win3.x and >os2 2.x
DWORD biSize;
LONG biWidth;
LONG biHeight;
WORD biPlanes;
WORD biBitCount;
little-endian byte order
A HEX DUMP from Bitmap file that generated by Photoshop 7.0
width = 200 px = c8
height = 200 px = c8
(draw something and save it as bitmap (os2) 24bit)
// 2 bytes, magic bytes = �BM�
42 4d
// 4 bytes FileSize; 2 bytes Reserved1; 2 bytes Reserved2; 4 bytes
BitmapOffset
dc d4 01 00 00 00 00 00 1a 00 00 00
// 4 bytes, BiLength, bmp header length = 12
0c 00 00 00
Assumed win2.x
// 2 bytes Width; 2 bytes height; 2 bytes biplanes; 2 bytes biBitCount
c8 00 c8 00 01 00 18 00 ff ff ff ff ff
it is predicable by assuming getimagesize reads 4 bytes for width and
height, by the fact getimagesize did so.
Assumed win3.x
// 4 bytes width; 4 bytes height; 2 bytes biplanes; 2 bytes biBitCount
c8 00 c8 00 01 00 18 00 ff ff ff ff ff
Width = c8 00 c8 00
Height = 01 00 18 00
Bits = ff ff
by the fact getimagesize did this mistake in \ext\standard\image.c
(lines 154)
This is how getimagesize () flush out from
print_r(getimagesize('D:\Inetpub\New PHP artbook gen 7\Test
pic\_color\bitmap_os2.bmp' ));
Array ( [0] => 13107400 [1] => 1572865 [2] => 6 [3] => width="13107400"
height="1572865" [bits] => 65535 [mime] => image/bmp )
Width = 13107400 = 00 c8 00 c8 // 4 bytes
Height = 1572865 = 00 18 00 01 // 4 bytes
Bits = 65535 = ff ff
You can observed that by convert width values to hex (little-endian) =
c8 00 c8 00
Because os2 1.x (older version of bitmap) is 2 bytes in length but PHP
fetch it by 4 bytes.
It should be fix by -> If header length equal to 12, do below
result->width = (((unsigned int)dim[ 1]) << 8) + ((unsigned int) dim[
0]);
result->height = (((unsigned int)dim[ 3]) << 8) + ((unsigned int) dim[
2]);
result->bits = (((unsigned int)dim[7]) << 8) + ((unsigned int)dim[6]);
I think, more length check should be done, if PHP wishing to extend the
getimagesize() function on bmp.file
Here are some of my suggestions;
If length ==12 then read as bitmap win2.x or os/2 1.x
If length == 40 then read as bitmap win3.x
If length == 108 then read as bitmap win95 (version 4)
If length >12 and not 40 or 108 (typically 64, varying in size/fields)
then read as bitmap IBM OS/2 2.x <- this part making me really headache
:P
In addition, this is a �very old� windows bitmap structure, win1.x
typedef struct _Win1xHeader
{
WORD Type; /* File type identifier (always 0) */
WORD Width; /* Width of the bitmap in pixels */
WORD Height; /* Height of the bitmap in scan lines */
WORD ByteWidth; /* Width of bitmap in bytes */
BYTE Planes; /* Number of color planes */
BYTE BitsPerPixel; /* Number of bits per pixel */
} WIN1XHEADER;
my system
PHP 4.3.3 (bundled version)
OS: Windows sp1 on NTFS part ion
Server : IIS 5.1
Intel P4 cpu
------------------------------------------------------------------------
[2003-09-18 15:40:10] [EMAIL PROTECTED]
You realized wrong. We are only reading the 12 bytes that are the same
in all formats. Thus the error must by elsewhere. If you find out where
feel free to provide a patch. Until then we simply do not support such
images.
------------------------------------------------------------------------
[2003-09-17 16:38:14] willertan1980 at yahoo dot com
Description:
------------
getimagesize () return incorrect values on bitmap files
SYSTEM PHP 4.3.1(bundled version gd >1.6) , WinXP IIS 5.1 server, Intel
P4 CPU
TESTED image : RGB 24bits OS/2 bitmap (BM) generated by Photoshop 7.0
>>HEX DUMP image data 1st 32bytes - bitmaps header 12 bytes
4d 2c d0 01 00 00 00 00 00 1a 00 00
00 0c 00
00 f0 00 a5 00 01 00 18 00 b0 7f 6d
a9 75 62
i realized that getimagesize () read OS/2 1.x bitmap by using Windows
3.x / OS/2 2.x structure, that is the reason the output was incorrect.
>>DUMP form getimagesize () :
Array ( [0] => 10813680 [1] => 1572865 [2] => 6 [3] => width="10813680"
height="1572865" [bits] => 43373 [mime] => image/bmp )
>>DUMP from my own function, bmp_info () (identical to values that
given by WinXP properties and photoshop 7)
Array ( [width] => 240 [height] => 165 [planes] => 1 [bits] => 24
[bi_type] => Bitmap [bi_hsz] => 12 [bi_htype] => OS/2 1.x [ncolor] =>
16777216 [xdensity] => 96 [ydensity] => 96 [density_mod] => 1
[component] => 3 [compression] => 0 )
>>Header for bmp >= Windows 2.x and OS/2 1.x
typedef struct _WinBMPFileHeader, 14 bytes
{
WORD FileType;
DWORD FileSize;
WORD Reserved1;
WORD Reserved2;
DWORD BitmapOffset;
} WINBMPFILEHEADER;
Bitmap Windows 3.x should use this format, always 40 bytes
typedef struct tagBITMAPINFOHEADER{
DWORD biSize;
LONG biWidth;
LONG biHeight;
WORD biPlanes;
WORD biBitCount;
DWORD biCompression;
DWORD biSizeImage;
LONG biXPelsPerMeter;
LONG biYPelsPerMeter;
DWORD biClrUsed;
DWORD biClrImportant;
} BITMAPINFOHEADER, *PBITMAPINFOHEADER;
but Bitmap Windows 2.x and OS/2 1.x should read like this, and its
always 12 bytes
typedef struct _Win2xBitmapHeader
{
DWORD Size;
SHORT Width;
SHORT Height;
WORD Planes;
WORD BitsPerPixel;
} WIN2XBITMAPHEADER;
REFERENCE
http://netghost.narod.ru/gff/graphics/summary/os2bmp.htm
http://netghost.narod.ru/gff/graphics/summary/micbmp.htm
Windows BITMAPINFO-
http://msdn.microsoft.com/library/en-us/gdi/bitmaps_0zn6.asp?frame=true
Windows BITMAPINFOHEADER-
http://msdn.microsoft.com/library/en-us/gdi/bitmaps_1rw2.asp?frame=true
http://www.fastgraph.com/help/bmp_header_format.html
*please remove this message if outdated or solved
------------------------------------------------------------------------
--
Edit this bug report at http://bugs.php.net/?id=25581&edit=1
