Opened 5 years ago

Closed 5 years ago

Last modified 4 years ago

#167 closed defect (fixed)

TIFF file decodes incorrectly

Reported by: ami_stuff Owned by:
Priority: normal Component: avcodec
Version: unspecified Keywords: tif roundup
Cc: Blocked By:
Blocking: Reproduced by developer: yes
Analyzed by developer: yes

Description

Issue 1925

Author: ami_stuff 	Date: 2010-05-08.17:03:35 	

Attached TIFF file decodes incorrectly, maybe because FFmpeg thinks it is <=8bit
image, while it is really 24bit.

IrfanView can decode it correctly.

--------

C:\>ffmpeg -i 11.tiff
FFmpeg version SVN-r23056, Copyright (c) 2000-2010 the FFmpeg developers
  built on May  8 2010 06:04:21 with gcc 4.4.2
  configuration: --enable-memalign-hack --cross-prefix=i686-mingw32- --cc=ccache
-i686-mingw32-gcc --arch=i686 --target-os=mingw32 --enable-runtime-cpudetect --e
nable-avisynth --enable-gpl --enable-version3 --enable-bzlib --enable-libgsm --e
nable-libfaad --enable-pthreads --enable-libvorbis --enable-libtheora --enable-l
ibspeex --enable-libmp3lame --enable-libopenjpeg --enable-libxvid --enable-libsc
hroedinger --enable-libx264 --enable-libopencore_amrwb --enable-libopencore_amrn
b
  libavutil     50.15. 1 / 50.15. 1
  libavcodec    52.67. 0 / 52.67. 0
  libavformat   52.62. 0 / 52.62. 0
  libavdevice   52. 2. 0 / 52. 2. 0
  libavfilter    1.19. 0 /  1.19. 0
  libswscale     0.10. 0 /  0.10. 0
Input #0, image2, from '11.tiff':
  Duration: 00:00:00.04, start: 0.000000, bitrate: N/A
    Stream #0.0: Video: tiff, pal8, 512x384, 25 tbr, 25 tbn, 25 tbc
At least one output file must be specified
Author: cehoyos 	Date: 2010-05-08.17:46:12 	

Works fine with ImageMagick.
Author: kostya 	Date: 2010-05-29.07:43:14 	

This TIFF uses (8 bits per sample) x (3 samples per pixel) scheme and 
SamplesPerPixel entry is ignored now. I'll try to support it.
Author: kostya 	Date: 2010-05-29.07:59:15 	

This patch should fix it but I cannot test if it does not break decoding 
other TIFF files now so I shan't apply it.

Attachments (2)

11.tiff (576.1 KB) - added by ami_stuff 5 years ago.
tiff.patch (4.3 KB) - added by ami_stuff 5 years ago.

Download all attachments as: .zip

Change History (5)

Changed 5 years ago by ami_stuff

Changed 5 years ago by ami_stuff

comment:1 Changed 5 years ago by saste

  • Analyzed by developer set
  • Component changed from undetermined to avcodec
  • Reproduced by developer set
  • Resolution set to fixed
  • Status changed from new to closed

Applied to Libav and pulled, fixed in commit:

commit 72153419b52c96b1ed4a1bfb7e7793ddde7d71b2
Merge: 410ca3b 552a999
Author: Michael Niedermayer <michaelni@gmx.at>
Date: Fri May 13 04:40:40 2011 +0200

Merge remote branch 'qatar/master'


  • qatar/master: (33 commits) [...] tiff: add support for SamplesPerPixel? tag in tiff_decode_tag() [...]

comment:2 Changed 5 years ago by cehoyos

  • Keywords tif added

comment:3 Changed 4 years ago by cehoyos

  • Keywords roundup added
Note: See TracTickets for help on using tickets.