HidHide icon indicating copy to clipboard operation
HidHide copied to clipboard

Can't open HidHide Configuration at all. Crash on line 91 of the file "FilterDriverProxy.cpp".

Open FireLion137 opened this issue 2 years ago • 1 comments

  • [x] I have searched open and closed issues for duplicates
  • [x] I am submitting a bug report for existing functionality that does not work as intended
  • [x] I have read https://vigem.org/Community-Support/
  • [x] This isn't a feature request or a discussion topic

Bug description

Simply i can't open HidHide, i tried different versions but i always get the same error: "Something unforseen has happened which cannot be recovered from. Program execution has to be terminated. Sorry for the inconvenience." I understand that's a problem that only I have, but i dont know ho to resolve it.

Steps to reproduce

  • Install HidHide.
  • Try to open it.

Actual result: "Something unforseen has happened which cannot be recovered from. Program execution has to be terminated. Sorry for the inconvenience." Expected result: It should open normally.

Machine info

CPU-Architecture: x64 AMD Ryzen 5 2600x Windows version: Windows 10 21H2 Software/driver version(s): 1.1.50 - 1.2.98

Any other helpful information

I don't know much about programming and this sort of stuff, only basics, but i tried debugging with Visual Studio 2019 and i can tell that the Debug of HidHideClient crash after a breakpoint on line 91 of the file "FilterDriverProxy.cpp". Line 91: if (FALSE == ::DeviceIoControl(device, IOCTL_GET_WHITELIST, nullptr, 0, buffer.data(), static_cast<DWORD>(buffer.size() * sizeof(WCHAR)), &needed, nullptr)) THROW_WIN32_LAST_ERROR;

FireLion137 avatar Aug 23 '22 00:08 FireLion137

We figured it out on Discord; this can happen if the whitelist (and probably also the blacklist) has grown so much that a buffer overflow happens. In this case it happened because a 3rd party application did an insert every time it was launched instead of checking if it was already on the list. To circumvent those, the driver should ideally check the list entries for duplicates before storing them. Leaving this open as a reminder for when there's time to implement this some time in 2023 😅

nefarius avatar Aug 30 '22 11:08 nefarius

Not really fixable.

nefarius avatar Oct 30 '23 13:10 nefarius