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.gisq...@gmail.com>: > > > 2014-08-14 4:17 GMT+02:00 Michael Niedermayer <michae...@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.gisq...@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.
signature.asc
Description: Digital signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
