Revise FsAccess handling in MetaProcess.

This commit is contained in:
jakcron 2020-03-15 13:12:19 +08:00
parent fbe0b2f33b
commit d8595d8957
2 changed files with 30 additions and 21 deletions

@ -1 +1 @@
Subproject commit e6a139c5c04ac132ee53f61bd50e6b2e3be12ad6 Subproject commit afc6f795fe8b3e347a5dbfc75e2e39bb3a8805e0

View file

@ -114,19 +114,13 @@ void MetaProcess::validateAciFromAcid(const nn::hac::AccessControlInfo& aci, con
std::cout << "[WARNING] ACI ProgramId: FAIL (Outside Legal Range)" << std::endl; std::cout << "[WARNING] ACI ProgramId: FAIL (Outside Legal Range)" << std::endl;
} }
for (size_t i = 0; i < aci.getFileSystemAccessControl().getFsaRightsList().size(); i++) auto fs_access = aci.getFileSystemAccessControl().getFsAccess();
auto desc_fs_access = acid.getFileSystemAccessControl().getFsAccess();
for (size_t i = 0; i < fs_access.size(); i++)
{ {
bool fsaRightFound = false; if (fs_access.test(i) && desc_fs_access.test(i) == false)
for (size_t j = 0; j < acid.getFileSystemAccessControl().getFsaRightsList().size() && fsaRightFound == false; j++)
{ {
if (aci.getFileSystemAccessControl().getFsaRightsList()[i] == acid.getFileSystemAccessControl().getFsaRightsList()[j]) std::cout << "[WARNING] ACI/FAC FsaRights: FAIL (" << nn::hac::FileSystemAccessUtil::getFsAccessFlagAsString(nn::hac::fac::FsAccessFlag(i)) << " not permitted)" << std::endl;
fsaRightFound = true;
}
if (fsaRightFound == false)
{
std::cout << "[WARNING] ACI/FAC FsaRights: FAIL (" << nn::hac::FileSystemAccessUtil::getFsaRightAsString(aci.getFileSystemAccessControl().getFsaRightsList()[i]) << " not permitted)" << std::endl;
} }
} }
@ -337,23 +331,38 @@ void MetaProcess::displayFac(const nn::hac::FileSystemAccessControl& fac)
std::cout << "[FS Access Control]" << std::endl; std::cout << "[FS Access Control]" << std::endl;
std::cout << " Format Version: " << std::dec << (uint32_t)fac.getFormatVersion() << std::endl; std::cout << " Format Version: " << std::dec << (uint32_t)fac.getFormatVersion() << std::endl;
if (fac.getFsaRightsList().size()) auto fs_access = fac.getFsAccess();
if (fs_access.any())
{ {
std::cout << " FS Rights:" << std::endl; std::cout << " FsAccess:" << std::endl;
for (size_t i = 0; i < fac.getFsaRightsList().size(); i++) // TODO this formatting loop could be a utils function
for (size_t flag = 0, printed = 0; flag < fs_access.size(); flag++)
{ {
if (i % 10 == 0) // skip unset flags
if (fs_access.test(flag) == false)
continue;
// format the strings
// for each 10 printed we do a new line
if (printed % 10 == 0)
{ {
if (i != 0) // skip new line when we haven't printed anything
if (printed != 0)
std::cout << std::endl; std::cout << std::endl;
std::cout << " "; std::cout << " ";
} }
std::cout << nn::hac::FileSystemAccessUtil::getFsaRightAsString(fac.getFsaRightsList()[i]); // within a line we want to separate the next string from the last one with a comma and a space
if (_HAS_BIT(mCliOutputMode, OUTPUT_EXTENDED)) else
std::cout << " (bit " << std::dec << (uint32_t)fac.getFsaRightsList()[i] << ")"; {
if (fac.getFsaRightsList()[i] != fac.getFsaRightsList().atBack())
std::cout << ", "; std::cout << ", ";
} }
printed++;
// output string info
std::cout << nn::hac::FileSystemAccessUtil::getFsAccessFlagAsString(nn::hac::fac::FsAccessFlag(flag));
if (_HAS_BIT(mCliOutputMode, OUTPUT_EXTENDED))
std::cout << " (bit " << std::dec << (uint32_t)flag << ")";
}
std::cout << std::endl; std::cout << std::endl;
} }