mirrors/Atmosphere
1
0
Fork 0
mirror of https://github.com/Atmosphere-NX/Atmosphere.git synced 2024-11-22 20:06:40 +00:00
Code Issues Releases Activity

fs: update IStorage::Check functions for 14.0.0

Browse source
This commit is contained in:
Michael Scire 2022-03-25 23:29:36 -07:00
parent 5ffbed1bee
commit a4a2cc2218
9 changed files with 108 additions and 73 deletions
Show stats Download patch file Download diff file Expand all files Collapse all files

4
fusee/program/source/fs/fusee_fs_file_storage.cpp
View file

@ -34,7 +34,7 @@ namespace ams::fs {
R_TRY(this->UpdateSize()); R_TRY(this->UpdateSize());
/* Ensure our access is valid. */ /* Ensure our access is valid. */
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return ReadFile(m_handle, offset, buffer, size, fs::ReadOption()); return ReadFile(m_handle, offset, buffer, size, fs::ReadOption());
} }
@ -50,7 +50,7 @@ namespace ams::fs {
R_TRY(this->UpdateSize()); R_TRY(this->UpdateSize());
/* Ensure our access is valid. */ /* Ensure our access is valid. */
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return WriteFile(m_handle, offset, buffer, size, fs::WriteOption()); return WriteFile(m_handle, offset, buffer, size, fs::WriteOption());
} }

47
fusee/program/source/fs/fusee_fs_storage.hpp
View file

@ -31,25 +31,38 @@ namespace ams::fs {
virtual Result GetSize(s64 *out) = 0; virtual Result GetSize(s64 *out) = 0;
public: public:
static inline bool CheckAccessRange(s64 offset, s64 size, s64 total_size) { static inline Result CheckAccessRange(s64 offset, s64 size, s64 total_size) {
return offset >= 0 && R_UNLESS(offset >= 0, fs::ResultInvalidOffset());
size >= 0 && R_UNLESS(size >= 0, fs::ResultInvalidSize());
size <= total_size && R_UNLESS(util::CanAddWithoutOverflow<s64>(offset, size), fs::ResultOutOfRange());
offset <= (total_size - size); R_UNLESS(offset + size <= total_size, fs::ResultOutOfRange());
R_SUCCEED();
} }
static inline bool CheckAccessRange(s64 offset, size_t size, s64 total_size) { static ALWAYS_INLINE Result CheckAccessRange(s64 offset, size_t size, s64 total_size) {
return CheckAccessRange(offset, static_cast<s64>(size), total_size); R_RETURN(CheckAccessRange(offset, static_cast<s64>(size), total_size));
} }
static inline bool CheckOffsetAndSize(s64 offset, s64 size) { static inline Result CheckOffsetAndSize(s64 offset, s64 size) {
return offset >= 0 && R_UNLESS(offset >= 0, fs::ResultInvalidOffset());
size >= 0 && R_UNLESS(size >= 0, fs::ResultInvalidSize());
offset <= (offset + size); R_UNLESS(util::CanAddWithoutOverflow<s64>(offset, size), fs::ResultOutOfRange());
R_SUCCEED();
} }
static inline bool CheckOffsetAndSize(s64 offset, size_t size) { static ALWAYS_INLINE Result CheckOffsetAndSize(s64 offset, size_t size) {
return CheckOffsetAndSize(offset, static_cast<s64>(size)); R_RETURN(CheckOffsetAndSize(offset, static_cast<s64>(size)));
}
static inline Result CheckOffsetAndSizeWithResult(s64 offset, s64 size, Result fail_result) {
R_TRY_CATCH(CheckOffsetAndSize(offset, size)) {
R_CONVERT_ALL(fail_result);
} R_END_TRY_CATCH;
R_SUCCEED();
}
static ALWAYS_INLINE Result CheckOffsetAndSizeWithResult(s64 offset, size_t size, Result fail_result) {
R_RETURN(CheckOffsetAndSizeWithResult(offset, static_cast<s64>(size), fail_result));
} }
}; };
@ -93,8 +106,8 @@ namespace ams::fs {
R_SUCCEED_IF(size == 0); R_SUCCEED_IF(size == 0);
/* Validate arguments and read. */ /* Validate arguments and read. */
R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument()); R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument());
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return m_storage.Read(m_offset + offset, buffer, size); return m_storage.Read(m_offset + offset, buffer, size);
} }
@ -103,8 +116,8 @@ namespace ams::fs {
R_SUCCEED_IF(size == 0); R_SUCCEED_IF(size == 0);
/* Validate arguments and write. */ /* Validate arguments and write. */
R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument()); R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument());
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return m_storage.Write(m_offset + offset, buffer, size); return m_storage.Write(m_offset + offset, buffer, size);
} }

43
libraries/libstratosphere/include/stratosphere/fs/fs_istorage.hpp
View file

@ -20,7 +20,7 @@
namespace ams::fs { namespace ams::fs {
/* ACCURATE_TO_VERSION: Unknown */ /* ACCURATE_TO_VERSION: 14.3.0.0 */
class IStorage { class IStorage {
public: public:
virtual ~IStorage() { /* ... */ } virtual ~IStorage() { /* ... */ }
@ -38,28 +38,41 @@ namespace ams::fs {
virtual Result OperateRange(void *dst, size_t dst_size, OperationId op_id, s64 offset, s64 size, const void *src, size_t src_size) = 0; virtual Result OperateRange(void *dst, size_t dst_size, OperationId op_id, s64 offset, s64 size, const void *src, size_t src_size) = 0;
virtual Result OperateRange(OperationId op_id, s64 offset, s64 size) { virtual Result OperateRange(OperationId op_id, s64 offset, s64 size) {
return this->OperateRange(nullptr, 0, op_id, offset, size, nullptr, 0); R_RETURN(this->OperateRange(nullptr, 0, op_id, offset, size, nullptr, 0));
} }
public: public:
static inline bool CheckAccessRange(s64 offset, s64 size, s64 total_size) { static inline Result CheckAccessRange(s64 offset, s64 size, s64 total_size) {
return offset >= 0 && R_UNLESS(offset >= 0, fs::ResultInvalidOffset());
size >= 0 && R_UNLESS(size >= 0, fs::ResultInvalidSize());
size <= total_size && R_UNLESS(util::CanAddWithoutOverflow<s64>(offset, size), fs::ResultOutOfRange());
offset <= (total_size - size); R_UNLESS(offset + size <= total_size, fs::ResultOutOfRange());
R_SUCCEED();
} }
static inline bool CheckAccessRange(s64 offset, size_t size, s64 total_size) { static ALWAYS_INLINE Result CheckAccessRange(s64 offset, size_t size, s64 total_size) {
return CheckAccessRange(offset, static_cast<s64>(size), total_size); R_RETURN(CheckAccessRange(offset, static_cast<s64>(size), total_size));
} }
static inline bool CheckOffsetAndSize(s64 offset, s64 size) { static inline Result CheckOffsetAndSize(s64 offset, s64 size) {
return offset >= 0 && R_UNLESS(offset >= 0, fs::ResultInvalidOffset());
size >= 0 && R_UNLESS(size >= 0, fs::ResultInvalidSize());
offset <= (offset + size); R_UNLESS(util::CanAddWithoutOverflow<s64>(offset, size), fs::ResultOutOfRange());
R_SUCCEED();
} }
static inline bool CheckOffsetAndSize(s64 offset, size_t size) { static ALWAYS_INLINE Result CheckOffsetAndSize(s64 offset, size_t size) {
return CheckOffsetAndSize(offset, static_cast<s64>(size)); R_RETURN(CheckOffsetAndSize(offset, static_cast<s64>(size)));
}
static inline Result CheckOffsetAndSizeWithResult(s64 offset, s64 size, Result fail_result) {
R_TRY_CATCH(CheckOffsetAndSize(offset, size)) {
R_CONVERT_ALL(fail_result);
} R_END_TRY_CATCH;
R_SUCCEED();
}
static ALWAYS_INLINE Result CheckOffsetAndSizeWithResult(s64 offset, size_t size, Result fail_result) {
R_RETURN(CheckOffsetAndSizeWithResult(offset, static_cast<s64>(size), fail_result));
} }
}; };

4
libraries/libstratosphere/include/stratosphere/fs/fs_memory_storage.hpp
View file

@ -34,7 +34,7 @@ namespace ams::fs {
/* Validate arguments. */ /* Validate arguments. */
R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument()); R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument());
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
/* Copy from memory. */ /* Copy from memory. */
std::memcpy(buffer, m_buf + offset, size); std::memcpy(buffer, m_buf + offset, size);
@ -47,7 +47,7 @@ namespace ams::fs {
/* Validate arguments. */ /* Validate arguments. */
R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument()); R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument());
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
/* Copy to memory. */ /* Copy to memory. */
std::memcpy(m_buf + offset, buffer, size); std::memcpy(m_buf + offset, buffer, size);

30
libraries/libstratosphere/include/stratosphere/fs/fs_substorage.hpp
View file

@ -75,28 +75,28 @@ namespace ams::fs {
public: public:
virtual Result Read(s64 offset, void *buffer, size_t size) override { virtual Result Read(s64 offset, void *buffer, size_t size) override {
/* Ensure we're initialized. */ /* Ensure we're initialized. */
R_UNLESS(this->IsValid(), fs::ResultNotInitialized()); R_UNLESS(this->IsValid(), fs::ResultNotInitialized());
/* Succeed immediately on zero-sized operation. */ /* Succeed immediately on zero-sized operation. */
R_SUCCEED_IF(size == 0); R_SUCCEED_IF(size == 0);
/* Validate arguments and read. */ /* Validate arguments and read. */
R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument()); R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument());
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return m_base_storage->Read(m_offset + offset, buffer, size); return m_base_storage->Read(m_offset + offset, buffer, size);
} }
virtual Result Write(s64 offset, const void *buffer, size_t size) override{ virtual Result Write(s64 offset, const void *buffer, size_t size) override{
/* Ensure we're initialized. */ /* Ensure we're initialized. */
R_UNLESS(this->IsValid(), fs::ResultNotInitialized()); R_UNLESS(this->IsValid(), fs::ResultNotInitialized());
/* Succeed immediately on zero-sized operation. */ /* Succeed immediately on zero-sized operation. */
R_SUCCEED_IF(size == 0); R_SUCCEED_IF(size == 0);
/* Validate arguments and write. */ /* Validate arguments and write. */
R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument()); R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument());
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return m_base_storage->Write(m_offset + offset, buffer, size); return m_base_storage->Write(m_offset + offset, buffer, size);
} }
@ -109,7 +109,8 @@ namespace ams::fs {
/* Ensure we're initialized and validate arguments. */ /* Ensure we're initialized and validate arguments. */
R_UNLESS(this->IsValid(), fs::ResultNotInitialized()); R_UNLESS(this->IsValid(), fs::ResultNotInitialized());
R_UNLESS(m_resizable, fs::ResultUnsupportedSetSizeForNotResizableSubStorage()); R_UNLESS(m_resizable, fs::ResultUnsupportedSetSizeForNotResizableSubStorage());
R_UNLESS(IStorage::CheckOffsetAndSize(m_offset, size), fs::ResultInvalidSize());
R_TRY(IStorage::CheckOffsetAndSize(m_offset, size));
/* Ensure that we're allowed to set size. */ /* Ensure that we're allowed to set size. */
s64 cur_size; s64 cur_size;
@ -135,12 +136,17 @@ namespace ams::fs {
/* Ensure we're initialized. */ /* Ensure we're initialized. */
R_UNLESS(this->IsValid(), fs::ResultNotInitialized()); R_UNLESS(this->IsValid(), fs::ResultNotInitialized());
/* Succeed immediately on zero-sized operation. */ /* If we're not invalidating, sanity check arguments. */
R_SUCCEED_IF(size == 0); if (op_id != fs::OperationId::Invalidate) {
/* Succeed immediately on zero-sized operation other than invalidate. */
R_SUCCEED_IF(size == 0);
/* Validate arguments and operate. */ /* Check access extents. */
R_UNLESS(IStorage::CheckOffsetAndSize(offset, size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckOffsetAndSize(offset, size));
return m_base_storage->OperateRange(dst, dst_size, op_id, m_offset + offset, size, src, src_size); }
/* Perform the operation. */
R_RETURN(m_base_storage->OperateRange(dst, dst_size, op_id, m_offset + offset, size, src, src_size));
} }
using IStorage::OperateRange; using IStorage::OperateRange;

16
libraries/libstratosphere/include/stratosphere/fssystem/fssystem_alignment_matching_storage.hpp
View file

@ -62,7 +62,7 @@ namespace ams::fssystem {
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckAccessRange(offset, size, bs_size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckAccessRange(offset, size, bs_size));
return AlignmentMatchingStorageImpl::Read(m_base_storage, work_buf, sizeof(work_buf), DataAlign, BufferAlign, offset, static_cast<char *>(buffer), size); return AlignmentMatchingStorageImpl::Read(m_base_storage, work_buf, sizeof(work_buf), DataAlign, BufferAlign, offset, static_cast<char *>(buffer), size);
} }
@ -80,7 +80,7 @@ namespace ams::fssystem {
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckAccessRange(offset, size, bs_size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckAccessRange(offset, size, bs_size));
return AlignmentMatchingStorageImpl::Write(m_base_storage, work_buf, sizeof(work_buf), DataAlign, BufferAlign, offset, static_cast<const char *>(buffer), size); return AlignmentMatchingStorageImpl::Write(m_base_storage, work_buf, sizeof(work_buf), DataAlign, BufferAlign, offset, static_cast<const char *>(buffer), size);
} }
@ -119,7 +119,7 @@ namespace ams::fssystem {
/* Get the base storage size. */ /* Get the base storage size. */
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckOffsetAndSize(offset, size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckOffsetAndSize(offset, size));
/* Operate on the base storage. */ /* Operate on the base storage. */
const auto valid_size = std::min(size, bs_size - offset); const auto valid_size = std::min(size, bs_size - offset);
@ -160,7 +160,7 @@ namespace ams::fssystem {
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckAccessRange(offset, size, bs_size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckAccessRange(offset, size, bs_size));
/* Allocate a pooled buffer. */ /* Allocate a pooled buffer. */
PooledBuffer pooled_buffer; PooledBuffer pooled_buffer;
@ -178,7 +178,7 @@ namespace ams::fssystem {
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckAccessRange(offset, size, bs_size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckAccessRange(offset, size, bs_size));
/* Allocate a pooled buffer. */ /* Allocate a pooled buffer. */
PooledBuffer pooled_buffer; PooledBuffer pooled_buffer;
@ -221,7 +221,7 @@ namespace ams::fssystem {
/* Get the base storage size. */ /* Get the base storage size. */
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckOffsetAndSize(offset, size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckOffsetAndSize(offset, size));
/* Operate on the base storage. */ /* Operate on the base storage. */
const auto valid_size = std::min(size, bs_size - offset); const auto valid_size = std::min(size, bs_size - offset);
@ -268,7 +268,7 @@ namespace ams::fssystem {
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckAccessRange(offset, size, bs_size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckAccessRange(offset, size, bs_size));
/* Allocate a pooled buffer. */ /* Allocate a pooled buffer. */
PooledBuffer pooled_buffer(m_data_align, m_data_align); PooledBuffer pooled_buffer(m_data_align, m_data_align);
@ -308,7 +308,7 @@ namespace ams::fssystem {
/* Get the base storage size. */ /* Get the base storage size. */
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckOffsetAndSize(offset, size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckOffsetAndSize(offset, size));
/* Operate on the base storage. */ /* Operate on the base storage. */
const auto valid_size = std::min(size, bs_size - offset); const auto valid_size = std::min(size, bs_size - offset);

27
libraries/libstratosphere/source/fs/common/fs_file_storage.cpp
View file

@ -33,7 +33,7 @@ namespace ams::fs {
R_TRY(this->UpdateSize()); R_TRY(this->UpdateSize());
/* Ensure our access is valid. */ /* Ensure our access is valid. */
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
size_t read_size; size_t read_size;
return m_base_file->Read(std::addressof(read_size), offset, buffer, size); return m_base_file->Read(std::addressof(read_size), offset, buffer, size);
@ -50,7 +50,7 @@ namespace ams::fs {
R_TRY(this->UpdateSize()); R_TRY(this->UpdateSize());
/* Ensure our access is valid. */ /* Ensure our access is valid. */
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return m_base_file->Write(offset, buffer, size, fs::WriteOption()); return m_base_file->Write(offset, buffer, size, fs::WriteOption());
} }
@ -73,20 +73,21 @@ namespace ams::fs {
Result FileStorage::OperateRange(void *dst, size_t dst_size, OperationId op_id, s64 offset, s64 size, const void *src, size_t src_size) { Result FileStorage::OperateRange(void *dst, size_t dst_size, OperationId op_id, s64 offset, s64 size, const void *src, size_t src_size) {
switch (op_id) { switch (op_id) {
case OperationId::Invalidate: case OperationId::Invalidate:
R_RETURN(m_base_file->OperateRange(OperationId::Invalidate, offset, size));
case OperationId::QueryRange: case OperationId::QueryRange:
if (size == 0) { if (size == 0) {
if (op_id == OperationId::QueryRange) { R_UNLESS(dst != nullptr, fs::ResultNullptrArgument());
R_UNLESS(dst != nullptr, fs::ResultNullptrArgument()); R_UNLESS(dst_size == sizeof(QueryRangeInfo), fs::ResultInvalidSize());
R_UNLESS(dst_size == sizeof(QueryRangeInfo), fs::ResultInvalidSize()); reinterpret_cast<QueryRangeInfo *>(dst)->Clear();
reinterpret_cast<QueryRangeInfo *>(dst)->Clear(); R_SUCCEED();
}
return ResultSuccess();
} }
R_TRY(this->UpdateSize()); R_TRY(this->UpdateSize());
R_UNLESS(IStorage::CheckOffsetAndSize(offset, size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckOffsetAndSize(offset, size));
return m_base_file->OperateRange(dst, dst_size, op_id, offset, size, src, src_size);
R_RETURN(m_base_file->OperateRange(dst, dst_size, op_id, offset, size, src, src_size));
default: default:
return fs::ResultUnsupportedOperateRangeForFileStorage(); R_THROW(fs::ResultUnsupportedOperateRangeForFileStorage());
} }
} }
@ -121,7 +122,7 @@ namespace ams::fs {
R_TRY(this->UpdateSize()); R_TRY(this->UpdateSize());
/* Ensure our access is valid. */ /* Ensure our access is valid. */
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return ReadFile(m_handle, offset, buffer, size, fs::ReadOption()); return ReadFile(m_handle, offset, buffer, size, fs::ReadOption());
} }
@ -140,7 +141,7 @@ namespace ams::fs {
R_TRY(this->UpdateSize()); R_TRY(this->UpdateSize());
/* Ensure our access is valid. */ /* Ensure our access is valid. */
R_UNLESS(IStorage::CheckAccessRange(offset, size, m_size), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, m_size));
return WriteFile(m_handle, offset, buffer, size, fs::WriteOption()); return WriteFile(m_handle, offset, buffer, size, fs::WriteOption());
} }

2
libraries/libstratosphere/source/fssystem/fssystem_alignment_matching_storage_impl.cpp
View file

@ -199,7 +199,7 @@ namespace ams::fssystem {
s64 bs_size = 0; s64 bs_size = 0;
R_TRY(this->GetSize(std::addressof(bs_size))); R_TRY(this->GetSize(std::addressof(bs_size)));
R_UNLESS(fs::IStorage::CheckAccessRange(offset, size, bs_size), fs::ResultOutOfRange()); R_TRY(fs::IStorage::CheckAccessRange(offset, size, bs_size));
/* Determine extents. */ /* Determine extents. */
const auto offset_end = offset + static_cast<s64>(size); const auto offset_end = offset + static_cast<s64>(size);

8
libraries/libstratosphere/source/fssystem/fssystem_integrity_verification_storage.cpp
View file

@ -92,7 +92,7 @@ namespace ams::fssystem {
R_UNLESS(offset <= data_size, fs::ResultInvalidOffset()); R_UNLESS(offset <= data_size, fs::ResultInvalidOffset());
/* Validate the access range. */ /* Validate the access range. */
R_UNLESS(IStorage::CheckAccessRange(offset, size, util::AlignUp(data_size, static_cast<size_t>(m_verification_block_size))), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, util::AlignUp(data_size, static_cast<size_t>(m_verification_block_size))));
/* Determine the read extents. */ /* Determine the read extents. */
size_t read_size = size; size_t read_size = size;
@ -174,7 +174,9 @@ namespace ams::fssystem {
/* Validate arguments. */ /* Validate arguments. */
R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument()); R_UNLESS(buffer != nullptr, fs::ResultNullptrArgument());
R_UNLESS(IStorage::CheckOffsetAndSize(offset, size), fs::ResultInvalidOffset());
/* Check the offset/size. */
R_TRY(IStorage::CheckOffsetAndSize(offset, size));
/* Validate the offset. */ /* Validate the offset. */
s64 data_size; s64 data_size;
@ -182,7 +184,7 @@ namespace ams::fssystem {
R_UNLESS(offset < data_size, fs::ResultInvalidOffset()); R_UNLESS(offset < data_size, fs::ResultInvalidOffset());
/* Validate the access range. */ /* Validate the access range. */
R_UNLESS(IStorage::CheckAccessRange(offset, size, util::AlignUp(data_size, static_cast<size_t>(m_verification_block_size))), fs::ResultOutOfRange()); R_TRY(IStorage::CheckAccessRange(offset, size, util::AlignUp(data_size, static_cast<size_t>(m_verification_block_size))));
/* Validate preconditions. */ /* Validate preconditions. */
AMS_ASSERT(util::IsAligned(offset, m_verification_block_size)); AMS_ASSERT(util::IsAligned(offset, m_verification_block_size));