diff mbox series

[FFmpeg-devel,2/7] avcodec/pgxdec: Avoid always-false checks

Message ID AS8PR01MB7944278D88D5E194F35EC2768FF99@AS8PR01MB7944.eurprd01.prod.exchangelabs.com
State Accepted
Commit be6cd7dfd122cf7cd8841fd41d02067867f25898
Headers show
Series [FFmpeg-devel,1/7] avcodec/pgxdec: Make better use of size check | expand

Checks

Context Check Description
yinshiyou/make_loongarch64 success Make finished
yinshiyou/make_fate_loongarch64 success Make fate finished
andriy/make_x86 success Make finished
andriy/make_fate_x86 success Make fate finished

Commit Message

Andreas Rheinhardt April 24, 2022, 4:42 a.m. UTC 
We have already checked that there is data to be read.

Signed-off-by: Andreas Rheinhardt <andreas.rheinhardt@outlook.com>
---
 libavcodec/pgxdec.c | 20 ++++++++++----------
 1 file changed, 10 insertions(+), 10 deletions(-)

Comments

Paul B Mahol April 24, 2022, 11:18 a.m. UTC | #1 
lgtm
diff mbox series

Patch

diff --git a/libavcodec/pgxdec.c b/libavcodec/pgxdec.c
index 154a683b4f..9c474036da 100644
--- a/libavcodec/pgxdec.c
+++ b/libavcodec/pgxdec.c
@@ -32,9 +32,9 @@  static int pgx_get_number(AVCodecContext *avctx, GetByteContext *g, int *number)
     *number = 0;
     while (1) {
         uint64_t temp;
-        if (!bytestream2_get_bytes_left(g))
+        if (bytestream2_get_bytes_left(g) <= 0)
             return AVERROR_INVALIDDATA;
-        digit = bytestream2_get_byte(g);
+        digit = bytestream2_get_byteu(g);
         if (digit == ' ' || digit == 0xA || digit == 0xD)
             break;
         else if (digit < '0' || digit > '9')
@@ -59,22 +59,22 @@  static int pgx_decode_header(AVCodecContext *avctx, GetByteContext *g,
     if (bytestream2_get_bytes_left(g) < 12)
         return AVERROR_INVALIDDATA;
 
-    bytestream2_skip(g, 6);
+    bytestream2_skipu(g, 6);
 
     // Is the component signed?
-    byte = bytestream2_peek_byte(g);
+    byte = bytestream2_peek_byteu(g);
     if (byte == '+') {
         *sign = 0;
-        bytestream2_skip(g, 1);
+        bytestream2_skipu(g, 1);
     } else if (byte == '-') {
         *sign = 1;
-        bytestream2_skip(g, 1);
+        bytestream2_skipu(g, 1);
     } else if (byte == 0)
         goto error;
 
-    byte = bytestream2_peek_byte(g);
+    byte = bytestream2_peek_byteu(g);
     if (byte == ' ')
-        bytestream2_skip(g, 1);
+        bytestream2_skipu(g, 1);
     else if (byte == 0)
         goto error;
 
@@ -104,9 +104,9 @@  error:
             for (j = 0; j < width; j++) {                                                   \
                 unsigned val;                                                               \
                 if (sign)                                                                   \
-                    val = (PIXEL)bytestream2_get_ ##suffix(g) + (1 << (depth - 1));         \
+                    val = (PIXEL)bytestream2_get_ ##suffix##u(g) + (1 << (depth - 1));      \
                 else                                                                        \
-                    val = bytestream2_get_ ##suffix(g);                                     \
+                    val = bytestream2_get_ ##suffix##u(g);                                  \
                 val <<= (D - depth);                                                        \
                 *(line + j) = val;                                                          \
             }                                                                               \