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 video_dst_linesize[0] and
> height.
> 
> This fixes 'Syscall param write(buf) points to uninitialised byte(s)'
> valgrind warnings.
> 
> Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
> ---
>  doc/examples/demuxing_decoding.c | 4 +---
>  1 file changed, 1 insertion(+), 3 deletions(-)

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 'lena255.jpg':
    Duration: 00:00:00.04, start: 0.000000, bitrate: 3124 kb/s
        Stream #0:0: Video: mjpeg, yuvj444p(pc, bt470bg/unknown/unknown), 
255x255, 25 tbr, 25 tbn, 25 tbc
    Demuxing video from file 'lena255.jpg' into 'out.raw'
    video_frame n:0 coded_n:0 pts:NOPTS
    Demuxing succeeded.
    Play the output video file with the command:
    ffplay -f rawvideo -pix_fmt yuvj444p -video_size 255x255 out.raw

but the printed command line for ffplay does not work as the stored
image is not containing teh chroma planes


[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

It is what and why we do it that matters, not just one of them.

Attachment: signature.asc
Description: Digital signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel

Reply via email to