admin/PowerShell-PowerShell
1
0
Fork 0
mirror of https://github.com/PowerShell/PowerShell synced 2026-06-08 12:12:50 +00:00
Code Issues Packages Projects Releases Wiki Activity

Use LOGIN_NAME_MAX instead of L_cuserid

Browse Source 
Since L_cuserid is only for cuserid, not getlogin_r

Unit test for L_cuserid removed because the macro may not be available
on all platforms.
This commit is contained in:
Andrew Schwartzmeyer
2015-08-03 13:00:34 -07:00
parent cbee353d0d
commit 9426e9c455
2 changed files with 5 additions and 5 deletions
Download Patch File Download Diff File Expand all files Collapse all files
impl/getusername.cpp
+2 -2
View File
@@ -43,8 +43,8 @@ BOOL GetUserName(WCHAR_T *lpBuffer, LPDWORD lpnSize)
}
// Get username from system in a thread-safe manner
char userName[L_cuserid];
int err = getlogin_r(userName, L_cuserid);
char userName[LOGIN_NAME_MAX];
int err = getlogin_r(userName, LOGIN_NAME_MAX);
// Map errno to Win32 Error Codes
if (err != 0) {
switch (errno) {
tests/test-getusername.cpp
+3 -3
View File
@@ -114,8 +114,8 @@ TEST_F(GetUserNameTest, BufferSizeAsUserNamePlusOne) {
TestSuccess();
}
TEST_F(GetUserNameTest, L_cuseridSize) {
// L_cuserid is big enough to hold any username
TestWithSize(L_cuserid);
TEST_F(GetUserNameTest, BufferSizeAsLoginNameMax) {
// LoginNameMax is big enough to hold any username
TestWithSize(LOGIN_NAME_MAX);
TestSuccess();
}