From afd31d93b534cf89d5d0e1fc0496b1f4d5a0fa5f Mon Sep 17 00:00:00 2001 From: Peter Bloomfield Date: Sat, 5 Jun 2021 10:14:24 -0400 Subject: [PATCH] tests/tls-certificate: Add context to the new test Add a comment to `pem_parser_no_sentinel()` to explain what it is testing and how it works. --- gio/tests/tls-certificate.c | 4 ++++ 1 file changed, 4 insertions(+) diff --git a/gio/tests/tls-certificate.c b/gio/tests/tls-certificate.c index ac046d15d..9dd415091 100644 --- a/gio/tests/tls-certificate.c +++ b/gio/tests/tls-certificate.c @@ -220,6 +220,10 @@ pem_parser_no_sentinel (void) memmove (pem_copy, pem, pem_len); g_free (pem); + /* Check whether the parser respects the @length parameter. + * pem_copy is allocated exactly pem_len bytes, so accessing memory + * outside its bounds will be detected by, for example, valgrind or + * asan. */ cert = g_tls_certificate_new_from_pem (pem_copy, pem_len, &error); g_assert_no_error (error); g_assert_nonnull (cert);