diff options
author | liamwhite <liamwhite@users.noreply.github.com> | 2024-02-24 22:24:01 +0100 |
---|---|---|
committer | GitHub <noreply@github.com> | 2024-02-24 22:24:01 +0100 |
commit | a93d249ac161e190b8c7046917bd9a3b6e79a0e0 (patch) | |
tree | e7589e51b5bf334aef78658b2e7efddcce3578dd /src/core/hle/service/set/system_settings_server.cpp | |
parent | Merge pull request #13153 from german77/defaultset (diff) | |
parent | service: set: Fix GetSettingsItemValue (diff) | |
download | yuzu-a93d249ac161e190b8c7046917bd9a3b6e79a0e0.tar yuzu-a93d249ac161e190b8c7046917bd9a3b6e79a0e0.tar.gz yuzu-a93d249ac161e190b8c7046917bd9a3b6e79a0e0.tar.bz2 yuzu-a93d249ac161e190b8c7046917bd9a3b6e79a0e0.tar.lz yuzu-a93d249ac161e190b8c7046917bd9a3b6e79a0e0.tar.xz yuzu-a93d249ac161e190b8c7046917bd9a3b6e79a0e0.tar.zst yuzu-a93d249ac161e190b8c7046917bd9a3b6e79a0e0.zip |
Diffstat (limited to 'src/core/hle/service/set/system_settings_server.cpp')
-rw-r--r-- | src/core/hle/service/set/system_settings_server.cpp | 13 |
1 files changed, 8 insertions, 5 deletions
diff --git a/src/core/hle/service/set/system_settings_server.cpp b/src/core/hle/service/set/system_settings_server.cpp index 6a7ea83fd..900d5408f 100644 --- a/src/core/hle/service/set/system_settings_server.cpp +++ b/src/core/hle/service/set/system_settings_server.cpp @@ -308,7 +308,7 @@ ISystemSettingsServer::ISystemSettingsServer(Core::System& system_) SetupSettings(); m_system_settings.region_code = - static_cast<SystemRegionCode>(Settings::values.region_index.GetValue()); + static_cast<SystemRegionCode>(::Settings::values.region_index.GetValue()); // TODO: Remove this when starter applet is fully functional EulaVersion eula_version{ @@ -715,7 +715,7 @@ Result ISystemSettingsServer::GetSettingsItemValueSize( } Result ISystemSettingsServer::GetSettingsItemValue( - OutBuffer<BufferAttr_HipcMapAlias> out_data, + Out<u64> out_size, OutBuffer<BufferAttr_HipcMapAlias> out_data, InLargeData<SettingItemName, BufferAttr_HipcPointer> setting_category_buffer, InLargeData<SettingItemName, BufferAttr_HipcPointer> setting_name_buffer) { const std::string setting_category{Common::StringFromBuffer(*setting_category_buffer)}; @@ -723,7 +723,7 @@ Result ISystemSettingsServer::GetSettingsItemValue( LOG_INFO(Service_SET, "called, category={}, name={}", setting_category, setting_name); - R_RETURN(GetSettingsItemValueImpl(out_data, setting_category, setting_name)); + R_RETURN(GetSettingsItemValueImpl(out_data, *out_size, setting_category, setting_name)); } Result ISystemSettingsServer::GetTvSettings(Out<TvSettings> out_tv_settings) { @@ -1363,13 +1363,16 @@ void ISystemSettingsServer::SetSaveNeeded() { m_save_needed = true; } -Result ISystemSettingsServer::GetSettingsItemValueImpl(std::vector<u8>& out_value, +Result ISystemSettingsServer::GetSettingsItemValueImpl(std::span<u8> out_value, u64& out_size, const std::string& category, const std::string& name) { auto settings{GetSettings()}; R_UNLESS(settings.contains(category) && settings[category].contains(name), ResultUnknown); - out_value = settings[category][name]; + ASSERT_MSG(out_value.size() >= settings[category][name].size(), + "Stored type is bigger than requested type"); + out_size = std::min<u64>(settings[category][name].size(), out_value.size()); + std::memcpy(out_value.data(), settings[category][name].data(), out_size); R_SUCCEED(); } |