diff options
| author | Michael Niedermayer <michael@niedermayer.cc> | 2020-08-15 01:07:44 +0200 |
|---|---|---|
| committer | Michael Niedermayer <michael@niedermayer.cc> | 2020-08-18 14:56:04 +0200 |
| commit | e8a88a16f78e66c8d7645b5f71dc8390b033fa70 (patch) | |
| tree | 48f084ded1ba7da9f7c5ce283ae734e64d21398c /libavformat/mpeg.c | |
| parent | d08c3f56ec1340b4cdf7b5f0be99a06d6a682abf (diff) | |
| download | ffmpeg-e8a88a16f78e66c8d7645b5f71dc8390b033fa70.tar.gz | |
avformat/mpeg: Check avio_read() return value in get_pts()
Found-by: Thierry Foucu <tfoucu@gmail.com>
Fixes: Use-of-uninitialized-value
Reviewed-by: Thierry Foucu <tfoucu@gmail.com>
Signed-off-by: Michael Niedermayer <michael@niedermayer.cc>
Diffstat (limited to 'libavformat/mpeg.c')
| -rw-r--r-- | libavformat/mpeg.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/libavformat/mpeg.c b/libavformat/mpeg.c index 265b2bd1ad..a5e17925ce 100644 --- a/libavformat/mpeg.c +++ b/libavformat/mpeg.c @@ -147,9 +147,12 @@ static int mpegps_read_header(AVFormatContext *s) static int64_t get_pts(AVIOContext *pb, int c) { uint8_t buf[5]; + int ret; buf[0] = c < 0 ? avio_r8(pb) : c; - avio_read(pb, buf + 1, 4); + ret = avio_read(pb, buf + 1, 4); + if (ret < 4) + return AV_NOPTS_VALUE; return ff_parse_pes_pts(buf); } |