On Wed, Feb 13, 2019 at 12:16:36PM +0000, Derek Buitenhuis wrote:
> On 12/02/2019 22:28, Michael Niedermayer wrote:
> > @@ -5048,7 +5048,7 @@ static int mov_read_sidx(MOVContext *c, AVIOContext
> > *pb, MOVAtom atom)
> > for (i = 0; i < c->fc->nb_streams; i++) {
> > st = c->fc->streams[i];
> > sc = st->priv_data;
> > - if (!sc->has_sidx) {
> > + if (ref_st && !sc->has_sidx) {
> > st->duration = sc->track_end =
> > av_rescale(ref_st->duration, sc->time_scale, ref_sc->time_scale);
> > }
> > }
>
> Is it perhaps better to not run this loop at all if ref_sc or ref_st do not
> exist?will commit with the check moved to the loop thanks [...] -- Michael GnuPG fingerprint: 9FF2128B147EF6730BADF133611EC787040B0FAB "You are 36 times more likely to die in a bathtub than at the hands of a terrorist. Also, you are 2.5 times more likely to become a president and 2 times more likely to become an astronaut, than to die in a terrorist attack." -- Thoughty2
signature.asc
Description: PGP signature
_______________________________________________ ffmpeg-devel mailing list ffmpeg-devel@ffmpeg.org http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
