diff options
author | Lioncash <mathew1800@gmail.com> | 2018-10-24 15:03:29 -0400 |
---|---|---|
committer | Lioncash <mathew1800@gmail.com> | 2018-10-24 18:22:20 -0400 |
commit | 3ec90dc6ef529590112eabc6df412fe18a085aab (patch) | |
tree | d1e70014dedad725d4328d0ce7ca895fb803f4de | |
parent | 77e705a8fa0a70dac2eb81b95fedc5e98d7c274e (diff) |
service/acc: Early return in failure case in LoadImage()
Allows unindenting the other branch's code.
-rw-r--r-- | src/core/hle/service/acc/acc.cpp | 16 |
1 files changed, 8 insertions, 8 deletions
diff --git a/src/core/hle/service/acc/acc.cpp b/src/core/hle/service/acc/acc.cpp index cf065c2e0..17e3aa0e2 100644 --- a/src/core/hle/service/acc/acc.cpp +++ b/src/core/hle/service/acc/acc.cpp @@ -104,20 +104,20 @@ private: rb.Push(RESULT_SUCCESS); const FileUtil::IOFile image(GetImagePath(user_id), "rb"); - if (!image.IsOpen()) { LOG_WARNING(Service_ACC, "Failed to load user provided image! Falling back to built-in backup..."); ctx.WriteBuffer(backup_jpeg); rb.Push<u32>(backup_jpeg_size); - } else { - const auto size = std::min<u32>(image.GetSize(), MAX_JPEG_IMAGE_SIZE); - std::vector<u8> buffer(size); - image.ReadBytes(buffer.data(), buffer.size()); - - ctx.WriteBuffer(buffer.data(), buffer.size()); - rb.Push<u32>(buffer.size()); + return; } + + const auto size = std::min<u32>(image.GetSize(), MAX_JPEG_IMAGE_SIZE); + std::vector<u8> buffer(size); + image.ReadBytes(buffer.data(), buffer.size()); + + ctx.WriteBuffer(buffer.data(), buffer.size()); + rb.Push<u32>(buffer.size()); } void GetImageSize(Kernel::HLERequestContext& ctx) { |