On Fri, Mar 13, 2020 at 11:28:50AM +0100, Anton Khirnov wrote:
> Makes sure it is only used for logging and nothing else.
> ---
>  libavcodec/h264_ps.c | 18 +++++++++---------
>  1 file changed, 9 insertions(+), 9 deletions(-)
> 
> diff --git a/libavcodec/h264_ps.c b/libavcodec/h264_ps.c
> index 1951bb1161..4ef25aa514 100644
> --- a/libavcodec/h264_ps.c
> +++ b/libavcodec/h264_ps.c
> @@ -104,14 +104,14 @@ static void remove_sps(H264ParamSets *s, int id)
>      av_buffer_unref(&s->sps_list[id]);
>  }
>  
> -static inline int decode_hrd_parameters(GetBitContext *gb, AVCodecContext 
> *avctx,
> +static inline int decode_hrd_parameters(GetBitContext *gb, void *logctx,

this is a double sided sword
while fields of logctx cannot be used its after this possible to pass
wrong things as logctx

[...]
-- 
Michael     GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB

Many things microsoft did are stupid, but not doing something just because
microsoft did it is even more stupid. If everything ms did were stupid they
would be bankrupt already.

Attachment: signature.asc
Description: PGP signature

_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
https://ffmpeg.org/mailman/listinfo/ffmpeg-devel

To unsubscribe, visit link above, or email
ffmpeg-devel-requ...@ffmpeg.org with subject "unsubscribe".

Reply via email to