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

2014-08-14 Thread 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 : > > > 2014-08-14 4:17 GMT+02:00 Michael Niedermayer : > >> causes > >> ./ffmpeg -icheckerboard_1080p_nuke_bigendian_12bit_alpha.dpx -f null - > >> to segfault > >> > >> see

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

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 4:17 GMT+02:00 Michael Niedermayer : > see http://samples.ffmpeg.org/image-samples/dpx_samples.zip Note that the date metadata in that file does not have a valid format either. -- Christophe ___ ffmpeg-devel mailing list ffmpeg-devel@ff

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

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 9:00 GMT+02:00 Christophe Gisquet : > 2014-08-14 4:17 GMT+02:00 Michael Niedermayer : >> 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: u

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

2014-08-14 Thread Christophe Gisquet
Hi, 2014-08-14 4:17 GMT+02:00 Michael Niedermayer : > causes > ./ffmpeg -icheckerboard_1080p_nuke_bigendian_12bit_alpha.dpx -f null - > to segfault > > see http://samples.ffmpeg.org/image-samples/dpx_samples.zip OK, thanks. Disclaimer, I'm not that knowledgeable in DPX, I'm just applying the anal

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

2014-08-13 Thread Michael Niedermayer
On Wed, Aug 13, 2014 at 10:21:51AM +, Christophe Gisquet wrote: > 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, whic

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

2014-08-13 Thread Christophe Gisquet
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 | 34 +