generic-library/bionic
1
0
Fork 0
Code Issues Pull Requests Releases Wiki Activity

Use delete[] to de-allocate pointers from new[]

Browse Source 
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
This commit is contained in:
Pirama Arumuga Nainar 2015-07-09 10:15:15 -07:00
parent 90de6820f7
commit 4d44675283
1 changed files with 6 additions and 6 deletions
Show Stats Download Patch File Download Diff File Expand all files Collapse all files

12
tests/buffer_tests.cpp
View File

@ -256,7 +256,7 @@ void RunSingleBufferAlignTest(
VerifyFencepost(&buf_align[len]); VerifyFencepost(&buf_align[len]);
} }
} }
delete buf; delete[] buf;
} }
void RunSrcDstBufferAlignTest( void RunSrcDstBufferAlignTest(
@ -292,8 +292,8 @@ void RunSrcDstBufferAlignTest(
VerifyFencepost(&dst_align[len]); VerifyFencepost(&dst_align[len]);
} }
} }
delete src; delete[] src;
delete dst; delete[] dst;
} }
void RunCmpBufferAlignTest( void RunCmpBufferAlignTest(
@ -344,8 +344,8 @@ void RunCmpBufferAlignTest(
} }
} }
} }
delete buf1; delete[] buf1;
delete buf2; delete[] buf2;
} }
void RunSingleBufferOverreadTest(void (*test_func)(uint8_t*, size_t)) { 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); ASSERT_TRUE(mprotect(&memory[pagesize], pagesize, PROT_READ | PROT_WRITE) == 0);
free(memory); free(memory);
delete dst; delete[] dst;
} }
void RunCmpBufferOverreadTest( void RunCmpBufferOverreadTest(