C / C++ / MFC

char * getmestring()
{
char *str = new char[20];

// FROM <a href = "http://www.codeproject.com/script/comments/forums.asp?msg=1371347&forumid=1647#xx1371359xx" rel="nofollow">Stephen Hewitt</a>[<a href = "http://www.codeproject.com/script/comments/forums.asp?msg=1371347&forumid=1647#xx1371359xx" target = "_blank" rel="nofollow">^</a>]
// str = "myname";
strcpy(str, "myname"); 
return str;
}
int main()
{

// FROM <a href = "http://www.codeproject.com/script/comments/forums.asp?msg=1371347&forumid=1647#xx1371347xx" rel="nofollow">namaskaaram</a>[<a href = "http://www.codeproject.com/script/comments/forums.asp?msg=1371347&forumid=1647#xx1371347xx" target = "_blank" rel="nofollow">^</a>]
//char name = getmestring();
char *name = getmestring(); 
cout<<name<<endl;
if(name != NULL)
{
delete [] name; 
}
}

BadKarma wrote:

//char name = getmestring();char *name = getmestring();

namaskaaram wrote:

assigning name =NULL and then deleting it will give u an error since u r trying to delete a block that points to the null address!!!!
rather the code should be more like.....

namaskaaram wrote:

int main(){char *name = NULL; name = getmestring(); cout<<name<<endl;if(name !="NULL)delete" []="" name;}<="" blockquote="">

change it to

if(name != NULL)
delete [] name;
name=NULL

"Opinions are neither right nor wrong. I cannot change your opinion. I can, however, change what influences your opinion." - David Crow

cheers,
Alok Gupta
VC Forum Q&A :- I/ IV

"The greatest good you can do for another is not just to share your riches but to reveal to him his own." - Benjamin Disraeli

extern int nVar_Subbu;

int nVar_Subbu = 1;

Return Values
Returns the previous progress indicator bar color, or CLR_DEFAULT if the progress indicator bar color is the default color.

COLORREF clrPrevious = (COLORREF)myCtrl.SendMessage(PBM_SETBARCOLOR, 0, RGB(255, 0, 0));