[FFmpeg-devel] [PATCH] lavu/imgutils: add consistency check to av_image_copy_plane()

[Stefano Sabatini]

On date Monday 2012-10-29 15:45:18 +0100, Stefano Sabatini encoded:
> On date Saturday 2012-09-08 02:15:17 +0200, Michael Niedermayer encoded:
> > On Sat, Sep 08, 2012 at 12:35:50AM +0200, Stefano Sabatini wrote:
> > > Add assertions and abort in case of invalid |dst_linesize| < bytewidth or
> > > |src_linesize| < bytewidth.
> > > 
> > > Avoid to silently corrupt memory.
> > > 
> > > TODO: bump micro
> > 
> > LGTM
> > 
> > thanks
> 
> Updated, the patch was raising an abort in case src or dst were
> NULL. Will push it in a few days if I read no comments.
> -- 
> FFmpeg = Fostering and Fiendish Monstrous Ponderous Educated Gigant

> From c64896ae7e5ca7ddecf5529169f64575698fc535 Mon Sep 17 00:00:00 2001
> From: Stefano Sabatini <stefasab at gmail.com>
> Date: Sat, 8 Sep 2012 00:31:41 +0200
> Subject: [PATCH] lavu/imgutils: add consistency checks to
>  av_image_copy_plane()
> 
> Add assertions and abort in case of invalid |dst_linesize| < bytewidth or
> |src_linesize| < bytewidth.
> 
> Avoid to silently corrupt memory.
> 
> TODO: bump micro
> ---
>  libavutil/imgutils.c |    3 +++
>  libavutil/imgutils.h |    3 +++
>  2 files changed, 6 insertions(+), 0 deletions(-)

Pushed.
-- 
FFmpeg = Fascinating & Fundamentalist Mythic Prodigious Elaborated Glue