[FFmpeg-devel] [PATCH] avformat/hlsenc: fix hls_ts_options with mpegts
Steven Liu
lingjiujianke at gmail.com
Fri Jan 31 14:10:00 EET 2020
Marton Balint <cus at passwd.hu> 于2020年1月31日周五 上午4:40写道:
>
>
>
> On Wed, 22 Jan 2020, Marton Balint wrote:
>
> > Was broken since cdbf8847ea97a985dfd55432e1384bb7fe5d2d3b.
>
> Ping, will apply soon.
>
> Thanks,
> Marton
>
> >
> > Signed-off-by: Marton Balint <cus at passwd.hu>
> > ---
> > libavformat/hlsenc.c | 36 +++++++++++++++++-------------------
> > 1 file changed, 17 insertions(+), 19 deletions(-)
> >
> > diff --git a/libavformat/hlsenc.c b/libavformat/hlsenc.c
> > index 2b3d3742d9..87b861d437 100644
> > --- a/libavformat/hlsenc.c
> > +++ b/libavformat/hlsenc.c
> > @@ -764,6 +764,7 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs)
> > AVFormatContext *oc;
> > AVFormatContext *vtt_oc = NULL;
> > int byterange_mode = (hls->flags & HLS_SINGLE_FILE) || (hls->max_seg_size > 0);
> > + int remaining_options;
> > int i, ret;
> >
> > ret = avformat_alloc_output_context2(&vs->avf, vs->oformat, NULL, NULL);
> > @@ -852,21 +853,25 @@ static int hls_mux_init(AVFormatContext *s, VariantStream *vs)
> > return ret;
> > }
> >
> > + av_dict_copy(&options, hls->format_options, 0);
> > if (hls->segment_type == SEGMENT_TYPE_FMP4) {
> > - int remaining_options;
> > -
> > - av_dict_copy(&options, hls->format_options, 0);
> > av_dict_set(&options, "fflags", "-autobsf", 0);
> > av_dict_set(&options, "movflags", "+frag_custom+dash+delay_moov", AV_DICT_APPEND);
> > - ret = avformat_init_output(oc, &options);
> > - remaining_options = av_dict_count(options);
> > - av_dict_free(&options);
> > - if (ret < 0)
> > - return ret;
> > - if (remaining_options) {
> > - av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n");
> > - return AVERROR(EINVAL);
> > - }
> > + } else {
> > + /* We only require one PAT/PMT per segment. */
> > + char period[21];
> > + snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1);
> > + av_dict_set(&options, "sdt_period", period, 0);
> > + av_dict_set(&options, "pat_period", period, 0);
> > + }
> > + ret = avformat_init_output(oc, &options);
> > + remaining_options = av_dict_count(options);
> > + av_dict_free(&options);
> > + if (ret < 0)
> > + return ret;
> > + if (remaining_options) {
> > + av_log(s, AV_LOG_ERROR, "Some of the provided format options are not recognized\n");
> > + return AVERROR(EINVAL);
> > }
> > avio_flush(oc->pb);
> > return 0;
> > @@ -1683,15 +1688,8 @@ static int hls_start(AVFormatContext *s, VariantStream *vs)
> > }
> > }
> > if (c->segment_type != SEGMENT_TYPE_FMP4) {
> > - /* We only require one PAT/PMT per segment. */
> > if (oc->oformat->priv_class && oc->priv_data) {
> > - char period[21];
> > -
> > - snprintf(period, sizeof(period), "%d", (INT_MAX / 2) - 1);
> > -
> > av_opt_set(oc->priv_data, "mpegts_flags", "resend_headers", 0);
> > - av_opt_set(oc->priv_data, "sdt_period", period, 0);
> > - av_opt_set(oc->priv_data, "pat_period", period, 0);
> > }
> > if (c->flags & HLS_SINGLE_FILE) {
> > set_http_options(s, &options, c);
> > --
> > 2.16.4
> >
LGTM
> > _______________________________________________
> > ffmpeg-devel mailing list
> > ffmpeg-devel at ffmpeg.org
> > https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
> >
> > To unsubscribe, visit link above, or email
> > ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
> _______________________________________________
> ffmpeg-devel mailing list
> ffmpeg-devel at ffmpeg.org
> https://ffmpeg.org/mailman/listinfo/ffmpeg-devel
>
> To unsubscribe, visit link above, or email
> ffmpeg-devel-request at ffmpeg.org with subject "unsubscribe".
More information about the ffmpeg-devel
mailing list