Set Pointer To Null - Memory Leak?

Question

1.00/5 (1 vote)

See more:

Hi,

vector<unsigned int>* pvecnDigit = new vector<unsigned int>;

pvecnDigit->push_back(2);

pvecnDigit = NULL;

Here, I did not use delete to kill vector pointer but set it to NULL. Is there any Memory Leak?

Posted 10-Aug-12 3:38am

AJ83

Add a Solution

3 solutions

Solution 3

I wonder why you even bothered to ask, if you already knew about using delete. If your reason for asking is that you want to know how to avoid having to take care of memory allocation and deallocation, just put that vector on the stack, rather than creating it on the heap:

C++

{
  vector<unsigned int> vecnDigit;
  vecnDigit.push_back(2);
}

Note that instead of creating the vector on the heap, I just put the declaration and use of the vector into a code block, limiting it's scope. That way your use of that vector is still limited to the lines of code between your declaration, and the end of this block. But as it's declared on the stack, you don't need to take care of allocation or deallocation.

Posted 10-Aug-12 4:34am

Stefan_Lang

Add a Solution

Add your solution here

Treat my content as plain text, not as HTML

Preview 0

…

Existing Members

Sign in to your account

...or Join us

Download, Vote, Comment, Publish.

Your Email
Password
Forgot your password?

Your Email
This email is in use. Do you need your password?
Optional Password

I have read and agree to the Terms of Service and Privacy Policy
Please subscribe me to the CodeProject newsletters

When answering a question please:

Read the question carefully.
Understand that English isn't everyone's first language so be lenient of bad spelling and grammar.
If a question is poorly phrased then either ask for clarification, ignore it, or edit the question and fix the problem. Insults are not welcome.
Don't tell someone to read the manual. Chances are they have and don't get it. Provide an answer or move on to the next question.

Let's work to help developers, not make them feel stupid.

This content, along with any associated source code and files, is licensed under The Code Project Open License (CPOL)

Orjan Westin · Accepted Answer · 2012-08-10T03:53:00

Solution 1

Yes. The object created by new still exist, but can't be accessed.

Posted 10-Aug-12 3:53am

Orjan Westin

Espen Harlinn · Accepted Answer · 2012-08-10T03:53:00

Solution 2

Yes, you need to delete the vector pointed to by pvecnDigit:

vector<unsigned int="">* pvecnDigit = new vector<unsigned int="">;
pvecnDigit->push_back(2);
delete pvecnDigit;
pvecnDigit = NULL;

Best regards
Espen Harlinn

Posted 10-Aug-12 3:53am

Espen Harlinn

Updated 10-Aug-12 3:56am

v2

Comments

Volynsky Alex 10-Aug-12 9:55am

Good answer!

Espen Harlinn 10-Aug-12 9:57am

Thank you :-D

Sergey Alexandrovich Kryukov 10-Aug-12 15:18pm

Of course, a 5.
--SA

Espen Harlinn 10-Aug-12 15:35pm

Thank you, Sergey :-D