Visual C++ Windows服务中CSimpleString::SetLength()抛COleException排查

生产环境Windows服务抛出COleException，通过崩溃转储文件定位到异常触发点在ATL::CSimpleString::SetLength函数中：因GetData()->nAllocLength为0，但尝试设置的nLength为38，触发AtlThrow(E_INVALIDARG)。

异常触发代码片段 #

void SetLength(_In_ int nLength)
{
    ATLASSERT( nLength >= 0 );
    ATLASSERT( nLength <= GetData()->nAllocLength );

    if( nLength < 0 || nLength > GetData()->nAllocLength)
        AtlThrow(E_INVALIDARG); // 此处抛出异常，因GetData()->nAllocLength为0而nLength为38

    GetData()->nDataLength = nLength;
    m_pszData[nLength] = 0;
}

调用栈 #

• ATL::CSimpleString::SetLength()
• ATL::CSimpleString::ReleaseBufferSelLength()
• ATL::CString::FormatV()
• ATL::CString::Format()
• ActiveSocket->ErrorText.Format("Socket %d failed during read?. Error %s", ActiveSocket->Handle, (LPCSTR)Socket::SockErrorAsPrintable(ActiveSocket->Handle));

环境配置：Windows SDK 10.0、v142平台工具集、ISO C++14标准，ActiveSocket->ErrorText为CString类型。

可能的根因分析 #

1. CString内部结构损坏
   CString的内部数据（GetData()指向的结构）被内存越界、野指针写入等操作篡改，导致nAllocLength被改为0。其他模块的内存错误可能间接影响该CString的状态。

2. ActiveSocket对象无效

   • ActiveSocket可能是野指针或已被释放的对象，访问ErrorText成员会导致未定义行为，使CString内部数据处于异常状态。
   • 多线程环境下未对ActiveSocket做同步处理，一个线程释放对象的同时，另一个线程调用Format，直接破坏CString成员。

3. 格式化参数异常
   Socket::SockErrorAsPrintable返回值存在问题：

   • 返回的LPCSTR指向已释放的临时栈内存，导致Format计算字符串长度时出错，最终触发长度超出分配内存的异常。
   • 该函数返回的字符串包含特殊字符或长度计算错误，使FormatV内部计算的目标长度异常。

4. 内存分配失败
   CString::Format会先计算格式化后的字符串长度再分配内存，若内存分配失败，CString会处于未正确初始化状态，后续调用SetLength时触发异常。

排查建议 #

• 检查崩溃转储中ActiveSocket的内存地址有效性，确认ErrorText的内部字段（m_pszData、nAllocLength、nDataLength）是否符合预期。
• 审计Socket::SockErrorAsPrintable的实现，确保返回的字符串指针有效，避免返回临时栈内存的指针。
• 检查ActiveSocket的多线程访问逻辑，确认是否有互斥锁等同步机制防止并发修改/访问。
• 启用Windows**页堆（PageHeap）**工具，跟踪内存损坏问题，定位是否有其他模块的越界写入操作影响CString内部结构。

内容的提问来源于stack exchange，提问作者iMan Biglari