From 8b348b3dd373c84cf8897c3c8a5c58e25dcf3300 Mon Sep 17 00:00:00 2001 From: Yaakov Date: Fri, 22 May 2020 14:33:01 +1000 Subject: [PATCH] dont crash if virtualallocex fails --- Notepad.Extensions.Logging/NotepadLogger.cs | 15 +++++++++------ 1 file changed, 9 insertions(+), 6 deletions(-) diff --git a/Notepad.Extensions.Logging/NotepadLogger.cs b/Notepad.Extensions.Logging/NotepadLogger.cs index f5552e5..7679e7b 100644 --- a/Notepad.Extensions.Logging/NotepadLogger.cs +++ b/Notepad.Extensions.Logging/NotepadLogger.cs @@ -97,10 +97,10 @@ namespace Notepad.Extensions.Logging break; case WindowKind.NotepadPlusPlus: - { - WriteToNotepadPlusPlus(hwnd, message); - break; - } + { + WriteToNotepadPlusPlus(hwnd, message); + break; + } } } @@ -114,10 +114,13 @@ namespace Notepad.Extensions.Logging // Look away now, before its too late. // - var threadID = GetWindowThreadProcessId(hwnd, out var remoteProcessId); + /* unused thread ID */ _ = GetWindowThreadProcessId(hwnd, out var remoteProcessId); using var remoteProcess = Process.GetProcessById(remoteProcessId); var mem = VirtualAllocEx(remoteProcess.Handle, IntPtr.Zero, (IntPtr)dataLength, MEM_COMMIT | MEM_RESERVE, PAGE_READWRITE); - if (mem == IntPtr.Zero) throw new Win32Exception(); + if (mem == IntPtr.Zero) + { + return; + } try {