Changeset 2569ec90 in mainline for generic/src/security/cap.c

Timestamp:

2006-06-05T18:15:30Z (18 years ago)

Author:

Jakub Jermar <jakub@…>

Branches:

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

Children:

b65caba1

Parents:

31e8ddd

Message:

In task_kill(), remove the task from the tasks_btree before proceeding.
Thus, when the kernel finds the task in the tasks_btree and locks it before
releasing tasks_lock, it is guaranteed that the task will not be destroyed
until the lock is held. If the kernel needs to unlock the task, do some operation
and lock it again, it should increase its refcount before doing so. In that case,
when releasing the lock, it must decrement the refcount and if it reaches
zero, it must call task_destroy().

File:

• generic/src/security/cap.c (modified) (3 diffs)

generic/src/security/cap.c

@@ -113 +113 @@
                 return (__native) ENOENT;
         }
+        
+        spinlock_lock(&t->lock);
+        cap_set(t, cap_get(t) | caps);
+        spinlock_unlock(&t->lock);
+        
         spinlock_unlock(&tasks_lock);
         
-        cap_set(t, cap_get(t) | caps);
+
         
         interrupts_restore(ipl);        
@@ -150 +155 @@
                 return (__native) ENOENT;
         }
-        spinlock_unlock(&tasks_lock);
 
         /*
@@ -158 +162 @@
          */
         if (!(cap_get(TASK) & CAP_CAP) || !(t == TASK)) {
+                spinlock_unlock(&tasks_lock);
                 interrupts_restore(ipl);
                 return (__native) EPERM;
         }
+        
+        spinlock_lock(&t->lock);
+        cap_set(t, cap_get(t) & ~caps);
+        spinlock_unlock(&t->lock);
 
-        cap_set(t, cap_get(t) & ~caps);
-        
+        spinlock_unlock(&tasks_lock);
+
         interrupts_restore(ipl);
         return 0;