[FFmpeg-cvslog] escape124: explicitly set get_bits1 variable

Vittorio Giovara git at videolan.org
Thu Nov 13 14:06:54 CET 2014


ffmpeg | branch: master | Vittorio Giovara <vittorio.giovara at gmail.com> | Tue Nov 11 17:40:09 2014 +0100| [1a9c1333b5d70b427c82cb98f383aa2fa9b2b319] | committer: Vittorio Giovara

escape124: explicitly set get_bits1 variable

Makes coverity less confused.

CC: libav-stable at libav.org
Bug-Id: CID 1203487

> http://git.videolan.org/gitweb.cgi/ffmpeg.git/?a=commit;h=1a9c1333b5d70b427c82cb98f383aa2fa9b2b319
---

 libavcodec/escape124.c |    7 ++++---
 1 file changed, 4 insertions(+), 3 deletions(-)

diff --git a/libavcodec/escape124.c b/libavcodec/escape124.c
index 30f22e0..6c4654c 100644
--- a/libavcodec/escape124.c
+++ b/libavcodec/escape124.c
@@ -147,10 +147,11 @@ static MacroBlock decode_macroblock(Escape124Context* s, GetBitContext* gb,
     // This function reads a maximum of 22 bits; the callers
     // guard this function appropriately
     unsigned block_index, depth;
-
-    if (get_bits1(gb)) {
+    int value = get_bits1(gb);
+    if (value) {
         static const char transitions[3][2] = { {2, 1}, {0, 2}, {1, 0} };
-        *codebook_index = transitions[*codebook_index][get_bits1(gb)];
+        value = get_bits1(gb);
+        *codebook_index = transitions[*codebook_index][value];
     }
 
     depth = s->codebooks[*codebook_index].depth;



More information about the ffmpeg-cvslog mailing list