commit 94dadcb1a1dced08a80ab9e637d60e9fb624d40c
author James Zern <jzern@google.com> Thu Sep 11 15:26:11 2014 -0700
committer James Zern <jzern@google.com> Tue Oct 14 12:02:16 2014 +0200
tree 5a4f7e2da77dfe090776e0d529b3e36da913ab66
parent 40b3a618ec5b5bad325ef301d4e91dbcacfc919c

webpmux: fix loop_count range check

explicitly check [0, 65535], the use of 'long' was removed in a prior
commit

(cherry picked from commit 0e23c487dad7a1bd9b2cff409144910386823c15)

Change-Id: I70d5bf286908459b5d4d619c657853f0e833b6ea

examples/webpmux.c

diff --git a/examples/webpmux.c b/examples/webpmux.c
index f387492..5fa5387 100644
--- a/examples/webpmux.c
+++ b/examples/webpmux.c
@@ -939,7 +939,7 @@
                 int parse_error = 0;
                 const int loop_count =
                     ExUtilGetInt(feature->args_[i].params_, 10, &parse_error);
-                if (loop_count != (int)loop_count) {
+                if (loop_count < 0 || loop_count > 65535) {
                   // Note: This is only a 'necessary' condition for loop_count
                   // to be valid. The 'sufficient' conditioned in checked in
                   // WebPMuxSetAnimationParams() method called later.