Skip to content

Commit 6ed1c64

Browse files
committed
Improve macro usage
1 parent eeeeca1 commit 6ed1c64

File tree

2 files changed

+4
-4
lines changed

2 files changed

+4
-4
lines changed

ProcessHacker/appsup.c

Lines changed: 1 addition & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -2075,7 +2075,7 @@ BOOLEAN PhShellOpenKey2(
20752075
PhUnicodeStringToStringRef(&valueName, &valueNameSr);
20762076

20772077
expandedKeyName = PhExpandKeyName(KeyName, TRUE);
2078-
NtSetValueKey(favoritesKeyHandle, &valueName, 0, REG_SZ, expandedKeyName->Buffer, (ULONG)expandedKeyName->Length + 2);
2078+
NtSetValueKey(favoritesKeyHandle, &valueName, 0, REG_SZ, expandedKeyName->Buffer, (ULONG)expandedKeyName->Length + sizeof(UNICODE_NULL));
20792079
PhDereferenceObject(expandedKeyName);
20802080

20812081
// Select our entry in regedit.

ProcessHacker/runas.c

Lines changed: 3 additions & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -246,7 +246,7 @@ PPH_STRING GetCurrentWinStaName(
246246
GetProcessWindowStation(),
247247
UOI_NAME,
248248
string->Buffer,
249-
(ULONG)string->Length + 2,
249+
(ULONG)string->Length + sizeof(UNICODE_NULL),
250250
NULL
251251
))
252252
{
@@ -1098,8 +1098,8 @@ INT_PTR CALLBACK PhpRunAsDlgProc(
10981098
createInfo.Password = PhGetStringOrEmpty(password);
10991099

11001100
// Whenever we can, try not to set the desktop name; it breaks a lot of things.
1101-
if (desktopName->Length != 0 && !PhEqualString2(desktopName, L"WinSta0\\Default", TRUE))
1102-
createInfo.DesktopName = desktopName->Buffer;
1101+
if (!PhIsNullOrEmptyString(desktopName) && !PhEqualString2(desktopName, L"WinSta0\\Default", TRUE))
1102+
createInfo.DesktopName = PhGetString(desktopName);
11031103

11041104
PhSetDesktopWinStaAccess();
11051105

0 commit comments

Comments
 (0)