summaryrefslogtreecommitdiff
path: root/src
diff options
context:
space:
mode:
authorLioncash <mathew1800@gmail.com>2018-12-30 20:59:52 -0500
committerLioncash <mathew1800@gmail.com>2018-12-30 20:59:54 -0500
commit205e6d3b975a7f699aeb0341c78a55562094c32b (patch)
tree6708c8120ba81f355788a011989ed9886be32f37 /src
parente5dfbe22ee458dc818e5e56daa7f1a43e6ed7205 (diff)
kernel/svc: Simplify thread core ID sanitizing in CreateThread
Rather than use a switch here, this can be collapsed into a simple range check, which is a little easier on the eyes.
Diffstat (limited to 'src')
-rw-r--r--src/core/hle/kernel/svc.cpp8
1 files changed, 1 insertions, 7 deletions
diff --git a/src/core/hle/kernel/svc.cpp b/src/core/hle/kernel/svc.cpp
index 660e6f577..352ab2d14 100644
--- a/src/core/hle/kernel/svc.cpp
+++ b/src/core/hle/kernel/svc.cpp
@@ -1233,13 +1233,7 @@ static ResultCode CreateThread(Handle* out_handle, VAddr entry_point, u64 arg, V
ASSERT(processor_id != THREADPROCESSORID_IDEAL);
}
- switch (processor_id) {
- case THREADPROCESSORID_0:
- case THREADPROCESSORID_1:
- case THREADPROCESSORID_2:
- case THREADPROCESSORID_3:
- break;
- default:
+ if (processor_id < THREADPROCESSORID_0 || processor_id > THREADPROCESSORID_3) {
LOG_ERROR(Kernel_SVC, "Invalid thread processor ID: {}", processor_id);
return ERR_INVALID_PROCESSOR_ID;
}