aboutsummaryrefslogtreecommitdiffstats
diff options
context:
space:
mode:
authorMichael Niedermayer <michael@niedermayer.cc>2020-10-22 18:18:43 +0200
committerMichael Niedermayer <michael@niedermayer.cc>2020-10-22 18:51:27 +0200
commitd34e4904cd6d965693b285713660f4e84200d60b (patch)
treec300c4d723b96a259a66618375b948c28943265b
parentc0d7fd269beed030fc767fee28d9dbe111bc4427 (diff)
downloadffmpeg-d34e4904cd6d965693b285713660f4e84200d60b.tar.gz
avformat/segafilm: Do not assume AV_CODEC_ID_NONE is 0
Suggested-by: Andreas Rheinhardt <andreas.rheinhardt@gmail.com> Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
-rw-r--r--libavformat/segafilm.c12
1 files changed, 6 insertions, 6 deletions
diff --git a/libavformat/segafilm.c b/libavformat/segafilm.c
index 17592c905e..91cd7b7c2b 100644
--- a/libavformat/segafilm.c
+++ b/libavformat/segafilm.c
@@ -144,11 +144,11 @@ static int film_read_header(AVFormatContext *s)
film->video_type = AV_CODEC_ID_NONE;
}
- if (!film->video_type && !film->audio_type)
+ if (film->video_type == AV_CODEC_ID_NONE && film->audio_type == AV_CODEC_ID_NONE)
return AVERROR_INVALIDDATA;
/* initialize the decoder streams */
- if (film->video_type) {
+ if (film->video_type != AV_CODEC_ID_NONE) {
st = avformat_new_stream(s, NULL);
if (!st)
return AVERROR(ENOMEM);
@@ -169,7 +169,7 @@ static int film_read_header(AVFormatContext *s)
}
}
- if (film->audio_type) {
+ if (film->audio_type != AV_CODEC_ID_NONE) {
st = avformat_new_stream(s, NULL);
if (!st)
return AVERROR(ENOMEM);
@@ -244,7 +244,7 @@ static int film_read_header(AVFormatContext *s)
film->sample_table[i].pts = AV_RB32(&scratch[8]) & 0x7FFFFFFF;
film->sample_table[i].keyframe = (scratch[8] & 0x80) ? 0 : AVINDEX_KEYFRAME;
video_frame_counter++;
- if (film->video_type)
+ if (film->video_type != AV_CODEC_ID_NONE)
av_add_index_entry(s->streams[film->video_stream_index],
film->sample_table[i].sample_offset,
film->sample_table[i].pts,
@@ -253,10 +253,10 @@ static int film_read_header(AVFormatContext *s)
}
}
- if (film->audio_type)
+ if (film->audio_type != AV_CODEC_ID_NONE)
s->streams[film->audio_stream_index]->duration = audio_frame_counter;
- if (film->video_type)
+ if (film->video_type != AV_CODEC_ID_NONE)
s->streams[film->video_stream_index]->duration = video_frame_counter;
film->current_sample = 0;