aboutsummaryrefslogtreecommitdiffstats
path: root/libavformat/hlsproto.c
diff options
context:
space:
mode:
authorAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-04-03 17:03:38 +0200
committerAndreas Rheinhardt <andreas.rheinhardt@gmail.com>2020-05-25 13:11:36 +0200
commit82bf41f3abce4a13e7c6ad1606eb708f371de87f (patch)
treea76e1dc947a8b2db0dbf821d5c8fcc5e61269a52 /libavformat/hlsproto.c
parentaf97c9865fe7a48b223e162eabce21cc180f305c (diff)
downloadffmpeg-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.c5
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 ?