SELinux: use do_each_thread as a proper do/while block
authorJames Morris <jmorris@namei.org>
Wed, 11 Jun 2008 15:42:35 +0000 (01:42 +1000)
committerJames Morris <jmorris@namei.org>
Mon, 14 Jul 2008 05:02:02 +0000 (15:02 +1000)
Use do_each_thread as a proper do/while block.  Sparse complained.

Signed-off-by: James Morris <jmorris@namei.org>
Acked-by: Stephen Smalley <sds@tycho.nsa.gov>
security/selinux/hooks.c

index 6e8d0e91c0ca5fd03140cfbc43a01ec3567dfe69..4130d64359a3672e7861319247429f5753fce788 100644 (file)
@@ -5196,12 +5196,12 @@ static int selinux_setprocattr(struct task_struct *p,
                        struct task_struct *g, *t;
                        struct mm_struct *mm = p->mm;
                        read_lock(&tasklist_lock);
-                       do_each_thread(g, t)
+                       do_each_thread(g, t) {
                                if (t->mm == mm && t != p) {
                                        read_unlock(&tasklist_lock);
                                        return -EPERM;
                                }
-                       while_each_thread(g, t);
+                       while_each_thread(g, t);
                        read_unlock(&tasklist_lock);
                }