This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
[C++ PATCH] Fix C++ violation in g++.jason/init3.C (PR testsuite/63299)
- From: Jakub Jelinek <jakub at redhat dot com>
- To: Jason Merrill <jason at redhat dot com>
- Cc: gcc-patches at gcc dot gnu dot org
- Date: Tue, 20 Sep 2016 00:04:30 +0200
- Subject: [C++ PATCH] Fix C++ violation in g++.jason/init3.C (PR testsuite/63299)
- Authentication-results: sourceware.org; auth=none
- Reply-to: Jakub Jelinek <jakub at redhat dot com>
Hi!
As valgrind reports, this testcase allocates the string with new [], but
deallocates with delete str.
Fixed thusly, regtested on x86_64-linux and i686-linux, ok for trunk?
2016-09-19 Maxim Ostapenko <m.ostapenko@samsung.com>
Jakub Jelinek <jakub@redhat.com>
PR testsuite/63299
* g++.old-deja/g++.jason/init3.C (My_string::~My_string): Use delete[]
instead of delete.
--- gcc/testsuite/g++.old-deja/g++.jason/init3.C.jj 2008-09-05 12:54:52.000000000 +0200
+++ gcc/testsuite/g++.old-deja/g++.jason/init3.C 2016-09-19 16:06:01.328022761 +0200
@@ -10,7 +10,7 @@ class My_string {
public:
My_string(const char* string);
My_string(const My_string &);
- ~My_string() { delete str; }
+ ~My_string() { delete [] str; }
char* char_p() { return str; }
};
Jakub