Opened 8 years ago
Closed 8 years ago
#699 closed defect (fixed)
BMP: support BMP OS/2 v2 with RLE compression
Reported by: | ami_stuff | Owned by: | |
---|---|---|---|
Priority: | normal | Component: | avcodec |
Version: | git-master | Keywords: | bmp |
Cc: | Blocked By: | ||
Blocking: | Reproduced by developer: | yes | |
Analyzed by developer: | no |
Description
The attached file decodes correctly in IrfanView/Photoshop?.
Seems like header check should be modified to support this file:
libavcodec/bmp.c:
depth = bytestream_get_le16(&buf); // if(ihsize == 40) comp = bytestream_get_le32(&buf); // else // comp = BMP_RGB; if(comp != BMP_RGB && comp != BMP_BITFIELDS && comp != BMP_RLE4 && comp != BMP_RLE8){
Attachments (1)
Change History (5)
comment:1 Changed 8 years ago by ami_stuff
Changed 8 years ago by ami_stuff
comment:2 Changed 8 years ago by ami_stuff
"the major difference between v1.x and v2.x files is the size of the bitmap header. This header is 14 bytes in length in v1.x BMP files and is 64 bytes in length in v2.x files."
and this seems to work
if(ihsize == 40 || ihsize == 64)
comment:3 Changed 8 years ago by cehoyos
- Component changed from undetermined to avcodec
- Keywords bmp added
- Reproduced by developer set
- Status changed from new to open
- Version changed from unspecified to git-master
Please consider sending a patch to ffmpeg-devel.
comment:4 Changed 8 years ago by cehoyos
- Resolution set to fixed
- Status changed from open to closed
Fixed by ami_stuff and Peter Ross.
Note: See
TracTickets for help on using
tickets.