ITADN

Unnecessary <WinSock2.h> include in hkMemoryRouter causing community false alarms

#133Openmmmr55 创建于 2026-03-12
M
mmmr55commented
<img width="1815" height="1202" alt="Image" src="https://github.com/user-attachments/assets/cad2978c-8de2-48e2-a63f-ab800478322d" /> <img width="1495" height="663" alt="Image" src="https://github.com/user-attachments/assets/474bcba2-88b0-4e57-80f0-62ff77f8c18a" /> Hi, thanks for the great work on this library. I noticed that in `hkMemoryRouter` (header/cpp), there is an `#include <WinSock2.h>`. It appears that this include is completely unused for the actual Havok memory routing logic, as `TlsGetValue` and the `DWORD` type only require `processthreadsapi.h` (or the standard `<windows.h>`). **The Issue:** Because this networking header is included in the compiled DLLs of mods using this template, some hardcore players and community members are reverse-engineering or scanning the DLLs and falsely accusing mod authors of sneaking in telemetry, spyware, or network backdoors into a single-player game. This is currently causing a lot of unnecessary drama in the modding community. **Suggestion:** Could we please remove the `#include <WinSock2.h>` if it is indeed redundant? This would greatly help downstream modders avoid baseless paranoia from players. I've attached a screenshot for reference. Thanks!
3 条评论