diff options
Diffstat (limited to 'fs/ioprio.c')
| -rw-r--r-- | fs/ioprio.c | 18 |
1 files changed, 16 insertions, 2 deletions
diff --git a/fs/ioprio.c b/fs/ioprio.c index 748cfb92dcc6..2f7d05c89922 100644 --- a/fs/ioprio.c +++ b/fs/ioprio.c | |||
| @@ -111,12 +111,14 @@ SYSCALL_DEFINE3(ioprio_set, int, which, int, who, int, ioprio) | |||
| 111 | read_lock(&tasklist_lock); | 111 | read_lock(&tasklist_lock); |
| 112 | switch (which) { | 112 | switch (which) { |
| 113 | case IOPRIO_WHO_PROCESS: | 113 | case IOPRIO_WHO_PROCESS: |
| 114 | rcu_read_lock(); | ||
| 114 | if (!who) | 115 | if (!who) |
| 115 | p = current; | 116 | p = current; |
| 116 | else | 117 | else |
| 117 | p = find_task_by_vpid(who); | 118 | p = find_task_by_vpid(who); |
| 118 | if (p) | 119 | if (p) |
| 119 | ret = set_task_ioprio(p, ioprio); | 120 | ret = set_task_ioprio(p, ioprio); |
| 121 | rcu_read_unlock(); | ||
| 120 | break; | 122 | break; |
| 121 | case IOPRIO_WHO_PGRP: | 123 | case IOPRIO_WHO_PGRP: |
| 122 | if (!who) | 124 | if (!who) |
| @@ -139,7 +141,12 @@ SYSCALL_DEFINE3(ioprio_set, int, which, int, who, int, ioprio) | |||
| 139 | break; | 141 | break; |
| 140 | 142 | ||
| 141 | do_each_thread(g, p) { | 143 | do_each_thread(g, p) { |
| 142 | if (__task_cred(p)->uid != who) | 144 | int match; |
| 145 | |||
| 146 | rcu_read_lock(); | ||
| 147 | match = __task_cred(p)->uid == who; | ||
| 148 | rcu_read_unlock(); | ||
| 149 | if (!match) | ||
| 143 | continue; | 150 | continue; |
| 144 | ret = set_task_ioprio(p, ioprio); | 151 | ret = set_task_ioprio(p, ioprio); |
| 145 | if (ret) | 152 | if (ret) |
| @@ -200,12 +207,14 @@ SYSCALL_DEFINE2(ioprio_get, int, which, int, who) | |||
| 200 | read_lock(&tasklist_lock); | 207 | read_lock(&tasklist_lock); |
| 201 | switch (which) { | 208 | switch (which) { |
| 202 | case IOPRIO_WHO_PROCESS: | 209 | case IOPRIO_WHO_PROCESS: |
| 210 | rcu_read_lock(); | ||
| 203 | if (!who) | 211 | if (!who) |
| 204 | p = current; | 212 | p = current; |
| 205 | else | 213 | else |
| 206 | p = find_task_by_vpid(who); | 214 | p = find_task_by_vpid(who); |
| 207 | if (p) | 215 | if (p) |
| 208 | ret = get_task_ioprio(p); | 216 | ret = get_task_ioprio(p); |
| 217 | rcu_read_unlock(); | ||
| 209 | break; | 218 | break; |
| 210 | case IOPRIO_WHO_PGRP: | 219 | case IOPRIO_WHO_PGRP: |
| 211 | if (!who) | 220 | if (!who) |
| @@ -232,7 +241,12 @@ SYSCALL_DEFINE2(ioprio_get, int, which, int, who) | |||
| 232 | break; | 241 | break; |
| 233 | 242 | ||
| 234 | do_each_thread(g, p) { | 243 | do_each_thread(g, p) { |
| 235 | if (__task_cred(p)->uid != user->uid) | 244 | int match; |
| 245 | |||
| 246 | rcu_read_lock(); | ||
| 247 | match = __task_cred(p)->uid == user->uid; | ||
| 248 | rcu_read_unlock(); | ||
| 249 | if (!match) | ||
| 236 | continue; | 250 | continue; |
| 237 | tmpio = get_task_ioprio(p); | 251 | tmpio = get_task_ioprio(p); |
| 238 | if (tmpio < 0) | 252 | if (tmpio < 0) |
