CAtlHttpClient的一个严重bug

我写的一个程序要从http服务器下载xml文件，就用了CAtlHttpClient这个http客户端类。在xml文件比较小的时候一切都顺利，
但当xml文件超过1M后，问题就时不时出现：不能下载xml文件了!
    什么原因呢？
    只能在本机调试了。为了尽快重现bug，我把xml文件增大到了3M多，下载的周期也由原来的2分钟缩短到30秒...经过20多个周期
问题又重现了：不能下载xml文件，陷在了下载中....
    在vc2003的工具栏中点击"全部中断"，打开"线程"窗口就呈现出所有的线程来。我们只关心用户线程。
    是不是死锁了？
    我挨个挨个线程的打开，然后观查调用栈，没有发现死锁，所有打开的线程都可以按F10运行....但却发现下载xml文件的线程
一直在ZEvtSyncSocket::Read()函数中(CAtlHttpClient实际是typedef CAtlHttpClientT<ZEvtSyncSocket> CAtlHttpClient)，跳
不出来。难道在Read里发生死锁了？试着按F10却是可以调试运行的，看来不是死锁。那是什么原因让它一直陷在Read中？
    先大概说一下调用栈:
    我的DownloadHttp("http地址")函数调用CAtlHttpClientT<TSocketClass>::Navigate()->......->CAtlHttpClientT<TSocketClass>::ReadBody...
    CAtlHttpClientT<TSocketClass>::ReadBody中有这样一个循环，写过tcp通信的朋友都很熟悉(//pgp是我加我注释):
    //pgp begin
    //循环的功能：不断的从网络底层缓存区中读数据，直到完成指定数量才跳出循环
    //nContentLen：xml文件的长度，从http头中获得
    //nCurrentBodyLen：已读完的xml文件长度，即当前读的进度.
    //当nCurrentBodyLen >= nContentLen时，跳出循环，即读完了xml文件的内容
    while (nCurrentBodyLen < nContentLen)
    {
dwRead = dwReadBuffSize;//pgp:缓冲区大小
//pgp:从ZEvtSyncSocket::Read()中读数据
if (!Read(readbuff, &dwRead))
    return false;
// notify user
if (m_pNavData)
{
   if (m_pNavData->pfnReadStatusCallback)
if (!m_pNavData->pfnReadStatusCallback(dwRead, m_pNavData->m_lParamRead))
return false;
}
//pgp:累加进度
nCurrentBodyLen += dwRead;
//pgp:把一次读出的数据保存起来
if (!m_current.Append((LPCSTR)(BYTE*)readbuff, dwRead))
{
    ATLASSERT(0);
    return false; // error!
}
m_pEnd = ((BYTE*)(LPCSTR)m_current) + m_current.GetLength();
    }
    
    线程就是陷在这个循环中，一直出不来。原因是，循环读了几次后，Read(readbuff, &dwRead)的输出参数dwRead每次都等于0，导致
nCurrentBodyLen += dwRead的当前进度值一直不变，始终不满足nCurrentBodyLen >= nContentLen这个结束循环的条件。
    那为什么Read返回0字节呢？返回0字节又表示什么意思呢？
    我们继续看ZEvtSyncSocket::Read()里的函数，我省略了一些与本问题无关的代码：
    //pgp
    //这是重叠模型读数据
inline bool ZEvtSyncSocket::Read(const unsigned char *pBuff, DWORD *pdwSize) 
{
        ...........
        
bool bRet = true;
WSABUF buff;
buff.buf = (char*)pBuff;
buff.len = *pdwSize;
*pdwSize = 0;
DWORD dwFlags = 0;
WSAOVERLAPPED o;
ZeroMemory(&o, sizeof(o)); // protect against re-entrency
m_csRead.Lock();
o.hEvent = m_hEventRead;
WSAResetEvent(o.hEvent);
//pgp:先投递一个读请求
if (WSARecv(m_socket, &buff, 1, pdwSize, &dwFlags, &o, 0))
{
DWORD dwLastError = WSAGetLastError();
if (dwLastError != WSA_IO_PENDING)
{
m_dwLastError = dwLastError;
bRet = false;
}
} // wait for the read to complete
if (bRet)
{
//pgp:等待Read事件的发生，等待超时为m_dwSocketTimeout，默认是10秒
if (WAIT_OBJECT_0 == WaitForSingleObject((HANDLE)o.hEvent, m_dwSocketTimeout))
{
dwFlags = 0;
//pgp:Read事件发生，调用WSAGetOverlappedResult返回一次读的字节数
//问题就在这里!!在这个函数被调用了N次后，pdwSize指向的整数值为0
//当调用WSAGetOverlappedResult后，第三个参数，即pdwSize返回0意味着什么？
//意味着远程服务器已断开连接!!可MS却当成正常读数据处理了，或说没处理第三个参数返回0的情况
//当远程服务器断开这个连接后，我们这边也应该关闭连接了(还有Event等)
if (WSAGetOverlappedResult(m_socket, &o, pdwSize, FALSE, &dwFlags))
bRet = true;
else
{
m_dwLastError = ::GetLastError();
bRet = false;
}
}
else
bRet = false;
} m_csRead.Unlock();
return bRet;
}        看代码里的解释，我们明白了出错的地方，那就很easy了。改下代码，处理下调用WSAGetOverlappedResult后，第三个参数返回0的情况:
    inline bool ZEvtSyncSocket::Read(const unsigned char *pBuff, DWORD *pdwSize)
    {
        .............
        if (WAIT_OBJECT_0 == WaitForSingleObject((HANDLE)o.hEvent, m_dwSocketTimeout))
{
dwFlags = 0;
if (WSAGetOverlappedResult(m_socket, &o, pdwSize, FALSE, &dwFlags))
{
    bRet = true;
    //pgp:如果第三个参数返回0，则bRet=false表示，这次下载失败了
    //函数栈中有函数自会清扫战场
    if(0 == *pdwSize)
        bRet = false;
}
else
{
    m_dwLastError = ::GetLastError();
    bRet = false;
}
}
else
    bRet = false;
..............
    }
    
    ok了，问题并不难。解决了，给老大有一个交代了