[FFmpeg-devel] [PATCH v1 3/4] avformat/rtpenc_mpegts: removed unused check of avformat_free_context

Michael Niedermayer michael at niedermayer.cc
Fri Nov 29 20:21:24 EET 2019


On Fri, Nov 29, 2019 at 01:15:59PM +0800, Steven Liu wrote:
> Signed-off-by: Steven Liu <lq at chinaffmpeg.org>
> ---
>  libavformat/rtpenc_mpegts.c | 3 +--
>  1 file changed, 1 insertion(+), 2 deletions(-)
> 
> diff --git a/libavformat/rtpenc_mpegts.c b/libavformat/rtpenc_mpegts.c
> index 45ba6fffe5..7d7377db7a 100644
> --- a/libavformat/rtpenc_mpegts.c
> +++ b/libavformat/rtpenc_mpegts.c
> @@ -106,8 +106,7 @@ fail:
>          av_dict_free(&mpegts_ctx->metadata);
>          avformat_free_context(mpegts_ctx);
>      }
> -    if (rtp_ctx)
> -        avformat_free_context(rtp_ctx);
> +    avformat_free_context(rtp_ctx);

this and all the other if() removials LGTM

thx

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

it is not once nor twice but times without number that the same ideas make
their appearance in the world. -- Aristotle
-------------- next part --------------
A non-text attachment was scrubbed...
Name: signature.asc
Type: application/pgp-signature
Size: 181 bytes
Desc: not available
URL: <https://ffmpeg.org/pipermail/ffmpeg-devel/attachments/20191129/39276839/attachment.sig>


More information about the ffmpeg-devel mailing list