Question

Consider the following C++ codes :

using namespace std;
vector<char*> aCharPointerRow;
aCharPointerRow.push_back("String_11");
aCharPointerRow.push_back("String_12");
aCharPointerRow.push_back("String_13");
for (int i=0; i<aCharPointerRow.size(); i++)  {
   cout << aCharPointerRow[i] << ",";
}
aCharPointerRow.clear();

After the aCharPointerRow.clear(); line, the character pointer elements in aCharPointerRow should all be removed.

Is there a memory leak in the above C++ code ? Do I need to explicitly free the memory allocated to the char* strings ? If yes, how ?

Thanks for any suggestion.

Answer 1

Is there a memory leak in the above C++ code?
There is no memory leak.

Since you never used new you do not need to call delete. You only need to deallocate dynamicmemory if it was allocated in first place.

Note that ideally, You should be using vector of std::string.

std::vector<std::string> str;
str.push_back("String_11");
str.push_back("String_12");
str.push_back("String_13");

You could use std::string.c_str() in case you need to get the underlying character pointer(char *), which lot of C api expect as an parameter.