virtualenv/FFmpeg
1
0
Fork 0
mirror of https://github.com/FFmpeg/FFmpeg.git synced 2024-11-26 19:01:44 +02:00
Code Issues Releases Activity

avformat/webm_chunk: Use API functions for child muxer

Browse Source 
instead of calling the write_header/packet/trailer functions directly
via the function pointers. Also, use distinct AVStreams for the child
AVFormatContext (up until now the two AVFormatContexts shared their
AVStreams because allocating their own was deemed too onerous).

Using the function pointers directly meant that the Matroska muxer's
init-function was never called, because init-functions were only
introduced a few months after webm_chunk has been added and no one
thought of/bothered to adapt webm_chunk for this (when the init-function
was added in b287d7ea, the code setting the timebase was moved to it,
so that the timebases were no longer set to ms-precision when using
the webm_chunk muxer; this has been fixed after some time in 42a635dd
by setting the timebases direcly (instead of calling the init-function)).

And when 982a98a0 added a deinit-function for the Matroska muxer, it
introduced memleaks in webm_chunk, because the child muxer's internal
structures were no longer freed when calling write_trailer directly.
(Given that the init function has never ever been called, the child
muxer has never ever been properly initialized, so that the
deinit-function was not called when freeing the child context.)

This commit stops calling the function pointers directly and instead
uses the standard API functions for muxers. This fixes the above
mentioned memleaks. (Memleaks are still possible on error. This will be
fixed in a future commit that adds a deinit-function to webm_chunk
itself.)

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com>
This commit is contained in:
Andreas Rheinhardt 2020-02-29 22:47:26 +01:00
parent a9004ffe0f
commit e70c2d02fc
1 changed files with 39 additions and 20 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

59
libavformat/webm_chunk.c
View File

@ -61,6 +61,8 @@ static int chunk_mux_init(AVFormatContext *s)
{
WebMChunkContext *wc = s->priv_data;
AVFormatContext *oc;
AVStream *st, *ost = s->streams[0];
AVDictionary *dict = NULL;
int ret;
ret = avformat_alloc_output_context2(&wc->avf, wc->oformat, NULL, NULL);
@ -72,17 +74,42 @@ static int chunk_mux_init(AVFormatContext *s)
oc->max_delay = s->max_delay;
oc->flags = s->flags;
oc->strict_std_compliance = s->strict_std_compliance;
oc->avoid_negative_ts = s->avoid_negative_ts;
av_dict_copy(&oc->metadata, s->metadata, 0);
*(const AVClass**)oc->priv_data = oc->oformat->priv_class;
av_opt_set_defaults(oc->priv_data);
av_opt_set_int(oc->priv_data, "dash", 1, 0);
av_opt_set_int(oc->priv_data, "cluster_time_limit", wc->chunk_duration, 0);
av_opt_set_int(oc->priv_data, "live", 1, 0);
if (!(st = avformat_new_stream(oc, NULL)))
return AVERROR(ENOMEM);
oc->streams = s->streams;
oc->nb_streams = s->nb_streams;
if ((ret = avcodec_parameters_copy(st->codecpar, ost->codecpar)) < 0 ||
(ret = av_dict_copy(&st->metadata, ost->metadata, 0)) < 0)
return ret;
st->sample_aspect_ratio = ost->sample_aspect_ratio;
st->disposition = ost->disposition;
avpriv_set_pts_info(st, ost->pts_wrap_bits, ost->time_base.num,
ost->time_base.den);
av_dict_set_int(&dict, "dash", 1, 0);
av_dict_set_int(&dict, "cluster_time_limit", wc->chunk_duration, 0);
av_dict_set_int(&dict, "live", 1, 0);
ret = avformat_init_output(oc, &dict);
av_dict_free(&dict);
if (ret < 0)
return ret;
// Copy the timing info back to the original stream
// so that the timestamps of the packets are directly usable
avpriv_set_pts_info(ost, st->pts_wrap_bits, st->time_base.num,
st->time_base.den);
// This ensures that the timestamps will already be properly shifted
// when the packets arrive here, so we don't need to shift again.
s->avoid_negative_ts = oc->avoid_negative_ts;
s->internal->avoid_negative_ts_use_pts =
oc->internal->avoid_negative_ts_use_pts;
oc->avoid_negative_ts = 0;
return 0;
}
@ -119,7 +146,6 @@ static int webm_chunk_write_header(AVFormatContext *s)
WebMChunkContext *wc = s->priv_data;
AVFormatContext *oc = NULL;
int ret;
int i;
AVDictionary *options = NULL;
char oc_filename[MAX_FILENAME_SIZE];
char *oc_url;
@ -152,14 +178,10 @@ static int webm_chunk_write_header(AVFormatContext *s)
return ret;
oc->pb->seekable = 0;
ret = oc->oformat->write_header(oc);
ret = avformat_write_header(oc, NULL);
ff_format_io_close(s, &oc->pb);
if (ret < 0)
return ret;
for (i = 0; i < s->nb_streams; i++) {
// ms precision is the de-facto standard timescale for mkv files.
avpriv_set_pts_info(s->streams[i], 64, 1, 1000);
}
return 0;
}
@ -192,7 +214,7 @@ static int chunk_end(AVFormatContext *s, int flush)
if (flush)
// Flush the cluster in WebM muxer.
oc->oformat->write_packet(oc, NULL);
av_write_frame(oc, NULL);
buffer_size = avio_close_dyn_buf(oc->pb, &buffer);
oc->pb = NULL;
ret = get_chunk_filename(s, 0, filename);
@ -239,9 +261,8 @@ static int webm_chunk_write_packet(AVFormatContext *s, AVPacket *pkt)
}
}
ret = oc->oformat->write_packet(oc, pkt);
return ret;
// We only have one stream, so use the non-interleaving av_write_frame.
return av_write_frame(oc, pkt);
}
static int webm_chunk_write_trailer(AVFormatContext *s)
@ -255,11 +276,9 @@ static int webm_chunk_write_trailer(AVFormatContext *s)
if (ret < 0)
goto fail;
}
oc->oformat->write_trailer(oc);
av_write_trailer(oc);
ret = chunk_end(s, 0);
fail:
oc->streams = NULL;
oc->nb_streams = 0;
avformat_free_context(oc);
return ret;
}