Re: [FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-12 Thread Andreas Cadhalpun
On 12.05.2015 21:12, Michael Niedermayer wrote: On Tue, May 12, 2015 at 04:02:44PM +0200, Andreas Cadhalpun wrote: On 12.05.2015 14:51, Michael Niedermayer wrote: On Tue, May 12, 2015 at 02:31:38PM +0200, Andreas Cadhalpun wrote: @@ -108,6 +109,14 @@ static int decode_packet(int *got_frame,

Re: [FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-12 Thread Michael Niedermayer
On Tue, May 12, 2015 at 04:02:44PM +0200, Andreas Cadhalpun wrote: On 12.05.2015 14:51, Michael Niedermayer wrote: On Tue, May 12, 2015 at 02:31:38PM +0200, Andreas Cadhalpun wrote: @@ -108,6 +109,14 @@ static int decode_packet(int *got_frame, int cached) (const

Re: [FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-12 Thread Michael Niedermayer
On Tue, May 12, 2015 at 09:48:45PM +0200, Andreas Cadhalpun wrote: On 12.05.2015 21:12, Michael Niedermayer wrote: On Tue, May 12, 2015 at 04:02:44PM +0200, Andreas Cadhalpun wrote: On 12.05.2015 14:51, Michael Niedermayer wrote: On Tue, May 12, 2015 at 02:31:38PM +0200, Andreas Cadhalpun

Re: [FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-12 Thread Andreas Cadhalpun
On 12.05.2015 00:28, Michael Niedermayer wrote: this breaks demuxing_decoding with pixel formats that use more than 1 plane for example: doc/examples/demuxing_decoding lena255.jpg out.raw /dev/null Could not find audio stream in input file 'lena255.jpg' Input #0, image2, from

Re: [FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-12 Thread Michael Niedermayer
On Tue, May 12, 2015 at 02:31:38PM +0200, Andreas Cadhalpun wrote: On 12.05.2015 00:28, Michael Niedermayer wrote: this breaks demuxing_decoding with pixel formats that use more than 1 plane for example: doc/examples/demuxing_decoding lena255.jpg out.raw /dev/null Could not find

Re: [FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-12 Thread Andreas Cadhalpun
On 12.05.2015 11:17, wm4 wrote: On Mon, 11 May 2015 23:34:32 +0200 Andreas Cadhalpun andreas.cadhal...@googlemail.com wrote: @@ -109,7 +108,7 @@ static int decode_packet(int *got_frame, int cached) pix_fmt, width, height); /* write to rawvideo file

Re: [FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-12 Thread Andreas Cadhalpun
On 12.05.2015 14:51, Michael Niedermayer wrote: On Tue, May 12, 2015 at 02:31:38PM +0200, Andreas Cadhalpun wrote: @@ -108,6 +109,14 @@ static int decode_packet(int *got_frame, int cached) (const uint8_t **)(frame-data), frame-linesize,

[FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-11 Thread Andreas Cadhalpun
Currently video_dst_bufsize is used, which is set to the return value of av_image_alloc, but this is the combined size of all planes. Only the first plane is written in fwrite. The size of the first plane is the product of video_dst_linesize[0] and height. This fixes 'Syscall param write(buf)

Re: [FFmpeg-devel] [PATCH] examples/demuxing_decoding: use correct size of video_dst_data[0]

2015-05-11 Thread Michael Niedermayer
On Mon, May 11, 2015 at 11:34:32PM +0200, Andreas Cadhalpun wrote: Currently video_dst_bufsize is used, which is set to the return value of av_image_alloc, but this is the combined size of all planes. Only the first plane is written in fwrite. The size of the first plane is the product of