[SCM] libav/experimental: Simplify checks, use that we know that cnt will not be < 0

siretart at users.alioth.debian.org siretart at users.alioth.debian.org
Sun Jun 30 15:55:49 UTC 2013


The following commit has been merged in the experimental branch:
commit c0a8b87600a1c24a65898d985c11adda56fa6104
Author: Reimar Döffinger <Reimar.Doeffinger at gmx.de>
Date:   Wed Jan 31 21:00:48 2007 +0000

    Simplify checks, use that we know that cnt will not be < 0
    
    Originally committed as revision 7787 to svn://svn.ffmpeg.org/ffmpeg/trunk

diff --git a/libavcodec/lzo.c b/libavcodec/lzo.c
index c78e1f3..54edc80 100644
--- a/libavcodec/lzo.c
+++ b/libavcodec/lzo.c
@@ -86,11 +86,11 @@ static inline int get_len(LZOContext *c, int x, int mask) {
 static inline void copy(LZOContext *c, int cnt) {
     register uint8_t *src = c->in;
     register uint8_t *dst = c->out;
-    if (src + cnt > c->in_end || src + cnt < src) {
+    if (cnt > c->in_end - src) {
         cnt = c->in_end - src;
         c->error |= LZO_INPUT_DEPLETED;
     }
-    if (dst + cnt > c->out_end || dst + cnt < dst) {
+    if (cnt > c->out_end - dst) {
         cnt = c->out_end - dst;
         c->error |= LZO_OUTPUT_FULL;
     }
@@ -121,7 +121,7 @@ static inline void copy_backptr(LZOContext *c, int back, int cnt) {
         c->error |= LZO_INVALID_BACKPTR;
         return;
     }
-    if (dst + cnt > c->out_end || dst +  cnt < dst) {
+    if (cnt > c->out_end - dst) {
         cnt = c->out_end - dst;
         c->error |= LZO_OUTPUT_FULL;
     }

-- 
Libav/FFmpeg packaging



More information about the pkg-multimedia-commits mailing list