From 4d44675283e6c92d3294592be4fce7c9d89c19b6 Mon Sep 17 00:00:00 2001 From: Pirama Arumuga Nainar Date: Thu, 9 Jul 2015 10:15:15 -0700 Subject: [PATCH] Use delete[] to de-allocate pointers from new[] Upstream clang has a new diagnostic, '-Wmismatched-new-delete' to generate a warning (that becomes an error with -Werror) when delete is used to deallocate pointers created with 'new[]'. This patch fixes 'delete's that trigger this warning/error. Change-Id: Id7b6f6c2df3e860e576de55e2f61f3d2be3a3986 --- tests/buffer_tests.cpp | 12 ++++++------ 1 file changed, 6 insertions(+), 6 deletions(-) diff --git a/tests/buffer_tests.cpp b/tests/buffer_tests.cpp index 496738211..7d830bb16 100644 --- a/tests/buffer_tests.cpp +++ b/tests/buffer_tests.cpp @@ -256,7 +256,7 @@ void RunSingleBufferAlignTest( VerifyFencepost(&buf_align[len]); } } - delete buf; + delete[] buf; } void RunSrcDstBufferAlignTest( @@ -292,8 +292,8 @@ void RunSrcDstBufferAlignTest( VerifyFencepost(&dst_align[len]); } } - delete src; - delete dst; + delete[] src; + delete[] dst; } void RunCmpBufferAlignTest( @@ -344,8 +344,8 @@ void RunCmpBufferAlignTest( } } } - delete buf1; - delete buf2; + delete[] buf1; + delete[] buf2; } void RunSingleBufferOverreadTest(void (*test_func)(uint8_t*, size_t)) { @@ -389,7 +389,7 @@ void RunSrcDstBufferOverreadTest(void (*test_func)(uint8_t*, uint8_t*, size_t)) } ASSERT_TRUE(mprotect(&memory[pagesize], pagesize, PROT_READ | PROT_WRITE) == 0); free(memory); - delete dst; + delete[] dst; } void RunCmpBufferOverreadTest( -- 2.11.0