Ruiling Song (12019-05-16):
> ctx is a pointer to pointer here.
>
> Signed-off-by: Ruiling Song <ruiling.s...@intel.com>
> ---
> libavutil/tx.c | 2 +-
> 1 file changed, 1 insertion(+), 1 deletion(-)
>
> diff --git a/libavutil/tx.c b/libavutil/tx.c
> index 934ef27c81..1690604040 100644
> --- a/libavutil/tx.c
> +++ b/libavutil/tx.c
> @@ -697,7 +697,7 @@ static int gen_mdct_exptab(AVTXContext *s, int len4,
> double scale)
>
> av_cold void av_tx_uninit(AVTXContext **ctx)
> {> - if (!ctx) > + if (!ctx || !(*ctx)) That would protect somebody stupid enough to call av_tx_uninit(NULL) instead of av_tx_uninit(&var). A hard crass is completely warranted in this case. An assert would be acceptable. > return; > > av_free((*ctx)->pfatab); Regards, -- Nicolas George
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".
