基于堆栈的缓冲区溢出

Question

运行我的代码时，出现以下错误：

GameLauncher.exe中0x00BA16A0处未处理的异常：堆栈cookie工具代码检测到基于堆栈的缓冲区溢出。

我不知道是什么原因造成的。 它是由以下代码引起的：

#include "stdafx.h"
#include <Windows.h>
#include <TlHelp32.h>
#include <iostream>

int main()
{

    std::cout << "Which process would you like to close? (Include .exe)" << std::endl;
    wchar_t userProcessToFind;
    std::wcin.getline(&userProcessToFind, 20);

    HANDLE processSnapshot;
    DWORD processID = 0;
    PROCESSENTRY32 processEntery;
    processEntery.dwSize = sizeof(PROCESSENTRY32);

    processSnapshot = CreateToolhelp32Snapshot(TH32CS_SNAPALL, processID);
    if(Process32First(processSnapshot, &processEntery) == TRUE)
    { 

        while (Process32Next(processSnapshot, &processEntery) == TRUE)
        {
            if (_wcsicmp(processEntery.szExeFile, &userProcessToFind) == 0)
            {
                HANDLE hProcess = OpenProcess(PROCESS_TERMINATE, FALSE, processEntery.th32ProcessID);

                TerminateProcess(hProcess, 0);

                CloseHandle(hProcess);
            }
        }

        CloseHandle(processSnapshot);
    }

    return 0;
}

Answer 1

在

wchar_t userProcessToFind;
std::wcin.getline(&userProcessToFind, 20);

您已经为单个wchar_t分配了空间，但是您尝试读取最多20个字符并将其放置在内存中的userProcessToFind地址userProcessToFind 。 当您尝试写入不属于&userProcessToFind内存时，这将导致堆栈损坏。 您需要做的是创建一个像

wchar_t userProcessToFind[20];
std::wcin.getline(userProcessToFind, 20);

或者您可以使用std::wstring ，您的代码将变为

std::wstring userProcessToFind;
std::getline(std::wcin, userProcessToFind);

这样做的好处是不必为进程名称使用任意大小，因为std::wstring可以缩放以适合输入。 如果需要将基础wchar_t*传递给函数，则可以使用std::wstring::c_str()来获取它。