[FFmpeg-cvslog] avcodec/iff: Use signed count
Michael Niedermayer
git at videolan.org
Sat Jul 13 00:04:44 EEST 2024
ffmpeg | branch: master | Michael Niedermayer <michael at niedermayer.cc> | Fri Jul 5 02:21:48 2024 +0200| [cfe66dfebb8a1e1394bcf834b6cc785f280ccecf] | committer: Michael Niedermayer
avcodec/iff: Use signed count
This is more a style fix than a bugfix (CID1604392 Overflowed constant)
Sponsored-by: Sovereign Tech Fund
Signed-off-by: Michael Niedermayer <michael at niedermayer.cc>
> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=cfe66dfebb8a1e1394bcf834b6cc785f280ccecf
---
libavcodec/iff.c | 4 ++--
1 file changed, 2 insertions(+), 2 deletions(-)
diff --git a/libavcodec/iff.c b/libavcodec/iff.c
index 4b3e8e0c21..13010b451e 100644
--- a/libavcodec/iff.c
+++ b/libavcodec/iff.c
@@ -523,7 +523,7 @@ static int decode_byterun2(uint8_t *dst, int height, int line_size,
GetByteContext *gb)
{
GetByteContext cmds;
- unsigned count;
+ int count;
int i, y_pos = 0, x_pos = 0;
if (bytestream2_get_be32(gb) != MKBETAG('V', 'D', 'A', 'T'))
@@ -531,7 +531,7 @@ static int decode_byterun2(uint8_t *dst, int height, int line_size,
bytestream2_skip(gb, 4);
count = bytestream2_get_be16(gb) - 2;
- if (bytestream2_get_bytes_left(gb) < count)
+ if (count < 0 || bytestream2_get_bytes_left(gb) < count)
return 0;
bytestream2_init(&cmds, gb->buffer, count);
More information about the ffmpeg-cvslog
mailing list