05.11.2016, 19:51, Andreas Cadhalpun kirjoitti: > On 05.11.2016 18:47, Andreas Cadhalpun wrote: >> On 05.11.2016 17:39, Anssi Hannula wrote: >>> @@ -1981,6 +1986,11 @@ static int hls_read_packet(AVFormatContext *s, >>> AVPacket *pkt) >>> >>> pls->ctx->streams[pls->pkt.stream_index]->time_base, >>> AV_TIME_BASE_Q); >>> >>> + /* There may be more situations where this would be useful, but >>> this at least >>> + * handles newly probed codecs properly (i.e. request_probe by >>> mpegts). */ >>> + if (ist->codecpar->codec_id != st->codecpar->codec_id) >>> + set_stream_info_from_input_stream(st, pls, ist); >> >> This has to set: >> ist->internal->need_context_update = 1; > ^ > Should have been 'st' not 'ist'.
Thanks for the comments, followup is a new set with need_context_update = 1 added to set_stream_info_from_input_stream() and an additional patch to add error checks for the use of avcodec_parameters_copy(). Anssi Hannula (3): avformat/hls: Factor copying stream info to a separate function avformat/hls: Fix probing mpegts audio streams that use probing avformat/hls: Add missing error check for avcodec_parameters_copy() -- Anssi Hannula _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel