Use arrays on stack and avoid use of new and delete operators.

Summary: Also fix an error found with LLVM_USE_SANITIZER=Address.

Reviewers: nelhage

Subscribers: libc-commits

Tags: #libc-project

Differential Revision: https://reviews.llvm.org/D68761

llvm-svn: 374374

2 files changed, 4 insertions, 12 deletions

diff --git a/libc/src/string/strcat/strcat_test.cpp b/libc/src/string/strcat/strcat_test.cpp
index 26bcae2373e..3b8a7a7e447 100644
--- a/libc/src/string/strcat/strcat_test.cpp
+++ b/libc/src/string/strcat/strcat_test.cpp
@@ -13,7 +13,7 @@
 
 TEST(StrCatTest, EmptyDest) {
   std::string abc = "abc";
-  char *dest = new char[4];
+  char dest[4];
 
   dest[0] = '\0';
 
@@ -21,13 +21,11 @@ TEST(StrCatTest, EmptyDest) {
   ASSERT_EQ(dest, result);
   ASSERT_EQ(std::string(dest), abc);
   ASSERT_EQ(std::string(dest).size(), abc.size());
-
-  delete[] dest;
 }
 
 TEST(StrCatTest, NonEmptyDest) {
   std::string abc = "abc";
-  char *dest = new char[4];
+  char dest[7];
 
   dest[0] = 'x';
   dest[1] = 'y';
@@ -38,6 +36,4 @@ TEST(StrCatTest, NonEmptyDest) {
   ASSERT_EQ(dest, result);
   ASSERT_EQ(std::string(dest), std::string("xyz") + abc);
   ASSERT_EQ(std::string(dest).size(), abc.size() + 3);
-
-  delete[] dest;
 }
diff --git a/libc/src/string/strcpy/strcpy_test.cpp b/libc/src/string/strcpy/strcpy_test.cpp
index 48f55f24649..e68ea5103db 100644
--- a/libc/src/string/strcpy/strcpy_test.cpp
+++ b/libc/src/string/strcpy/strcpy_test.cpp
@@ -13,19 +13,17 @@
 
 TEST(StrCpyTest, EmptyDest) {
   std::string abc = "abc";
-  char *dest = new char[4];
+  char dest[4];
 
   char *result = __llvm_libc::strcpy(dest, abc.c_str());
   ASSERT_EQ(dest, result);
   ASSERT_EQ(std::string(dest), abc);
   ASSERT_EQ(std::string(dest).size(), abc.size());
-
-  delete[] dest;
 }
 
 TEST(StrCpyTest, OffsetDest) {
   std::string abc = "abc";
-  char *dest = new char[7];
+  char dest[7];
 
   dest[0] = 'x';
   dest[1] = 'y';
@@ -35,6 +33,4 @@ TEST(StrCpyTest, OffsetDest) {
   ASSERT_EQ(dest + 3, result);
   ASSERT_EQ(std::string(dest), std::string("xyz") + abc);
   ASSERT_EQ(std::string(dest).size(), abc.size() + 3);
-
-  delete[] dest;
 }