On Tue, 5 Jun 2018, Tomas Härdin wrote:
tor 2018-05-31 klockan 02:05 +0200 skrev Marton Balint:It cannot happen for video streams. > Signed-off-by: Marton Balint <c...@passwd.hu> --- libavformat/mxfdec.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/libavformat/mxfdec.c b/libavformat/mxfdec.c index e80ef62d57..cf1cd71987 100644 --- a/libavformat/mxfdec.c +++ b/libavformat/mxfdec.c @@ -3221,7 +3221,7 @@ static int mxf_set_pts(MXFContext *mxf, AVStream *st, AVPacket *pkt, int64_t nex if (mxf->nb_index_tables >= 1 && mxf->current_edit_unit < t->nb_ptses) { pkt->dts = mxf->current_edit_unit + t->first_dts; pkt->pts = t->ptses[mxf->current_edit_unit]; - } else if (track && track->intra_only) { + } else if (track->intra_only) {I wonder what actually caused this check to be inserted. That is, what could lead to track being NULL that is no longer the case?
I don't know, I cc'd the original patch author about it. IMHO this was never needed because only streams which are added in mxf_add_metadata_stream have their priv_data (track) set to NULL, and those streams are always DATA.
Regards, Marton _______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
