summaryrefslogtreecommitdiffstats
path: root/src/core/hle/kernel
diff options
context:
space:
mode:
authorSubv <subv2112@gmail.com>2017-01-11 19:20:14 +0100
committerSubv <subv2112@gmail.com>2017-01-11 22:38:05 +0100
commit1ddff1451140ef58058237a3198d363b96dc238e (patch)
tree4c8d364f61477d80de05f56917d77b566975de1e /src/core/hle/kernel
parentThread: Added priority range checking to svcSetThreadPriority and removed priority clamping code from Thread::SetPriority. (diff)
downloadyuzu-1ddff1451140ef58058237a3198d363b96dc238e.tar
yuzu-1ddff1451140ef58058237a3198d363b96dc238e.tar.gz
yuzu-1ddff1451140ef58058237a3198d363b96dc238e.tar.bz2
yuzu-1ddff1451140ef58058237a3198d363b96dc238e.tar.lz
yuzu-1ddff1451140ef58058237a3198d363b96dc238e.tar.xz
yuzu-1ddff1451140ef58058237a3198d363b96dc238e.tar.zst
yuzu-1ddff1451140ef58058237a3198d363b96dc238e.zip
Diffstat (limited to 'src/core/hle/kernel')
-rw-r--r--src/core/hle/kernel/thread.cpp10
1 files changed, 2 insertions, 8 deletions
diff --git a/src/core/hle/kernel/thread.cpp b/src/core/hle/kernel/thread.cpp
index 5ba9abf29..3b7555d87 100644
--- a/src/core/hle/kernel/thread.cpp
+++ b/src/core/hle/kernel/thread.cpp
@@ -353,14 +353,8 @@ static void ResetThreadContext(ARM_Interface::ThreadContext& context, u32 stack_
ResultVal<SharedPtr<Thread>> Thread::Create(std::string name, VAddr entry_point, s32 priority,
u32 arg, s32 processor_id, VAddr stack_top) {
- if (priority < THREADPRIO_HIGHEST || priority > THREADPRIO_LOWEST) {
- s32 new_priority = MathUtil::Clamp<s32>(priority, THREADPRIO_HIGHEST, THREADPRIO_LOWEST);
- LOG_WARNING(Kernel_SVC, "(name=%s): invalid priority=%d, clamping to %d", name.c_str(),
- priority, new_priority);
- // TODO(bunnei): Clamping to a valid priority is not necessarily correct behavior... Confirm
- // validity of this
- priority = new_priority;
- }
+ ASSERT_MSG(priority >= THREADPRIO_HIGHEST && priority <= THREADPRIO_LOWEST,
+ "Invalid thread priority");
if (!Memory::IsValidVirtualAddress(entry_point)) {
LOG_ERROR(Kernel_SVC, "(name=%s): invalid entry %08x", name.c_str(), entry_point);