From 50472a98ea9eeb6e271983e030377f4a3097f620 Mon Sep 17 00:00:00 2001 From: Akash Mozumdar Date: Fri, 18 Jan 2019 12:51:45 -0500 Subject: [PATCH] fix buffer overrun --- vnrhook/texthook.cc | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/vnrhook/texthook.cc b/vnrhook/texthook.cc index ad4a1dd..af03a11 100644 --- a/vnrhook/texthook.cc +++ b/vnrhook/texthook.cc @@ -244,7 +244,7 @@ DWORD WINAPI TextHook::Reader(LPVOID hookPtr) if (This->hp.type & USING_UNICODE) dataLen = wcslen((wchar_t*)currentAddress) * 2; else dataLen = strlen((char*)currentAddress); - + if (dataLen > PIPE_BUFFER_SIZE - 1) continue; // results in silly error msg but oh well memcpy(buffer, (void*)currentAddress, dataLen + 1); TextOutput({ GetCurrentProcessId(), This->address, 0, 0 }, buffer, dataLen); }