summaryrefslogtreecommitdiff
path: root/src/core
diff options
context:
space:
mode:
authorGravatar Lioncash2021-04-19 12:30:31 -0400
committerGravatar Lioncash2021-04-19 12:30:34 -0400
commitb7d04d848ddcae5cc412b794bea12bf7cc5df296 (patch)
tree2f2c54c870fc96efea8844f4e40f65ac47bd0363 /src/core
parentMerge pull request #6208 from lat9nq/boost-static (diff)
downloadyuzu-b7d04d848ddcae5cc412b794bea12bf7cc5df296.tar.gz
yuzu-b7d04d848ddcae5cc412b794bea12bf7cc5df296.tar.xz
yuzu-b7d04d848ddcae5cc412b794bea12bf7cc5df296.zip
arp: Prevent uninitialized read of launch member variable
If anything happened to call arp functions in the wrong order and called IRegistrar's Issue function before SetApplicationLaunchProperty, we'd read from an uninitialized ApplicationLaunchProperty instance. Instead, we can always initialize it so if this does happen, then the outcome of doing such a thing is at least consistently reproducible.
Diffstat (limited to 'src/core')
-rw-r--r--src/core/hle/service/glue/arp.cpp2
1 files changed, 1 insertions, 1 deletions
diff --git a/src/core/hle/service/glue/arp.cpp b/src/core/hle/service/glue/arp.cpp
index 322125135..e6d9d1b24 100644
--- a/src/core/hle/service/glue/arp.cpp
+++ b/src/core/hle/service/glue/arp.cpp
@@ -240,7 +240,7 @@ private:
240 240
241 std::function<ResultCode(u64, ApplicationLaunchProperty, std::vector<u8>)> issue_process_id; 241 std::function<ResultCode(u64, ApplicationLaunchProperty, std::vector<u8>)> issue_process_id;
242 bool issued = false; 242 bool issued = false;
243 ApplicationLaunchProperty launch; 243 ApplicationLaunchProperty launch{};
244 std::vector<u8> control; 244 std::vector<u8> control;
245}; 245};
246 246