On Tue, 26 Sep 2017 11:12:41 -0300
James Almer <jamr...@gmail.com> wrote:
> 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.
Maybe that should wait until side data merging is fully removed (which
could cause this), to avoid creating yet another pointless discussion.
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
