[FFmpeg-devel] [PATCH v2] format/imfdec: improve error handling when selecting tracks for playback
pal at sandflow.com
pal at sandflow.com
Mon Sep 19 18:26:18 EEST 2022
From: Pierre-Anthony Lemieux <pal at palemieux.com>
---
libavformat/imfdec.c | 15 ++++++++++++---
1 file changed, 12 insertions(+), 3 deletions(-)
diff --git a/libavformat/imfdec.c b/libavformat/imfdec.c
index 5bbe7a53f8..9ba8b6de8b 100644
--- a/libavformat/imfdec.c
+++ b/libavformat/imfdec.c
@@ -681,8 +681,11 @@ static IMFVirtualTrackPlaybackCtx *get_next_track_with_minimum_timestamp(AVForma
{
IMFContext *c = s->priv_data;
IMFVirtualTrackPlaybackCtx *track;
-
AVRational minimum_timestamp = av_make_q(INT32_MAX, 1);
+
+ if (!c->track_count)
+ return NULL;
+
for (uint32_t i = c->track_count; i > 0; i--) {
av_log(s, AV_LOG_TRACE, "Compare track %d timestamp " AVRATIONAL_FORMAT
" to minimum " AVRATIONAL_FORMAT
@@ -697,8 +700,6 @@ static IMFVirtualTrackPlaybackCtx *get_next_track_with_minimum_timestamp(AVForma
}
}
- av_log(s, AV_LOG_DEBUG, "Found next track to read: %d (timestamp: %lf / %lf)\n",
- track->index, av_q2d(track->current_timestamp), av_q2d(minimum_timestamp));
return track;
}
@@ -761,6 +762,14 @@ static int imf_read_packet(AVFormatContext *s, AVPacket *pkt)
track = get_next_track_with_minimum_timestamp(s);
+ if (!track) {
+ av_log(s, AV_LOG_ERROR, "No track found for playback\n");
+ return AVERROR_INVALIDDATA;
+ }
+
+ av_log(s, AV_LOG_DEBUG, "Found track %d to read at timestamp %lf\n",
+ track->index, av_q2d(track->current_timestamp));
+
ret = get_resource_context_for_timestamp(s, track, &resource);
if (ret)
return ret;
--
2.25.1
More information about the ffmpeg-devel
mailing list