Имеет ли это утечку памяти?

Question

Имеет ли код ниже утечку памяти. Я подозреваю, что это не так, но инструменты, которые я использую для их обнаружения (Visual Studio + Parasoft C++ test), ничего не мешают. Если это так, я могу это исправить?

//A dynamically allocated array of char pointers 
int numOfStrings = 10, numOfChars = 32; 
char** data = new char*[numOfStrings]; 

//Generate each each individual string 
for(int i = 0; i <numOfStrings; i++) 
    data[i] = new char[numOfChars]; 

//moves the elements 1-5 in the array to the right by one 
int index = 1, boundary = 5, sizeToMove = (boundary - index) * sizeof(numOfChars); 
memmove(&data[index + 1],&data[index],sizeToMove); 

delete[] data; 

EDIT:

Я должен упомянуть, я попытался итерация каждой отдельной строки, как показано ниже, но происходит исключение.

for(int i = 0; i< numOfStrings; i++) 
    delete [] data [i]; 

Answer 1

Да, у вас есть утечка!

Вы должны удалить каждый указатель в массиве перед удалением указателя, указывающего на них.

//delete each array 
for(int i = 0; i <numOfStrings; i++) 
    delete[] data[i]; 

//this is a single pointer, not an array 
delete[] data; 

Теперь у вас не будет утечка

Answer 2

Да, это так. Когда вы удаляете

delete[] data; 

Вы освобождаете память, выделенную для данных. Однако выделенная память

data[i] = new char[numOfChars]; 

по-прежнему не освобожден.

Перед удалением данных вам необходимо перебрать data и удалить каждый data[i].

Как правило, вы должны быть уверены, что у вас есть столько delete s, сколько new s.
Здесь у вас есть numOfStrings + 1new s и только один delete.

еще одна утечка

Поскольку вы делаете

int index = 1, boundary = 5, sizeToMove = (boundary - index) * sizeof(numOfChars); 
memmove(&data[index + 1],&data[index],sizeToMove); 

(Вы не двигаетесь сдвигая пять мест, как вы думали, но 4 места (5 - 1 = 4))

После этой операции

data[2] will get the value of data[1] 

data[2] <- data[1] 
data[3] <- data[2] 
data[4] <- data[3] 
data[5] <- data[4] 

И что было указано data[5] будет потеряно.
data[2], data[1] будет иметь такое же значение (точка в том же месте)

Это может также объяснить, почему вы получаете Segfault, когда вы хотите удалить, итерация data

Answer 3

правило: для каждого «нового ', ДОЛЖЕН быть соответствующий вызов' delete '. У вас этого нет, поэтому у вас есть утечка.