[FFmpeg-cvslog] avcodec/h261dec: Use skip_1stop_8data_bits()

Michael Niedermayer git at videolan.org
Wed Oct 30 18:35:34 CET 2013


ffmpeg | branch: master | Michael Niedermayer <michaelni at gmx.at> | Wed Oct 30 18:28:00 2013 +0100| [711e981276d02895605a7099c4f873771d616c77] | committer: Michael Niedermayer

avcodec/h261dec: Use skip_1stop_8data_bits()

Signed-off-by: Michael Niedermayer <michaelni at gmx.at>

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

 libavcodec/h261dec.c |   14 ++++----------
 1 file changed, 4 insertions(+), 10 deletions(-)

diff --git a/libavcodec/h261dec.c b/libavcodec/h261dec.c
index 2995d8c..26b8855 100644
--- a/libavcodec/h261dec.c
+++ b/libavcodec/h261dec.c
@@ -125,11 +125,8 @@ static int h261_decode_gob_header(H261Context *h)
     }
 
     /* GEI */
-    while (get_bits1(&s->gb) != 0) {
-        skip_bits(&s->gb, 8);
-        if (get_bits_left(&s->gb) <= 0)
-            return AVERROR_INVALIDDATA;
-    }
+    if (skip_1stop_8data_bits(&s->gb) < 0)
+        return AVERROR_INVALIDDATA;
 
     if (s->qscale == 0) {
         av_log(s->avctx, AV_LOG_ERROR, "qscale has forbidden 0 value\n");
@@ -508,11 +505,8 @@ static int h261_decode_picture_header(H261Context *h)
     skip_bits1(&s->gb); /* Reserved */
 
     /* PEI */
-    while (get_bits1(&s->gb) != 0) {
-        skip_bits(&s->gb, 8);
-        if (get_bits_left(&s->gb) <= 0)
-            return AVERROR_INVALIDDATA;
-    }
+    if (skip_1stop_8data_bits(&s->gb) < 0)
+        return AVERROR_INVALIDDATA;
 
     /* H.261 has no I-frames, but if we pass AV_PICTURE_TYPE_I for the first
      * frame, the codec crashes if it does not contain all I-blocks



More information about the ffmpeg-cvslog mailing list