Baptiste Coudurier <[email protected]> added the comment:

On 07/14/2010 02:06 PM, mcfrisk wrote:
>
> mcfrisk<[email protected]>  added the comment:
>
> Are you really sure, 'cos this patch to libswscale helps?
>
> My ffmpeg is at svn 24240 and libswscale at svn 31722 plus the patch.
>
> ----------
> status: closed ->  new
> type: bug ->  patch
>
> ________________________________________________
> FFmpeg issue tracker<[email protected]>
> <https://roundup.ffmpeg.org/issue2092>
> ________________________________________________
>
>
> libswscale_fix_mlt_crash.patch
>
>
> commit 1881b95f7853240447ba4f13cb44c08cc6f095fc
> Author: Mikko Rapeli<[email protected]>
> Date:   Wed Jul 14 23:40:54 2010 +0300
>
>      fix mlt crash
>
> diff --git a/swscale.c b/swscale.c
> index 159bf9c..be350fa 100644
> --- a/swscale.c
> +++ b/swscale.c
> @@ -1337,7 +1337,10 @@ static int yuv422pToYuy2Wrapper(SwsContext *c, const 
> uint8_t* src[], int srcStri
>   {
>       uint8_t *dst=dstParam[0] + dstStride[0]*srcSliceY;
>
> -    
> yuv422ptoyuy2(src[0],src[1],src[2],dst,c->srcW,srcSliceH,srcStride[0],srcStride[1],dstStride[0]);
> +    if (src[0] != 0&&
> +        src[1] != 0&&
> +        src[2] != 0)
> +       
> yuv422ptoyuy2(src[0],src[1],src[2],dst,c->srcW,srcSliceH,srcStride[0],srcStride[1],dstStride[0]);

well if the function is really yuv422ptoyuy2 then src[0], src[1] and 
src[2] _shall_ be set there are 3 planes Y, U, V.

________________________________________________
FFmpeg issue tracker <[email protected]>
<https://roundup.ffmpeg.org/issue2092>
________________________________________________

Reply via email to