0

我们的应用程序浏览命名管道列表并查找由我们的应用程序创建的命名管道。

如果我们的命名管道不存在，我们继续创建一个。然而，最近我们的客户报告说

应用程序崩溃在线：

fileName = std::wstring(TmpInfo->FileName);

您可以查看下面随附的 windbg 故障转储报告。崩溃是零星的，并非始终可重现。你能帮我找出问题吗？转换TmpInfo->FileName成wstring. 请注意，该组件是一个 ActiveX 控件。我们的 ActiveX 正在使其他应用程序崩溃。

以下代码负责枚举命名管道。

typedef struct {
        ULONG                   NextEntryOffset;
        ULONG                   FileIndex;
        ULONG                   FileNameLength;
        WCHAR                   FileName[1];    
    } FILE_NAMES_INFORMATION, *PFILE_NAMES_INFORMATION;


inline void EnumerateRunningPipes(std::vector<std::wstring>& pipeNames, std::wstring stringLookup,      

                                     bool useStringLookup, bool 

truncatePipeDirPrefix)
    {
        LONG ntStatus;   
        IO_STATUS_BLOCK IoStatus;       
        BOOL bReset = TRUE;       
        PFILE_NAMES_INFORMATION fileNameInfo, TmpInfo;              

        fileNameInfo = (PFILE_NAMES_INFORMATION) new BYTE[1024];

        HANDLE hPipe = CreateFile("//./pipe/", 
            GENERIC_READ,
            FILE_SHARE_READ | FILE_SHARE_WRITE | FILE_SHARE_DELETE,
            NULL,
            OPEN_EXISTING,
            0,
            NULL);

        if(hPipe == INVALID_HANDLE_VALUE)
            throw CEnumeratePipeException("Could not get handle to root directory for pipes.",
            CEnumeratePipeException::ErrorType::CREATE_FILE, GetLastError());       

        USES_CONVERSION;

        locale loc;
        mbstate_t state = {0};

        while(true)
        {
            memset(fileNameInfo, 0, 1024);
            ntStatus = NtQueryDirectoryFile( hPipe, NULL, NULL, NULL, &IoStatus, fileNameInfo, 

1024,
                FileNameInformation, FALSE, NULL, bReset );

            if (ntStatus!=NO_ERROR)   
            {   
                if (ntStatus == STATUS_NO_MORE_FILES)                       


                    break;              

                std::stringstream sstream;      
                sstream << "NtQueryDirectoryFile error " << ntStatus;

                logger->writeLog(sstream.str()); 
                throw CEnumeratePipeException(
                    "NtQueryDirectoryFile could not get information about the directory or 

file.", 
                    CEnumeratePipeException::ErrorType::QUERY_DIR, ntStatus);
            }   

            TmpInfo = fileNameInfo;                         

            while(true)   
            {           
                const int endStringAt = TmpInfo->FileNameLength / sizeof(WCHAR);        



                std::wstring fileName;

                try
                {                                       
                    fileName = std::wstring(TmpInfo->FileName);//The access violation 

occurs at this line of code.
                    fileName = fileName.substr(0, endStringAt);
                }
                catch (...)
                {
                    logger->writeLog("Caught unknown exception.");

                    if (TmpInfo->FileName == NULL)
                    {
                        logger->writeLog("TmpInfo->FileName = NULL");           


                    }                   
                }

                if (useStringLookup)
                {               
                    if (fileName.find(stringLookup) != std::wstring::npos)
                    {       
                        if (!truncatePipeDirPrefix)
                            fileName.insert(0, L"\\\\.\\pipe\\");
                        pipeNames.push_back(fileName);  
                    }   
                }
                else
                {
                    pipeNames.push_back(fileName);
                }

                if(TmpInfo->NextEntryOffset==0)
                    break;   

                TmpInfo = (PFILE_NAMES_INFORMATION)((DWORD)TmpInfo+TmpInfo->NextEntryOffset);
            }   

            bReset = FALSE;
        }   

        delete fileNameInfo;
    }

以下是故障转储报告的一部分：

FAULTING_IP：
msvcr100!memcpy+57 [f:\dd\vctools\crt_bld\SELF_X86\crt\src\INTEL\memcpy.asm @ 185]
78aa1ed7 f3a5 rep movs dword ptr es:[edi],dword ptr [esi]

EXCEPTION_RECORD: ffffffff -- (.exr 0xffffffffffffffff)
异常地址：78aa1ed7 (msvcr100!memcpy+0x00000057)
异常代码：c0000005（访问冲突）
异常标志：00000000
数量参数：2
参数[0]：00000000
参数[1]：0d6b8000
尝试从地址 0d6b8000 读取

SYMBOL_STACK_INDEX：0

SYMBOL_NAME: msvcr100!memcpy+57

FOLLOWUP_NAME：机器所有者

模块名称：msvcr100

IMAGE_NAME：msvcr100.dll

DEBUG_FLR_IMAGE_TIMESTAMP：4ba1dbbe

FAILURE_BUCKET_ID：STRING_DEREFERENCE_c0000005_msvcr100.dll!memcpy

桶 ID：

APPLICATION_FAULT_STRING_DEREFERENCE_CODE_ADDRESS_MISMATCH_INVALID_POINTER_READ_WRONG_SYMBOLS_msvcr100!memcpy

+57
4

2 回答 2

2

我在文档中没有看到任何FILE_NAMES_INFORMATION说明该FileName成员以空值终止的内容。并且该FileNameLength成员的存在表明它不是空终止的。

所以看起来很可能

fileName = std::wstring(TmpInfo->FileName);

正在读取TmpInfo缓冲区的末尾，直到遇到两个连续的空字节。如果在到达不可读的虚拟内存区域之前没有遇到空字节，您将遇到访问冲突。

我建议替换这两行代码：

fileName = std::wstring(TmpInfo->FileName);
fileName = fileName.substr(0, endStringAt);

有了这个：

fileName = std::wstring(TmpInfo->FileName, TmpInfo->FileNameLength);

该构造函数只会FileNameLength从缓冲区中读取字符，因此它不应该崩溃。而且效率更高！

于 2012-05-14T14:22:14.000 回答
1

TmpInfo 和 TmpInfo->FileName 是如何定义的？TmpInfo->FileName 崩溃时是否有示例值？

我的猜测是，在某些情况下它不会被 null 终止，导致std::wstring' 构造函数读取到不属于它的内存中。

于 2012-05-14T14:20:37.770 回答