[FFmpeg-devel] [PATCH 1/4] dpx: use aligned line starts

[Michael Niedermayer]

On Thu, Aug 14, 2014 at 09:24:51AM +0200, Christophe Gisquet wrote:
> Hi,
> 
> 2014-08-14 9:00 GMT+02:00 Christophe Gisquet <christophe.gisquet at gmail.com>:
> 
> > 2014-08-14 4:17 GMT+02:00 Michael Niedermayer <michaelni at gmx.at>:
> >> causes
> >> ./ffmpeg -icheckerboard_1080p_nuke_bigendian_12bit_alpha.dpx -f null -
> >> to segfault
> >>
> >> see http://samples.ffmpeg.org/image-samples/dpx_samples.zip
> 
> Forgot to add this last bit: updated patch to not crash and produce
> the same output for the above samples. I haven't checked yet if they
> actually look fine.
> 
> -- 
> Christophe

>  dpx.c |   42 ++++++++++++++++++++++++++++++------------
>  1 file changed, 30 insertions(+), 12 deletions(-)
> 0977b4f9f88e7dc81c872e513571ec34f56156fd  0001-dpx-use-aligned-line-starts.patch
> From ea8003852db0e6f64ec50f2ca644c9b51247cb5f Mon Sep 17 00:00:00 2001
> From: Christophe Gisquet <christophe.gisquet at gmail.com>
> Date: Wed, 13 Aug 2014 00:37:00 +0200
> Subject: [PATCH 1/2] dpx: use aligned line starts
> 
> SMPTE 268M-2003 specifies that each line starts at a 4-bytes boundary.
> Therefore, modify correspondingly the input buffer strides and size.
> 
> Partially fixes ticket #3692: DLAD_8b_3c_big.dpx still has inverted
> colors, which might be related to endianness.
> ---
>  libavcodec/dpx.c | 42 ++++++++++++++++++++++++++++++------------
>  1 file changed, 30 insertions(+), 12 deletions(-)

applied

thanks

[...]

-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is what and why we do it that matters, not just one of them.
-------------- next part --------------
A non-text attachment was scrubbed...
Name: not available
Type: application/pgp-signature
Size: 181 bytes
Desc: Digital signature
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20140814/0bcb8531/attachment.asc>