mirror of
https://github.com/dolphin-emu/dolphin.git
synced 2024-11-14 21:37:52 -07:00
75d032161f
Fixes https://bugs.dolphin-emu.org/issues/10654. To quote the documenation file included with the program tgctogcm: "TGC's are miniaturized .gcm images with a 32kB header. The embedded gcm contains some bogus data, namely: -FST Location (0x424 in gcm) -DOL Location (0x420 in gcm) -FST File offsets (all files are offset/spoofed by a certain amount)" Dolphin has been handling the values at 0x420 and 0x424 by simply overwriting them with a working value (just like tgctogcm does), but it has used a different approach for the file offsets in the FST. Instead of changing the offsets that are stored in the FST, Dolphin changed where the files actually are placed on the virtual disc. My hope was that this would make the loading times more accurate to how they are when running a TGC file as part of a larger disc. However, there are TGC files where we would need to move files backwards on the disc in order to do this (this is what issue 10654 is about), so the approach we have been using is flawed. This change makes Dolphin overwrite offsets in the FST instead, like tgctogcm does. Other than making Dolphin handle the affected TGC files correctly, this change also makes it so that unnecessary padding data isn't written if you use Dolphin to convert a TGC file to an ISO file. This feature is not actually implemented in Dolphin as of now, but I'm planning to add it in the near future as part of a larger feature.
123 lines
3.9 KiB
C++
123 lines
3.9 KiB
C++
// Copyright 2016 Dolphin Emulator Project
|
|
// Licensed under GPLv2+
|
|
// Refer to the license.txt file included.
|
|
|
|
#include "DiscIO/TGCBlob.h"
|
|
|
|
#include <algorithm>
|
|
#include <memory>
|
|
#include <string>
|
|
#include <type_traits>
|
|
#include <utility>
|
|
#include <vector>
|
|
|
|
#include "Common/File.h"
|
|
#include "Common/Swap.h"
|
|
|
|
namespace
|
|
{
|
|
u32 SubtractBE32(u32 minuend_be, u32 subtrahend_le)
|
|
{
|
|
return Common::swap32(Common::swap32(minuend_be) - subtrahend_le);
|
|
}
|
|
|
|
void Replace(u64 offset, u64 size, u8* out_ptr, u64 replace_offset, u64 replace_size,
|
|
const u8* replace_ptr)
|
|
{
|
|
const u64 replace_start = std::max(offset, replace_offset);
|
|
const u64 replace_end = std::min(offset + size, replace_offset + replace_size);
|
|
|
|
if (replace_end > replace_start)
|
|
{
|
|
std::copy(replace_ptr + (replace_start - replace_offset),
|
|
replace_ptr + (replace_end - replace_offset), out_ptr + (replace_start - offset));
|
|
}
|
|
}
|
|
|
|
template <typename T>
|
|
void Replace(u64 offset, u64 size, u8* out_ptr, u64 replace_offset, const T& replace_value)
|
|
{
|
|
static_assert(std::is_trivially_copyable_v<T>);
|
|
|
|
const u8* replace_ptr = reinterpret_cast<const u8*>(&replace_value);
|
|
Replace(offset, size, out_ptr, replace_offset, sizeof(T), replace_ptr);
|
|
}
|
|
} // namespace
|
|
|
|
namespace DiscIO
|
|
{
|
|
std::unique_ptr<TGCFileReader> TGCFileReader::Create(File::IOFile file)
|
|
{
|
|
TGCHeader header;
|
|
if (file.Seek(0, SEEK_SET) && file.ReadArray(&header, 1) && header.magic == TGC_MAGIC)
|
|
return std::unique_ptr<TGCFileReader>(new TGCFileReader(std::move(file)));
|
|
|
|
return nullptr;
|
|
}
|
|
|
|
TGCFileReader::TGCFileReader(File::IOFile file) : m_file(std::move(file))
|
|
{
|
|
m_file.Seek(0, SEEK_SET);
|
|
m_file.ReadArray(&m_header, 1);
|
|
|
|
m_size = m_file.GetSize();
|
|
|
|
const u32 fst_offset = Common::swap32(m_header.fst_real_offset);
|
|
const u32 fst_size = Common::swap32(m_header.fst_size);
|
|
m_fst.resize(fst_size);
|
|
if (!m_file.Seek(fst_offset, SEEK_SET) || !m_file.ReadBytes(m_fst.data(), m_fst.size()))
|
|
m_fst.clear();
|
|
|
|
constexpr size_t FST_ENTRY_SIZE = 12;
|
|
if (m_fst.size() < FST_ENTRY_SIZE)
|
|
return;
|
|
|
|
// This calculation can overflow, but this is not a problem, because in that case
|
|
// the old_offset + file_area_shift calculation later also overflows, cancelling it out
|
|
const u32 file_area_shift = Common::swap32(m_header.file_area_real_offset) -
|
|
Common::swap32(m_header.file_area_virtual_offset) -
|
|
Common::swap32(m_header.tgc_header_size);
|
|
|
|
const size_t claimed_fst_entries = Common::swap32(m_fst.data() + 8);
|
|
const size_t fst_entries = std::min(claimed_fst_entries, m_fst.size() / FST_ENTRY_SIZE);
|
|
for (size_t i = 0; i < fst_entries; ++i)
|
|
{
|
|
// If this is a file (as opposed to a directory)...
|
|
if (m_fst[i * FST_ENTRY_SIZE] == 0)
|
|
{
|
|
// ...change its offset
|
|
const u32 old_offset = Common::swap32(m_fst.data() + i * FST_ENTRY_SIZE + 4);
|
|
const u32 new_offset = Common::swap32(old_offset + file_area_shift);
|
|
Replace<u32>(0, m_fst.size(), m_fst.data(), i * FST_ENTRY_SIZE + 4, new_offset);
|
|
}
|
|
}
|
|
}
|
|
|
|
u64 TGCFileReader::GetDataSize() const
|
|
{
|
|
return m_size - Common::swap32(m_header.tgc_header_size);
|
|
}
|
|
|
|
bool TGCFileReader::Read(u64 offset, u64 nbytes, u8* out_ptr)
|
|
{
|
|
const u32 tgc_header_size = Common::swap32(m_header.tgc_header_size);
|
|
|
|
if (m_file.Seek(offset + tgc_header_size, SEEK_SET) && m_file.ReadBytes(out_ptr, nbytes))
|
|
{
|
|
const u32 replacement_dol_offset = SubtractBE32(m_header.dol_real_offset, tgc_header_size);
|
|
const u32 replacement_fst_offset = SubtractBE32(m_header.fst_real_offset, tgc_header_size);
|
|
|
|
Replace<u32>(offset, nbytes, out_ptr, 0x0420, replacement_dol_offset);
|
|
Replace<u32>(offset, nbytes, out_ptr, 0x0424, replacement_fst_offset);
|
|
Replace(offset, nbytes, out_ptr, Common::swap32(replacement_fst_offset), m_fst.size(),
|
|
m_fst.data());
|
|
|
|
return true;
|
|
}
|
|
|
|
m_file.Clear();
|
|
return false;
|
|
}
|
|
|
|
} // namespace DiscIO
|