Changeset f6d2c81 in mainline for kernel/generic/src/proc/thread.c

Timestamp:

2007-05-30T19:50:24Z (18 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

lfn, master, serial, ticket/834-toolchain-update, topic/msim-upgrade, topic/simplify-dev-export

Children:

c31e536

Parents:

51ec40f

Message:

Fix two memory leaks.

In kernel, kernel_uarg structure needs to be deallocated when a thread
with userspace context is destroyed.

In userspace, the return value of the SYS_THREAD_CREATE must be checked
for error conditions and in case of error, uarg and stack must be freed
up.

File:

• kernel/generic/src/proc/thread.c (modified) (5 diffs)

kernel/generic/src/proc/thread.c

@@ -300 +300 @@
                 task_destroy(t->task);
         
+        /*
+         * If the thread had a userspace context, free up its kernel_uarg
+         * structure.
+         */
+        if (t->flags & THREAD_FLAG_USPACE) {
+                ASSERT(t->thread_arg);
+                free(t->thread_arg);
+        }
+
         slab_free(thread_slab, t);
 }
@@ -313 +322 @@
  * @param name      Symbolic name.
  * @param uncounted Thread's accounting doesn't affect accumulated task
- *       accounting.
+ *                  accounting.
  *
  * @return New thread's structure on success, NULL on failure.
@@ -319 +328 @@
  */
 thread_t *thread_create(void (* func)(void *), void *arg, task_t *task,
-        int flags, char *name, bool uncounted)
+    int flags, char *name, bool uncounted)
 {
         thread_t *t;
@@ -638 +647 @@
  *
  */
-unative_t sys_thread_create(uspace_arg_t *uspace_uarg, char *uspace_name, thread_id_t *uspace_thread_id)
+unative_t sys_thread_create(uspace_arg_t *uspace_uarg, char *uspace_name,
+    thread_id_t *uspace_thread_id)
 {
         thread_t *t;
@@ -661 +671 @@
                 thread_ready(t);
                 if (uspace_thread_id != NULL)
-                        return (unative_t) copy_to_uspace(uspace_thread_id, &t->tid,
-                            sizeof(t->tid));
+                        return (unative_t) copy_to_uspace(uspace_thread_id,
+                            &t->tid, sizeof(t->tid));
                 else
                         return 0;