CAsyncSocket problem

[Kazz]

Hi, I just started getting into C++/MFC and I've hit my first snag. I'm making a simple program that downloads the contents of a webpage and then sorts out href's. The problem comes into play in the OnReceive event handler. I'm getting 'The instruction at 0x77f475cc referenced by 0x0000000. The memory could not be "written"' right after I get the first packet. I suppose I'm doing something wrong with a pointer? Null maybe? It ONLY does this when I compile for release, it works perfectly when I compile for debug. After changing around some compiler options (turning off optimizations) it still does the same thing. I've done step by step debugging and the last thing I'm seeing is 'Receiving HTML'. I've looked over it for a long time and even showed it to a few friends, they couldn't figure it out either. My guess is it's the pBuf pointer but I don't see why, if anybody could give me some tips I would appreciate it! Here's my code.. void CReconDlg::OnReceive() { char *pBuf = new char[1024]; CString strRecvd, strConlen = "Content-Length:", actualLen; char *pConlen; int wholePacket, int result, int lenLen, int endPos, int iBufSize = 1024, int iRcvd, int packetStart; // get the data iRcvd = m_sConSocket.Receive(pBuf, iBufSize); // strip any garbage off of the buffer pBuf[iRcvd] = NULL; // error trap if (iRcvd == SOCKET_ERROR){ MessageBox("Error on receiving socket"); }else{ // debug step and append buffer to CString for manipulation m_sStat = "Receiving HTML.."; strRecvd += pBuf; } // search for the Content-Length so we know just how much data we're getting pConlen = strstr(strRecvd, strConlen); // if it's found parse out the Content-Length: header feild and append everything AFTER the header to a new string if (pConlen != NULL){ result = (int)(pConlen - strRecvd + 1) + 15; endPos = strRecvd.Find("\n", result) - 1; lenLen = (endPos - result); actualLen = strRecvd.Left(endPos); realLen = actualLen.Right(lenLen); packetStart = strRecvd.Find("\r\n\r\n", result); int packetLen1 = (strlen(strRecvd) - packetStart); m_strIn += strRecvd.Right(packetLen1); }else{ // if there's no header just throw the entire string onto our 'raw packet' m_strIn += strRecvd; } // get lengths wholePacket = strlen(m_strIn) - 4; CString display ; display.Format("%d", wholePacket); // have we got all of it? if so continue to parse.. if (rea

[Michael Dunn]

First thing I noticed is that this: pBuf[iRcvd] = NULL; is a buffer overrun. What happens if iRcvd is 1024? You write a 0 to one byte past the end of the array, overwriting whatever is on the stack at that point. --Mike-- The Internet is a place where absolutely nothing happens.   -- Strong Bad 1ClickPicGrabber - Grab & organize pictures from your favorite web pages, with 1 click! My really out-of-date homepage Sonork-100.19012 Acid_Helm