C / C++ / MFC

class GUIQueue
{
public:
    char *data;
};
 
void main( void )
{
    GUIQueue *newMsg = (GUIQueue *) malloc(sizeof(GUIQueue));
    
    newMsg->data = (char *) malloc(15 * sizeof(char)); // room for 15 chars
    
    if (newMsg->data != NULL)
    {
        free(newMsg->data);
        newMsg->data = NULL;
    }
 
    if (newMsg != NULL)
    {
        free(newMsg);
        newMsg = NULL;
    }
}

"Take only what you need and leave the land as you found it." - Native American Proverb

"Take only what you need and leave the land as you found it." - Native American Proverb

Jader89 wrote:

Any ideas why?

"Take only what you need and leave the land as you found it." - Native American Proverb

struct GUIQueue<br />
{<br />
	bool isBmp;<br />
	int size;<br />
	char* data;<br />
};

GUIQueue* newMsg = (GUIQueue*)malloc( sizeof(GUIQueue) );<br />
memset( newMsg, 0, sizeof(GUIQueue) );<br />
newMsg->data = (char*)malloc( sizeof(msg.u.reqvid) );<br />
memcpy( newMsg->data, &msg, sizeof(msg.u.reqvid) + sizeof( MsgHeader ) );<br />
newMsg->size = sizeof(msg.u.reqvid) + sizeof(MsgHeader);<br />
newMsg->isBmp = true;<br />
GUIQ.push( newMsg );<br />
<br />
GUIQueue* test = (GUIQueue*)GUIQ.front();<br />
GUIQ.pop();<br />
if( test->data != NULL )<br />
{<br />
	free(test->data);<br />
	test->data = NULL;<br />
}<br />
if( test != NULL )<br />
{<br />
	free(test);<br />
	test = NULL;<br />
}

DavidCrow wrote:

newMsg->data = (char*)malloc( sizeof(msg.u.reqvid) );

/// assuming msg.u.reqvid is the length of the data..<br />
newMsg->data = (char*)malloc( msg.u.reqvid * sizeof(char ) ); 

"Take only what you need and leave the land as you found it." - Native American Proverb

"Take only what you need and leave the land as you found it." - Native American Proverb