fix #37595 (mcrypt_generic calculates data length in wrong way)

diff --git a/ext/mcrypt/mcrypt.c b/ext/mcrypt/mcrypt.c
index 878c883611d46fc4d96574d7377c07a36fd61ea5..a76ae2ee58a23bc7598baa1cc4115951d1da821a 100644 (file)
--- a/ext/mcrypt/mcrypt.c
+++ b/ext/mcrypt/mcrypt.c
@@ -496,7 +496,7 @@ PHP_FUNCTION(mcrypt_generic)
        /* Check blocksize */
        if (mcrypt_enc_is_block_mode(pm->td) == 1) { /* It's a block algorithm */
                block_size = mcrypt_enc_get_block_size(pm->td);
-               data_size = (((Z_STRLEN_PP(data) - 1) / block_size) + 1) * block_size;
+               data_size = ((Z_STRLEN_PP(data) / block_size) + 1) * block_size;
                data_s = emalloc(data_size + 1);
                memset(data_s, 0, data_size);
                memcpy(data_s, Z_STRVAL_PP(data), Z_STRLEN_PP(data));

diff --git a/ext/mcrypt/tests/bug37595.phpt b/ext/mcrypt/tests/bug37595.phpt
new file mode 100644 (file)
index 0000000..a74b8cf
Binary files /dev/null and b/ext/mcrypt/tests/bug37595.phpt differ