generic-library/ffmpeg
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

dpx: always write the image offset

Browse Source 
According to the DPX file format description found at
http://www.fileformat.info/format/dpx/egff.htm the ImageElement part of
the GenericImageHeader also contains an an offset to the real image data
beside the same member that can be found in the GenericFileHeader.

Libav keeps this member empty (=0) while some applications expects it to
be filled properly. FATE test updated accordingly.

Bug-Id: 742
Signed-off-by: Vittorio Giovara <vittorio.giovara@gmail.com>
This commit is contained in:
Zoltan Kovacs 2014-12-05 02:42:10 +00:00 committed by Vittorio Giovara
parent 250e077ee9
commit 4843a9c74f
2 changed files with 2 additions and 1 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

1
libavcodec/dpxenc.c
View File

@ -144,6 +144,7 @@ static int encode_frame(AVCodecContext *avctx, AVPacket *pkt,
buf[802] = 2; /* linear colorimetric */ buf[802] = 2; /* linear colorimetric */
buf[803] = s->bits_per_component; buf[803] = s->bits_per_component;
write16(buf + 804, s->bits_per_component == 10 ? 1 : 0); /* packing method */ write16(buf + 804, s->bits_per_component == 10 ? 1 : 0); /* packing method */
write32(buf + 808, HEADER_SIZE); /* data offset */
/* Image source information header */ /* Image source information header */
write32(buf + 1628, avctx->sample_aspect_ratio.num); write32(buf + 1628, avctx->sample_aspect_ratio.num);

2
tests/ref/lavf/dpx
View File

@ -1,3 +1,3 @@
808ea110635774252439722a48329d61 *./tests/data/images/dpx/02.dpx d2f0b4e854fda2d3b3bee84cef80593c *./tests/data/images/dpx/02.dpx
./tests/data/images/dpx/%02d.dpx CRC=0x28c7369b ./tests/data/images/dpx/%02d.dpx CRC=0x28c7369b
305792 ./tests/data/images/dpx/02.dpx 305792 ./tests/data/images/dpx/02.dpx