flodavid/suyu
1
0
Fork 0
forked from suyu/suyu
Code Pull requests Activity

service/acc: Correct error case within TrySelectUserWithoutInteraction()

Browse source 
empty() in this case will always return false, since the returned
container is a std::array. Instead, check if all given users are invalid
before returning the error code.
This commit is contained in:
Lioncash 2018-11-14 17:40:47 -05:00
parent e1ea8cc721
commit cd47af8af0
1 changed files with 4 additions and 2 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

6
src/core/hle/service/acc/acc.cpp
View file

@ -252,8 +252,10 @@ void Module::Interface::TrySelectUserWithoutInteraction(Kernel::HLERequestContex
rb.PushRaw<u128>(INVALID_UUID); rb.PushRaw<u128>(INVALID_UUID);
return; return;
} }
auto user_list = profile_manager->GetAllUsers();
if (user_list.empty()) { const auto user_list = profile_manager->GetAllUsers();
if (std::all_of(user_list.begin(), user_list.end(),
[](const auto& user) { return user.uuid == INVALID_UUID; })) {
rb.Push(ResultCode(-1)); // TODO(ogniK): Find the correct error code rb.Push(ResultCode(-1)); // TODO(ogniK): Find the correct error code
rb.PushRaw<u128>(INVALID_UUID); rb.PushRaw<u128>(INVALID_UUID);
return; return;