diff options
author | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-04-03 17:03:38 +0200 |
---|---|---|
committer | Andreas Rheinhardt <andreas.rheinhardt@gmail.com> | 2020-05-25 13:11:36 +0200 |
commit | 82bf41f3abce4a13e7c6ad1606eb708f371de87f (patch) | |
tree | a76e1dc947a8b2db0dbf821d5c8fcc5e61269a52 /libavformat/hlsproto.c | |
parent | af97c9865fe7a48b223e162eabce21cc180f305c (diff) | |
download | ffmpeg-82bf41f3abce4a13e7c6ad1606eb708f371de87f.tar.gz |
avformat: Replace ffurl_close() by ffurl_closep() where appropriate
It avoids leaving dangling pointers behind in memory.
Also remove redundant checks for whether the URLContext to be closed is
already NULL.
Reviewed-by: Anton Khirnov <anton@khirnov.net>
Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
Diffstat (limited to 'libavformat/hlsproto.c')
-rw-r--r-- | libavformat/hlsproto.c | 5 |
1 files changed, 2 insertions, 3 deletions
diff --git a/libavformat/hlsproto.c b/libavformat/hlsproto.c index e5673e5e03..de45f771d6 100644 --- a/libavformat/hlsproto.c +++ b/libavformat/hlsproto.c @@ -178,7 +178,7 @@ static int hls_close(URLContext *h) free_segment_list(s); free_variant_list(s); - ffurl_close(s->seg_hd); + ffurl_closep(&s->seg_hd); return 0; } @@ -260,8 +260,7 @@ start: return ret; } if (s->seg_hd) { - ffurl_close(s->seg_hd); - s->seg_hd = NULL; + ffurl_closep(&s->seg_hd); s->cur_seq_no++; } reload_interval = s->n_segments > 0 ? |