diff options
author | Lioncash <mathew1800@gmail.com> | 2018-07-23 23:24:27 +0200 |
---|---|---|
committer | Lioncash <mathew1800@gmail.com> | 2018-07-23 23:24:29 +0200 |
commit | 2b497e58306b98eddaf74a3a1fecc6fdd8b8c855 (patch) | |
tree | ae3aff4ac8401c14df31e970d6cf4109ad710112 /src/core/loader/nro.cpp | |
parent | nro: Make constructor explicit (diff) | |
download | yuzu-2b497e58306b98eddaf74a3a1fecc6fdd8b8c855.tar yuzu-2b497e58306b98eddaf74a3a1fecc6fdd8b8c855.tar.gz yuzu-2b497e58306b98eddaf74a3a1fecc6fdd8b8c855.tar.bz2 yuzu-2b497e58306b98eddaf74a3a1fecc6fdd8b8c855.tar.lz yuzu-2b497e58306b98eddaf74a3a1fecc6fdd8b8c855.tar.xz yuzu-2b497e58306b98eddaf74a3a1fecc6fdd8b8c855.tar.zst yuzu-2b497e58306b98eddaf74a3a1fecc6fdd8b8c855.zip |
Diffstat (limited to '')
-rw-r--r-- | src/core/loader/nro.cpp | 34 |
1 files changed, 24 insertions, 10 deletions
diff --git a/src/core/loader/nro.cpp b/src/core/loader/nro.cpp index 44158655c..f6f12c31c 100644 --- a/src/core/loader/nro.cpp +++ b/src/core/loader/nro.cpp @@ -68,22 +68,27 @@ static_assert(sizeof(AssetHeader) == 0x38, "AssetHeader has incorrect size."); AppLoader_NRO::AppLoader_NRO(FileSys::VirtualFile file) : AppLoader(file) { NroHeader nro_header{}; - if (file->ReadObject(&nro_header) != sizeof(NroHeader)) + if (file->ReadObject(&nro_header) != sizeof(NroHeader)) { return; + } if (file->GetSize() >= nro_header.file_size + sizeof(AssetHeader)) { - u64 offset = nro_header.file_size; + const u64 offset = nro_header.file_size; AssetHeader asset_header{}; - if (file->ReadObject(&asset_header, offset) != sizeof(AssetHeader)) + if (file->ReadObject(&asset_header, offset) != sizeof(AssetHeader)) { return; + } - if (asset_header.format_version != 0) + if (asset_header.format_version != 0) { LOG_WARNING(Loader, "NRO Asset Header has format {}, currently supported format is 0. If " "strange glitches occur with metadata, check NRO assets.", asset_header.format_version); - if (asset_header.magic != Common::MakeMagic('A', 'S', 'E', 'T')) + } + + if (asset_header.magic != Common::MakeMagic('A', 'S', 'E', 'T')) { return; + } if (asset_header.nacp.size > 0) { nacp = std::make_unique<FileSys::NACP>(std::make_shared<FileSys::OffsetVfsFile>( @@ -130,8 +135,9 @@ bool AppLoader_NRO::LoadNro(FileSys::VirtualFile file, VAddr load_base) { // Build program image Kernel::SharedPtr<Kernel::CodeSet> codeset = Kernel::CodeSet::Create(""); std::vector<u8> program_image = file->ReadBytes(PageAlignSize(nro_header.file_size)); - if (program_image.size() != PageAlignSize(nro_header.file_size)) + if (program_image.size() != PageAlignSize(nro_header.file_size)) { return {}; + } for (std::size_t i = 0; i < nro_header.segments.size(); ++i) { codeset->segments[i].addr = nro_header.segments[i].offset; @@ -187,29 +193,37 @@ ResultStatus AppLoader_NRO::Load(Kernel::SharedPtr<Kernel::Process>& process) { } ResultStatus AppLoader_NRO::ReadIcon(std::vector<u8>& buffer) { - if (icon_data.empty()) + if (icon_data.empty()) { return ResultStatus::ErrorNotUsed; + } + buffer = icon_data; return ResultStatus::Success; } ResultStatus AppLoader_NRO::ReadProgramId(u64& out_program_id) { - if (nacp == nullptr) + if (nacp == nullptr) { return ResultStatus::ErrorNotUsed; + } + out_program_id = nacp->GetTitleId(); return ResultStatus::Success; } ResultStatus AppLoader_NRO::ReadRomFS(FileSys::VirtualFile& dir) { - if (romfs == nullptr) + if (romfs == nullptr) { return ResultStatus::ErrorNotUsed; + } + dir = romfs; return ResultStatus::Success; } ResultStatus AppLoader_NRO::ReadTitle(std::string& title) { - if (nacp == nullptr) + if (nacp == nullptr) { return ResultStatus::ErrorNotUsed; + } + title = nacp->GetApplicationName(); return ResultStatus::Success; } |