On 9/26/2017 9:14 AM, wm4 wrote:
> On Mon, 25 Sep 2017 16:44:45 -0300
> James Almer <jamr...@gmail.com> wrote:
>
>> On 9/25/2017 4:28 PM, Rostislav Pehlivanov wrote:
>>> Signed-off-by: Rostislav Pehlivanov <atomnu...@gmail.com>
>>> ---
>>> ffprobe.c | 3 +++
>>> 1 file changed, 3 insertions(+)
>>>
>>> diff --git a/ffprobe.c b/ffprobe.c
>>> index b2e8949d9f..171f856c2d 100644
>>> --- a/ffprobe.c
>>> +++ b/ffprobe.c
>>> @@ -2227,6 +2227,9 @@ static void show_frame(WriterContext *w, AVFrame
>>> *frame, AVStream *stream,
>>> if (tag)
>>> print_str(tag->key, tag->value);
>>> print_int("size", sd->size);
>>> + } else if (sd->type == AV_FRAME_DATA_GAMMA) {
>>
>> Check that sd->size >= sizeof(AVRational) as well.
>
> It's guaranteed to be that size.
Other side data types without a related struct check for size. See gop
timecode, expecting an int64_t, and display matrix, expecting 9 int32_t.
Maybe those should also be removed if that's indeed the case.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
