Chromium Code Reviews| Index: chrome/browser/password_manager/password_manager_util_win.cc |
| diff --git a/chrome/browser/password_manager/password_manager_util_win.cc b/chrome/browser/password_manager/password_manager_util_win.cc |
| index 994a174dc4ee42af5f6e6264e00abb378ac69aff..d5f7f3e8db3274844e033d02b14d17d243b6d3b7 100644 |
| --- a/chrome/browser/password_manager/password_manager_util_win.cc |
| +++ b/chrome/browser/password_manager/password_manager_util_win.cc |
| @@ -129,6 +129,7 @@ bool CheckBlankPasswordWithPrefs(const WCHAR* username, |
| LOGON32_PROVIDER_DEFAULT, |
| &handle); |
| + auto last_error = GetLastError(); |
|
brucedawson
2015/09/12 00:33:55
CloseHandle could affect LastError. Unlikely, but
|
| // Win XP and later return ERROR_ACCOUNT_RESTRICTION for blank password. |
| if (logon_result) |
| CloseHandle(handle); |
| @@ -138,7 +139,7 @@ bool CheckBlankPasswordWithPrefs(const WCHAR* username, |
| // ERROR_ACCOUNT_RESTRICTION. |
| // http://msdn.microsoft.com/en-us/library/windows/desktop/ms681385 |
| blank_password = (logon_result || |
| - GetLastError() == ERROR_ACCOUNT_RESTRICTION); |
| + last_error == ERROR_ACCOUNT_RESTRICTION); |
| } |
| // Account for clock skew between pulling the password age and |