diff options
| author | Justus Winter <4winter@informatik.uni-hamburg.de> | 2015-05-29 14:44:39 +0200 |
|---|---|---|
| committer | Justus Winter <4winter@informatik.uni-hamburg.de> | 2015-05-29 15:32:43 +0200 |
| commit | 99a43d36c83d318f24edd8c6d80f060686a6c398 (patch) | |
| tree | c4ab9a60249e92a8ef8d2b61a5cddef9e9e8d44b /kern/thread.c | |
| parent | b2b8bf37934087f8fb2a246316ecf9e91955ec57 (diff) | |
| download | gnumach-99a43d36c83d318f24edd8c6d80f060686a6c398.tar.gz gnumach-99a43d36c83d318f24edd8c6d80f060686a6c398.tar.bz2 gnumach-99a43d36c83d318f24edd8c6d80f060686a6c398.zip | |
kern: fix argument handling
Previously, the processor argument was not checked. If called with a
non-processor argument (like a task), `processor' is set to NULL,
triggering a page fault. Likewise for the other functions.
* kern/processor.c (processor_get_assignment): Fix argument handling.
* kern/task.c (task_get_assignment): Likewise.
* kern/thread.c (thread_get_assignment): Likewise.
Diffstat (limited to 'kern/thread.c')
| -rw-r--r-- | kern/thread.c | 3 |
1 files changed, 3 insertions, 0 deletions
diff --git a/kern/thread.c b/kern/thread.c index f52c95b8..8b1e9f58 100644 --- a/kern/thread.c +++ b/kern/thread.c @@ -1974,6 +1974,9 @@ kern_return_t thread_get_assignment( thread_t thread, processor_set_t *pset) { + if (thread == THREAD_NULL) + return KERN_INVALID_ARGUMENT; + *pset = thread->processor_set; pset_reference(*pset); return KERN_SUCCESS; |