summaryrefslogtreecommitdiff
path: root/libavcodec/ccaption_dec.c
diff options
context:
space:
mode:
authorClément Bœsch <u@pkh.me>2016-01-05 21:54:28 +0100
committerClément Bœsch <u@pkh.me>2016-01-07 22:42:39 +0100
commitd64fe951c2ee8a85c615c33490889f45c4663862 (patch)
tree814e7b712f6919a069653709ace7035d4325282e /libavcodec/ccaption_dec.c
parent04826b7ff02f3ad2b7ffa4257587df64ee88882c (diff)
lavc/ccaption_dec: fix always true condition
No idea why this wasn't ever detected by a static analyzer.
Diffstat (limited to 'libavcodec/ccaption_dec.c')
-rw-r--r--libavcodec/ccaption_dec.c2
1 files changed, 1 insertions, 1 deletions
diff --git a/libavcodec/ccaption_dec.c b/libavcodec/ccaption_dec.c
index 788e96a38e..8ee2c651e2 100644
--- a/libavcodec/ccaption_dec.c
+++ b/libavcodec/ccaption_dec.c
@@ -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 ) ||