lavc/ccaption_dec: fix always true condition
authorClément Bœsch <u@pkh.me>
Tue, 5 Jan 2016 20:54:28 +0000 (21:54 +0100)
committerClément Bœsch <u@pkh.me>
Thu, 7 Jan 2016 21:42:39 +0000 (22:42 +0100)
No idea why this wasn't ever detected by a static analyzer.

libavcodec/ccaption_dec.c

index 788e96a..8ee2c65 100644 (file)
@@ -455,7 +455,7 @@ static int process_cc608(CCaptionSubContext *ctx, int64_t pts, uint8_t hi, uint8
     int ret = 0;
     if (hi == ctx->prev_cmd[0] && lo == ctx->prev_cmd[1]) {
         /* ignore redundant command */
-    } else if ( (hi == 0x10 && (lo >= 0x40 || lo <= 0x5f)) ||
+    } else if ( (hi == 0x10 && (lo >= 0x40 && lo <= 0x5f)) ||
               ( (hi >= 0x11 && hi <= 0x17) && (lo >= 0x40 && lo <= 0x7f) ) ) {
         handle_pac(ctx, hi, lo);
     } else if ( ( hi == 0x11 && lo >= 0x20 && lo <= 0x2f ) ||