下面的代码是从MS Async Filter复制的。假设下面的代码调用的是CancelIo或CancelIoEx。我不知道CancelIoEx是在哪里被调用的。假设typedef表示CancelIoEx,但从未被调用。这行bResult = (pfnCancelIoEx)(m_hFile, NULL);到底在做什么?
// Cancel: Cancels pending I/O requests.
HRESULT CFileStream::Cancel()
{
CAutoLock lock(&m_CritSec);
// Use CancelIoEx if available, otherwise use CancelIo.
typedef BOOL (*CANCELIOEXPROC)(HANDLE hFile, LPOVERLAPPED lpOverlapped);
BOOL bResult = 0;
CANCELIOEXPROC pfnCancelIoEx = NULL;
HMODULE hKernel32 = LoadLibrary(L"Kernel32.dll");
if (hKernel32){
//propably bad code !!! Take Care.
bResult = (pfnCancelIoEx)(m_hFile, NULL);
FreeLibrary(hKernel32);
}
else {
bResult = CancelIo(m_hFile);
}
if (!bResult) {
return HRESULT_FROM_WIN32(GetLastError());
}
return S_OK;
}发布于 2013-04-20 01:42:56
假设这就是所有的代码,它里面有一个严重的bug。这一点:
CANCELIOEXPROC pfnCancelIoEx = NULL;将pfnCancelIoEx定义为指向其签名与CancelIoEx的签名匹配的函数的指针。指针被初始化为NULL值,明显的意图是稍后将其指向CancelIoEx。
该函数是在Kernel32.dll中定义的,因此加载该函数是合乎逻辑的下一步。如果成功,则代码应继续执行以下操作:
pfnCancelIoEx = GetProcAddress(hKernel32, "CancelIoEx");然后它应该检查结果。然而,它不做这两件事中的任何一个。
接下来,在这一行上:
bResult = (pfnCancelIoEx)(m_hFile, NULL);它尝试调用pfnCancelIoEx指向的函数。然而,这个指针永远不会从它的初始null值改变,所以这将尝试取消引用null指针,从而导致未定义的行为,并可能导致崩溃。
https://stackoverflow.com/questions/16109681
复制相似问题