[FFmpeg-devel] [PATCH] Fix incorrect enum type used for a local variable

[Eugene Zemtsov]

Any feedback?

On Thu, Oct 24, 2024 at 6:52 PM <ezemtsov at google.com> wrote:

> From: Eugene Zemtsov <eugene at chromium.org>
>
> It's AVPacketSideDataType, not AVFrameSideDataType.
>
> Bug: https://issues.chromium.org/issues/374797732
> Change-Id: If75702c6d639ca63827cc3370477de00544d3c0f
> Reviewed-on:
> https://chromium-review.googlesource.com/c/chromium/third_party/ffmpeg/+/5950926
> Reviewed-by
> <https://chromium-review.googlesource.com/c/chromium/third_party/ffmpeg/+/5950926Reviewed-by>:
> Ted (Chromium) Meyer <tmathmeyer at chromium.org>
> ---
>  libavcodec/decode.c | 2 +-
>  1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavcodec/decode.c b/libavcodec/decode.c
> index c331bb8596..fbb6be2971 100644
> --- a/libavcodec/decode.c
> +++ b/libavcodec/decode.c
> @@ -1466,7 +1466,7 @@ static int side_data_map(AVFrame *dst,
>
>  {
>      for (int i = 0; map[i].packet < AV_PKT_DATA_NB; i++) {
> -        const enum AVFrameSideDataType type_pkt   = map[i].packet;
> +        const enum AVPacketSideDataType type_pkt   = map[i].packet;
>          const enum AVFrameSideDataType type_frame = map[i].frame;
>          const AVPacketSideData *sd_pkt;
>          AVFrameSideData *sd_frame;
> --
> 2.47.0.163.g1226f6d8fa-goog
>
>

-- 
Thanks,
Eugene Zemtsov.