unit-tests: Verify two bytes at once when testing chunk_clear()

This reduces the chances of arbitrary test failures if the memory area
already got overwritten.

diff --git a/src/libstrongswan/tests/suites/test_chunk.c b/src/libstrongswan/tests/suites/test_chunk.c
index e373fbdb623e5e841c00775f773d2fde8e2d22f2..34ace28944871341f5b76a6a7ddfec6b5b28082b 100644 (file)
--- a/src/libstrongswan/tests/suites/test_chunk.c
+++ b/src/libstrongswan/tests/suites/test_chunk.c
@@ -117,10 +117,13 @@ START_TEST(test_chunk_clear)
        }
        chunk_clear(&chunk);
        /* check memory area of freed chunk. We can't use ck_assert() for this
-        * test directly, as it might allocate data at the freed area. */
-       for (i = 0; i < 64; i++)
+        * test directly, as it might allocate data at the freed area.  comparing
+        * two bytes at once reduces the chances of conflicts if memory got
+        * overwritten already */
+       for (i = 0; i < 64; i += 2)
        {
-               if (ptr[i] != 0 && ptr[i] == i)
+               if (ptr[i] != 0 && ptr[i] == i &&
+                       ptr[i+1] != 0 && ptr[i+1] == i+1)
                {
                        cleared = FALSE;
                        break;