Don't mutate dst_size
This lead to `dst_size` not having the intended value in subsequent code.
Signed-off-by: Gilles Peskine <Gilles.Peskine@arm.com>
diff --git a/tests/suites/test_suite_base64.function b/tests/suites/test_suite_base64.function
index 5d8ed9b..3bd9932 100644
--- a/tests/suites/test_suite_base64.function
+++ b/tests/suites/test_suite_base64.function
@@ -116,9 +116,8 @@
if (result == 0 && dst_size != 0) {
mbedtls_free(dst);
dst = NULL;
- dst_size -= 1;
- TEST_CALLOC(dst, dst_size);
- TEST_EQUAL(mbedtls_base64_decode(dst, dst_size, &len,
+ TEST_CALLOC(dst, dst_size - 1);
+ TEST_EQUAL(mbedtls_base64_decode(dst, dst_size - 1, &len,
src, src_len),
MBEDTLS_ERR_BASE64_BUFFER_TOO_SMALL);
TEST_EQUAL(correct_dst_len, len);