On 22/11/2016 22:22, Andreas Cadhalpun wrote:
On 10.11.2016 22:24, Andreas Cadhalpun wrote:
Signed-off-by: Andreas Cadhalpun <andreas.cadhal...@googlemail.com>
---
libavformat/sbgdec.c | 2 +-
1 file changed, 1 insertion(+), 1 deletion(-)
diff --git a/libavformat/sbgdec.c b/libavformat/sbgdec.c
index bb020d7..cbedd12 100644
--- a/libavformat/sbgdec.c
+++ b/libavformat/sbgdec.c
@@ -927,7 +927,7 @@ static void expand_timestamps(void *log, struct sbg_script
*s)
}
}
if (s->start_ts == AV_NOPTS_VALUE)
- s->start_ts = s->opt_start_at_first ? s->tseq[0].ts.t : now;
+ s->start_ts = (s->opt_start_at_first && s->tseq) ? s->tseq[0].ts.t :
now;
s->end_ts = s->opt_duration ? s->start_ts + s->opt_duration :
AV_NOPTS_VALUE; /* may be overridden later by -E option */
cur_ts = now;
Ping. It would be good to have this fixed in 3.2.1.
I don't see how s->tseq can be NULL unless the functions are externally
invoked without a proper state (which they shouldn't be because they're
static).
--
Josh
_______________________________________________
ffmpeg-devel mailing list
ffmpeg-devel@ffmpeg.org
http://ffmpeg.org/mailman/listinfo/ffmpeg-devel
