Segmentation fault when deleting template array

Question

So i need to create an array of dynamic size containing pairs of objects and I have the following piece of code:

#include <utility>
#include <iostream>
#include <string>

using namespace std;

int main() {

    cout << "Hey #1" << endl;

    pair<string, int> *array;

    array = new pair<string, int>[4];

    cout << "Hey #2" << endl;

    delete array;

    cout << "Hey #3" << endl;
}

and the output I get is

Hey #1
Hey #2
Segmentation fault (core dumped)

which means that something goes wrong when the delete operator gets called?

What am I missing here? Any help would be appreciated!

`array = new pair[4];` -- What are you really trying to accomplish here? Second, do not call your variable `array`, as there is a `std::array` class that exists in standard C++. — PaulMcKenzie, Mar 19 '20 at 13:37
Since you stated you are creating an array, what form of `delete` should you be using when deleting an array? Also `std::array, 4>` would avoid having to use `new`. — PaulMcKenzie, Mar 19 '20 at 13:39

score 4 · Answer 1 · answered Mar 19 '20 at 13:38

4

You need to use the operator delete []

delete [] array;

answered Mar 19 '20 at 13:38

Vlad from Moscow

301,070
26
186
335

score 4 · Answer 2 · edited Mar 19 '20 at 14:04

4

When you allocate objects with new, you need to free them with delete; when you allocate objects with new[] you need to free them with delete[]. Change your code to:

delete[] array;

edited Mar 19 '20 at 14:04

Ðаn

10,934
11
59
95

answered Mar 19 '20 at 13:40

DeducibleSteak

1,398
11
23

score 4 · Answer 3 · edited Mar 19 '20 at 14:03

4

When deleting a dynamic array use delete[] instead of delete to delete multiple dynamically allocated data. Hope this solves your problem.

edited Mar 19 '20 at 14:03

Ðаn

10,934
11
59
95

answered Mar 19 '20 at 13:40

Hussam

93
5

Segmentation fault when deleting template array

3 Answers3