From: Ilia Alshanetsky <iliaa@php.net>
Date: Sun, 25 Jan 2009 18:27:39 +0000 (+0000)
Subject: MFB: Improved fix for bug #47174 & added a test
X-Git-Tag: php-5.2.9RC1~36
X-Git-Url: https://granicus.if.org/sourcecode?a=commitdiff_plain;h=38969d726bf6ed1d082b85bb6547f441f25b0b07;p=php

MFB: Improved fix for bug #47174 & added a test
---

diff --git a/ext/standard/base64.c b/ext/standard/base64.c
index b163ce4849..b4a291461d 100644
--- a/ext/standard/base64.c
+++ b/ext/standard/base64.c
@@ -158,7 +158,6 @@ PHPAPI unsigned char *php_base64_decode_ex(const unsigned char *str, int length,
 				efree(result);
 				return NULL;
 			}
-			i++;
 			continue;
 		}
 
diff --git a/ext/standard/tests/url/bug47174.phpt b/ext/standard/tests/url/bug47174.phpt
new file mode 100644
index 0000000000..6d08063253
--- /dev/null
+++ b/ext/standard/tests/url/bug47174.phpt
@@ -0,0 +1,18 @@
+--TEST--
+Bug #47174 (base64_decode() interprets pad char in mid string as terminator)
+--FILE--
+<?php
+if (base64_decode("dGVzdA==") == base64_decode("dGVzdA==CRAP")) {
+    echo "Same octect data - Signature Valid\n";
+} else {
+    echo "Invalid Signature\n";
+}
+
+$in = base64_encode("foo") . '==' . base64_encode("bar");
+var_dump($in, base64_decode($in));
+
+?>
+--EXPECT--
+Invalid Signature
+string(10) "Zm9v==YmFy"
+string(6) "foobar"