[FFmpeg-devel] [PATCH 1/3] lavu/frame: add av_frame_check_align().

[Muhammad Faiz]

On Thu, May 18, 2017 at 5:20 PM, Nicolas George <george at nsup.org> wrote:
> Le nonidi 29 floréal, an CCXXV, Muhammad Faiz a écrit :
>> Missing linesize check.
>
> No, check again.

Oh, I'm sorry. My fault.

>
>> I tested the behaviour of av_frame_get_buffer(frame, 32) and check
>> linesize[0], here the result of some different pixel formats:
>> yuv420p: 32
>> rgb24: 96
>> rgba: 32
>>
>> So linesize constraint acutually depends on pixel format.
>
> No, check again.

I mean for rgb24 format, when alignment is 32, linesize should be
multiple of 96 (32 is not enough).

Thank's.