Merge pull request brix#87 from evilboxingdragonslayer/develop

evanvosberg · web-flow · commit 4b7673ebffdd · 2018-09-13T23:34:24.000+02:00
fixed infinite loop bug in zero padding unpad
diff --git a/src/pad-zeropadding.js b/src/pad-zeropadding.js
@@ -17,9 +17,11 @@ CryptoJS.pad.ZeroPadding = {
 
         // Unpad
         var i = data.sigBytes - 1;
-        while (!((dataWords[i >>> 2] >>> (24 - (i % 4) * 8)) & 0xff)) {
-            i--;
+        for (var i = data.sigBytes - 1; i >= 0; i--) {
+            if (((dataWords[i >>> 2] >>> (24 - (i % 4) * 8)) & 0xff)) {
+                data.sigBytes = i + 1;
+                break;
+            }
         }
-        data.sigBytes = i + 1;
     }
 };
