C / C++ / MFC

Stephen Hewitt17-Nov-10 0:55

Stephen Hewitt

17-Nov-10 0:55

The code you showed (CSMTP::get_response) isn't in the stack you gave!

Steve

Cedric Moonen17-Nov-10 0:57

17-Nov-10 0:57

Please, don't repost your question. This is considered rude.
Besides, you have been given advice to use your debugger, did you already tried that ? If you are in a hurry, why don't you try to solve the problem instead of waiting that people give you the complete solution.
We've been giving you everything that you need to fix your problem. It seems you are not really willing to learn as your username would suggest it...

Cédric Moonen
Software developer

Charting control [v3.0]
OpenGL game tutorial in C++

Le@rner17-Nov-10 1:00

17-Nov-10 1:00

ok sir

actully i m not find out the exact problem by call stack window.

i dont know how to use it.

so...

Cedric Moonen17-Nov-10 1:06

17-Nov-10 1:06

When the crash occurs, you can see the call stack window and double click on lines in the call stack. This will bring you to the line in question. Now, try to locate the crash somewhere in your code and put breakpoints (F9) before the line which crashes. Then you can step into the code and inspect the variables which are causing problems (here it seems it is a CString which seems corrupted). For instance you can verify that the strings are properly formated and valid pointers.
You can't expect doing any serious software development without using a debugger. When there's a crash like that, your first reflex is to first try to see which information you can extract with the debugger. Then, if you still didn't fix your problem and are still stuck, you can come and ask a question by providing the information from the debugger.

Cédric Moonen
Software developer

Charting control [v3.0]
OpenGL game tutorial in C++

CPallini17-Nov-10 1:21

CPallini

17-Nov-10 1:21

Why are you returning the dead (i.e. out-of scope) variable buf? Smile | :)

If the Lord God Almighty had consulted me before embarking upon the Creation, I would have recommended something simpler.
-- Alfonso the Wise, 13th Century King of Castile.

This is going on my arrogant assumptions. You may have a superb reason why I'm completely wrong.
-- Iain Clarke

[My articles]

Code-o-mat17-Nov-10 2:08

Code-o-mat

17-Nov-10 2:08

CPallini wrote:
returning the dead

sounds like a horror movie from the "old times", "Return of the dead variable", now in color! You could say it's a coding horror movie. Smile | :)

> The problem with computers is that they do what you tell them to do and not what you want them to do. <
> Leela: Fry, you're wasting your life sitting in front of that TV. You need to get out and see the real world.
Fry: But this is HDTV. It's got better resolution than the real world <

CPallini17-Nov-10 2:16

CPallini

17-Nov-10 2:16

Code-o-mat wrote:
sounds like a horror movie from the "old times", "Return of the dead variable", now in color! You could say it's a coding horror movie

More like a 'code horror' of the new generations... Roll eyes | :rolleyes:

Re: Please tell me whats wrong with function? [modified]

Code-o-mat17-Nov-10 2:21

Code-o-mat

17-Nov-10 2:21

Tam dam daaaam!!!

Alain Rist17-Nov-10 3:50

17-Nov-10 3:50

His function signature is CString CSMTP::get_response() so the return statement will construct a CString with buf and return it. The problem is not in that construct, it was that buf was not null terminated as expected by the constructor. See previous thread for more Smile | :)

Edit: Just seen that the OP code now initializes buf to empty, so the problem is elsewhere, probably around a call to CMemFile::Free().

cheers,
AR

When the wise (person) points at the moon the fool looks at the finger (Chinese proverb)

modified on Wednesday, November 17, 2010 11:02 AM

CPallini17-Nov-10 5:26

CPallini

17-Nov-10 5:26

Yep, you're right (I've overlooked the signature).
Smile | :)

super17-Nov-10 2:23

super

17-Nov-10 2:23

Hi,
Please post and introduce yourself in General Indian forum[^]
In GIT,we would be able to guide with basic guidelines about posting in Codeprojects.

cheers,
Super
------------------------------------------
Too much of good is bad,mix some evil in it

David Crow17-Nov-10 3:15

David Crow

17-Nov-10 3:15

Why not set a breakpoint on the first statement in this function and then use the debugger to single-step through the code?

"One man's wage rise is another man's price increase." - Harold Wilson

"Fireproof doesn't mean the fire will never come. It means when the fire comes that you will be able to withstand it." - Michael Simmons

"Man who follows car will be exhausted." - Confucius

#ifndef

vidhyaej16-Nov-10 20:21

vidhyaej

16-Nov-10 20:21

hi friends,
i am a beginner in C++.
Can any one give me an idea about the use of #ifndef and #define ?

Re: #ifndef

Alain Rist16-Nov-10 21:04

How can handle this exception?

16-Nov-10 21:04

http://www.cplusplus.com/doc/tutorial/preprocessor/[^]
cheers,
AR

When the wise (person) points at the moon the fool looks at the finger (Chinese proverb)

Re: #ifndef

David Crow17-Nov-10 3:16

David Crow

17-Nov-10 3:16

See here.

Le@rner16-Nov-10 19:00

16-Nov-10 19:00

Hi all,

in my application this execption occur please describe me how can i thow or handle this.

and what is the type of this execption like,memory,File etc.

First-chance exception at 0x0126a8a4 in test.exe: 0xC0000005: Access violation reading location 0xfeeefeee.
Unhandled exception at 0x0126a8a4 in test.exe: 0xC0000005: Access violation reading location 0xfeeefeee.

please help me for this.

Alain Rist16-Nov-10 21:00

Re: How can handle this exception? [modified]

16-Nov-10 21:00

Hi,

Le@rner wrote:
First-chance exception at 0x0126a8a4 in test.exe: 0xC0000005: Access violation reading location 0xfeeefeee

Your code loaded at 0x0126a8a4 attempts to read memory through an unitialized pointer.
When it happens, use the call stack in your debugger to find your faulty code and fix it Smile | :)

cheers,
AR

When the wise (person) points at the moon the fool looks at the finger (Chinese proverb)

Le@rner16-Nov-10 21:06

16-Nov-10 21:06

in diffrent condition exception is occur call stack of all condition mentioned here

condition 1:

	wininet.dll!761cf6ea() 	
 	[Frames below may be incorrect and/or missing, no symbols loaded for wininet.dll]	
 	wininet.dll!7620ecc0() 	
 	wininet.dll!761c9986() 	
 	wininet.dll!761df27b() 	
 	wininet.dll!761df0f1() 	
 	wininet.dll!761c57b3() 	
 	wininet.dll!761c5c59() 	
 	wininet.dll!761df4c3() 	
>	Test.exe!CAmHttpSocket::OpenUrl(const char * url=0x23b38df0)  Line 225 + 0x17 bytes	C++
 	Test.exe!CAmHttpSocket::GetHeaders(const char * url=0x23b38df0)  Line 187	C++
 	Test.exe!CHttp_Mon_Thread::HTTPResponseThread()  Line 58 + 0xe bytes	C++
 	Test.exe!CHttp_Mon_Thread::OnTimer(unsigned int idEvent=15901, long dwTime=10345141)  Line 140 + 0x7 bytes	C++
 	Test.exe!CWinThread::DispatchThreadMessageEx(tagMSG * pMsg=0x37f106c0)  Line 771	C++
 	Test.exe!AfxInternalPreTranslateMessage(tagMSG * pMsg=0x37f106c0)  Line 227 + 0xd bytes	C++
 	Test.exe!CWinThread::PreTranslateMessage(tagMSG * pMsg=0x37f106c0)  Line 777 + 0x8 bytes	C++
 	Test.exe!AfxPreTranslateMessage(tagMSG * pMsg=0x37f106c0)  Line 255	C++
 	Test.exe!AfxInternalPumpMessage()  Line 178 + 0xf bytes	C++
 	Test.exe!CWinThread::Run()  Line 629 + 0x7 bytes	C++
 	Test.exe!_AfxThreadEntry(void * pParam=0x0017f00c)  Line 126	C++
 	Test.exe!_callthreadstartex()  Line 348 + 0x6 bytes	C
 	Test.exe!_threadstartex(void * ptd=0x23adbb70)  Line 326 + 0x5 bytes	C
 	kernel32.dll!7662eccb() 	
 	ntdll.dll!778dd24d() 	
 	ntdll.dll!778dd45f()

condition 2:

>	Test.exe!ATL::CStringData::Release()  Line 111 + 0x7 bytes	C++
 	Test.exe!ATL::CSimpleStringT<char,0>::operator=(const ATL::CSimpleStringT<char,0> & strSrc="10022*Socket Error")  Line 300	C++
 	Test.exe!CSmtp_Mon_Thread::SmtpResponseThread(ATL::CStringT<char,StrTraitMFC<char,ATL::ChTraitsCRT<char> > > URL_Name="http://exby.org", unsigned int Port=1079503120)  Line 141 + 0x16 bytes	C++
 	Test.exe!CSmtp_Mon_Thread::OnSmtpTimer(unsigned int idEvent=17291, long dwTime=10345141)  Line 69	C++
 	Test.exe!CWinThread::DispatchThreadMessageEx(tagMSG * pMsg=0x14e5cd50)  Line 771	C++
 	Test.exe!AfxInternalPreTranslateMessage(tagMSG * pMsg=0x14e5cd50)  Line 227 + 0xd bytes	C++
 	Test.exe!CWinThread::PreTranslateMessage(tagMSG * pMsg=0x14e5cd50)  Line 777 + 0x8 bytes	C++
 	Test.exe!AfxPreTranslateMessage(tagMSG * pMsg=0x14e5cd50)  Line 255	C++
 	Test.exe!AfxInternalPumpMessage()  Line 178 + 0xf bytes	C++
 	Test.exe!CWinThread::Run()  Line 629 + 0x7 bytes	C++
 	Test.exe!_AfxThreadEntry(void * pParam=0x003af3a4)  Line 126	C++
 	Test.exe!_callthreadstartex()  Line 348 + 0x6 bytes	C
 	Test.exe!_threadstartex(void * ptd=0x21f742d8)  Line 326 + 0x5 bytes	C
 	kernel32.dll!7662eccb() 	
 	[Frames below may be incorrect and/or missing, no symbols loaded for kernel32.dll]	
 	ntdll.dll!778dd24d() 	
 	ntdll.dll!778dd45f()

condition 3:

>	Test.exe!ATL::CStringData::Release()  Line 111 + 0x7 bytes	C++
 	Test.exe!CErrorNotifications::~CErrorNotifications()  Line 52 + 0x3e bytes	C++
 	Test.exe!CErrorNotifications::`scalar deleting destructor'()  + 0x8 bytes	C++
 	Test.exe!CWinThread::Delete()  Line 591	C++
 	Test.exe!AfxEndThread(unsigned int nExitCode=0, int bDelete=1)  Line 381	C++
 	Test.exe!CErrorNotifications::ErrorNotificationThread()  Line 182	C++
 	Test.exe!CErrorNotifications::InitInstance()  Line 59	C++
 	Test.exe!_AfxThreadEntry(void * pParam=0x4458f9a4)  Line 113 + 0x7 bytes	C++
 	Test.exe!_callthreadstartex()  Line 348 + 0x6 bytes	C
 	Test.exe!_threadstartex(void * ptd=0x22175248)  Line 326 + 0x5 bytes	C
 	kernel32.dll!7662eccb() 	
 	[Frames below may be incorrect and/or missing, no symbols loaded for kernel32.dll]	
 	ntdll.dll!778dd24d() 	
 	ntdll.dll!778dd45f()

now please help me

modified on Wednesday, November 17, 2010 4:31 AM

Cedric Moonen16-Nov-10 21:17

16-Nov-10 21:17

Use your debugger to inspect the problem more in detail. Step into the code to inspect the different variables.

Cédric Moonen
Software developer

Charting control [v3.0]
OpenGL game tutorial in C++

Alain Rist16-Nov-10 21:23

16-Nov-10 21:23

Le@rner wrote:
now please help me

Follow your code in the debugger ahead of the place where you generate a socket error with "http://exby.org".

When the wise (person) points at the moon the fool looks at the finger (Chinese proverb)

Le@rner16-Nov-10 22:41

16-Nov-10 22:41

its generate using this line

CSMTP smtp;

StatusTh  = smtp.get_response();

//where
CString CSMTP::get_response()
{
	try
	{
		char buf[240];
		sock_error=WSAGetLastError();
		if(sock_error!=0)
		{
			if(sock_error==10048)
				sock_str = "10048*Address already in use";
			else if(sock_error==10049)
				sock_str = "10049*Cannot assign requested address";
			else if(sock_error==10050)
				sock_str = "10050*Network is down";
			else if(sock_error==10051)
				sock_str = "10051*Network is unreachable";
			else if(sock_error==10060)
				sock_str = "10060*Connection timed out";
			else if(sock_error==10064)
				sock_str = "10064*Host is down";
			else if(sock_error==10061)
				sock_str = "10061*Connection refused";
			else if(sock_error==10054)
				sock_str = "10054*Connection reset by peer";
			else
			{
				sock_str.Format("%d",sock_error);
				sock_str = sock_str + "*Socket Error";
			}
			return sock_str;
		}
		recv(sock,buf,sizeof(buf),0);
		sock_error=WSAGetLastError();
		if(sock_error!=0)
		{
			if(sock_error==10048)
				sock_str = "10048*Address already in use";
			else if(sock_error==10049)
				sock_str = "10049*Cannot assign requested address";
			else if(sock_error==10050)
				sock_str = "10050*Network is down";
			else if(sock_error==10051)
				sock_str = "10051*Network is unreachable";
			else if(sock_error==10060)
				sock_str = "10060*Connection timed out";
			else if(sock_error==10064)
				sock_str = "10064*Host is down";
			else if(sock_error==10061)
				sock_str = "10061*Connection refused";
			else if(sock_error==10054)
				sock_str = "10054*Connection reset by peer";
			else
			{
				sock_str.Format("%d",sock_error);
				sock_str = sock_str + "*Socket Error";
			}
			return sock_str;
		}
		else
		{ 
			CString temp = buf;
			return temp;
		}
	}
	catch( CMemoryException* e )
	{
	   // Handle the out-of-memory exception here.
	   e->Delete();
	}
	catch( CFileException* e )
	{
	   // Handle the file exceptions here.
	   e->Delete();
	}
	catch( CException* e )
	{
	   // Handle all other types of exceptions here.
	   e->Delete();
	}
}

Cedric Moonen16-Nov-10 22:51

16-Nov-10 22:51

You should really learn to use your debugger properly. You can't come post a question on the forum each time your program crashes. The debugger is a very powerfull tool and you should really start to learn how to use it, it can make you gain a lot of time. It's very difficult to help you without being able to use a debugger.

Cédric Moonen
Software developer

Charting control [v3.0]
OpenGL game tutorial in C++

Alain Rist16-Nov-10 22:55

16-Nov-10 22:55

Le@rner wrote:

else
{
    CString temp = buf;
    return temp;
}

Your problem is here. Guess why Smile | :)

When the wise (person) points at the moon the fool looks at the finger (Chinese proverb)

Le@rner16-Nov-10 23:15