[FFmpeg-cvslog] avformat/hlsenc: fix av_bprint_finalize() usage

Limin Wang git at videolan.org
Thu Jul 2 16:32:38 EEST 2020


ffmpeg | branch: master | Limin Wang <lance.lmwang at gmail.com> | Mon Jun 22 22:04:36 2020 +0800| [fdda832603c7f027b3d0541289182ad221c1439c] | committer: Limin Wang

avformat/hlsenc: fix av_bprint_finalize() usage

Don't need to do double check by the description of the API.

Reviewed-by:   Nicolas George <george at nsup.org>
Signed-off-by: Limin Wang <lance.lmwang at gmail.com>

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=fdda832603c7f027b3d0541289182ad221c1439c
---

 libavformat/hlsenc.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
index f7a4f30c47..a34da2f515 100644
--- a/libavformat/hlsenc.c
+++ b/libavformat/hlsenc.c
@@ -398,7 +398,7 @@ static int replace_str_data_in_filename(char **s, const char *filename, char pla
         av_bprint_finalize(&buf, NULL);
         return AVERROR(ENOMEM);
     }
-    if ((ret = av_bprint_finalize(&buf, &new_filename)) < 0 || !new_filename)
+    if ((ret = av_bprint_finalize(&buf, &new_filename)) < 0)
         return ret;
     *s = new_filename;
     return found_count;
@@ -448,7 +448,7 @@ static int replace_int_data_in_filename(char **s, const char *filename, char pla
         av_bprint_finalize(&buf, NULL);
         return AVERROR(ENOMEM);
     }
-    if ((ret = av_bprint_finalize(&buf, &new_filename)) < 0 || !new_filename)
+    if ((ret = av_bprint_finalize(&buf, &new_filename)) < 0)
         return ret;
     *s = new_filename;
     return found_count;



More information about the ffmpeg-cvslog mailing list