Greg Kroah-Hartman | b244131 | 2017-11-01 15:07:57 +0100 | [diff] [blame] | 1 | // SPDX-License-Identifier: GPL-2.0 |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 2 | /* |
Lai Jiangshan | 47c5980 | 2008-10-18 20:28:07 -0700 | [diff] [blame] | 3 | * device_cgroup.c - device cgroup subsystem |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 4 | * |
| 5 | * Copyright 2007 IBM Corp |
| 6 | */ |
| 7 | |
| 8 | #include <linux/device_cgroup.h> |
| 9 | #include <linux/cgroup.h> |
| 10 | #include <linux/ctype.h> |
| 11 | #include <linux/list.h> |
| 12 | #include <linux/uaccess.h> |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 13 | #include <linux/seq_file.h> |
Tejun Heo | 5a0e3ad | 2010-03-24 17:04:11 +0900 | [diff] [blame] | 14 | #include <linux/slab.h> |
Lai Jiangshan | 47c5980 | 2008-10-18 20:28:07 -0700 | [diff] [blame] | 15 | #include <linux/rcupdate.h> |
Li Zefan | b4046f0 | 2009-04-02 16:57:32 -0700 | [diff] [blame] | 16 | #include <linux/mutex.h> |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 17 | |
Li Zefan | b4046f0 | 2009-04-02 16:57:32 -0700 | [diff] [blame] | 18 | static DEFINE_MUTEX(devcgroup_mutex); |
| 19 | |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 20 | enum devcg_behavior { |
| 21 | DEVCG_DEFAULT_NONE, |
| 22 | DEVCG_DEFAULT_ALLOW, |
| 23 | DEVCG_DEFAULT_DENY, |
| 24 | }; |
| 25 | |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 26 | /* |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 27 | * exception list locking rules: |
Li Zefan | b4046f0 | 2009-04-02 16:57:32 -0700 | [diff] [blame] | 28 | * hold devcgroup_mutex for update/read. |
Lai Jiangshan | 47c5980 | 2008-10-18 20:28:07 -0700 | [diff] [blame] | 29 | * hold rcu_read_lock() for read. |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 30 | */ |
| 31 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 32 | struct dev_exception_item { |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 33 | u32 major, minor; |
| 34 | short type; |
| 35 | short access; |
| 36 | struct list_head list; |
Pavel Emelyanov | 4efd1a1 | 2008-07-25 01:47:07 -0700 | [diff] [blame] | 37 | struct rcu_head rcu; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 38 | }; |
| 39 | |
| 40 | struct dev_cgroup { |
| 41 | struct cgroup_subsys_state css; |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 42 | struct list_head exceptions; |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 43 | enum devcg_behavior behavior; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 44 | }; |
| 45 | |
Pavel Emelyanov | b66862f | 2008-06-05 22:46:24 -0700 | [diff] [blame] | 46 | static inline struct dev_cgroup *css_to_devcgroup(struct cgroup_subsys_state *s) |
| 47 | { |
Tejun Heo | a7c6d55 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 48 | return s ? container_of(s, struct dev_cgroup, css) : NULL; |
Pavel Emelyanov | b66862f | 2008-06-05 22:46:24 -0700 | [diff] [blame] | 49 | } |
| 50 | |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 51 | static inline struct dev_cgroup *task_devcgroup(struct task_struct *task) |
| 52 | { |
Tejun Heo | 073219e | 2014-02-08 10:36:58 -0500 | [diff] [blame] | 53 | return css_to_devcgroup(task_css(task, devices_cgrp_id)); |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 54 | } |
| 55 | |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 56 | /* |
Li Zefan | b4046f0 | 2009-04-02 16:57:32 -0700 | [diff] [blame] | 57 | * called under devcgroup_mutex |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 58 | */ |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 59 | static int dev_exceptions_copy(struct list_head *dest, struct list_head *orig) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 60 | { |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 61 | struct dev_exception_item *ex, *tmp, *new; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 62 | |
Tejun Heo | 4b1c784 | 2012-11-06 09:16:53 -0800 | [diff] [blame] | 63 | lockdep_assert_held(&devcgroup_mutex); |
| 64 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 65 | list_for_each_entry(ex, orig, list) { |
| 66 | new = kmemdup(ex, sizeof(*ex), GFP_KERNEL); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 67 | if (!new) |
| 68 | goto free_and_exit; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 69 | list_add_tail(&new->list, dest); |
| 70 | } |
| 71 | |
| 72 | return 0; |
| 73 | |
| 74 | free_and_exit: |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 75 | list_for_each_entry_safe(ex, tmp, dest, list) { |
| 76 | list_del(&ex->list); |
| 77 | kfree(ex); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 78 | } |
| 79 | return -ENOMEM; |
| 80 | } |
| 81 | |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 82 | /* |
Li Zefan | b4046f0 | 2009-04-02 16:57:32 -0700 | [diff] [blame] | 83 | * called under devcgroup_mutex |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 84 | */ |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 85 | static int dev_exception_add(struct dev_cgroup *dev_cgroup, |
| 86 | struct dev_exception_item *ex) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 87 | { |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 88 | struct dev_exception_item *excopy, *walk; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 89 | |
Tejun Heo | 4b1c784 | 2012-11-06 09:16:53 -0800 | [diff] [blame] | 90 | lockdep_assert_held(&devcgroup_mutex); |
| 91 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 92 | excopy = kmemdup(ex, sizeof(*ex), GFP_KERNEL); |
| 93 | if (!excopy) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 94 | return -ENOMEM; |
| 95 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 96 | list_for_each_entry(walk, &dev_cgroup->exceptions, list) { |
| 97 | if (walk->type != ex->type) |
Pavel Emelyanov | d1ee297 | 2008-06-05 22:46:28 -0700 | [diff] [blame] | 98 | continue; |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 99 | if (walk->major != ex->major) |
Pavel Emelyanov | d1ee297 | 2008-06-05 22:46:28 -0700 | [diff] [blame] | 100 | continue; |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 101 | if (walk->minor != ex->minor) |
Pavel Emelyanov | d1ee297 | 2008-06-05 22:46:28 -0700 | [diff] [blame] | 102 | continue; |
| 103 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 104 | walk->access |= ex->access; |
| 105 | kfree(excopy); |
| 106 | excopy = NULL; |
Pavel Emelyanov | d1ee297 | 2008-06-05 22:46:28 -0700 | [diff] [blame] | 107 | } |
| 108 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 109 | if (excopy != NULL) |
| 110 | list_add_tail_rcu(&excopy->list, &dev_cgroup->exceptions); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 111 | return 0; |
| 112 | } |
| 113 | |
| 114 | /* |
Li Zefan | b4046f0 | 2009-04-02 16:57:32 -0700 | [diff] [blame] | 115 | * called under devcgroup_mutex |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 116 | */ |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 117 | static void dev_exception_rm(struct dev_cgroup *dev_cgroup, |
| 118 | struct dev_exception_item *ex) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 119 | { |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 120 | struct dev_exception_item *walk, *tmp; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 121 | |
Tejun Heo | 4b1c784 | 2012-11-06 09:16:53 -0800 | [diff] [blame] | 122 | lockdep_assert_held(&devcgroup_mutex); |
| 123 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 124 | list_for_each_entry_safe(walk, tmp, &dev_cgroup->exceptions, list) { |
| 125 | if (walk->type != ex->type) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 126 | continue; |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 127 | if (walk->major != ex->major) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 128 | continue; |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 129 | if (walk->minor != ex->minor) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 130 | continue; |
| 131 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 132 | walk->access &= ~ex->access; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 133 | if (!walk->access) { |
Pavel Emelyanov | 4efd1a1 | 2008-07-25 01:47:07 -0700 | [diff] [blame] | 134 | list_del_rcu(&walk->list); |
Lai Jiangshan | 6034f7e | 2011-03-15 18:07:57 +0800 | [diff] [blame] | 135 | kfree_rcu(walk, rcu); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 136 | } |
| 137 | } |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 138 | } |
| 139 | |
Jerry Snitselaar | 53eb8c8 | 2013-02-21 16:41:31 -0800 | [diff] [blame] | 140 | static void __dev_exception_clean(struct dev_cgroup *dev_cgroup) |
| 141 | { |
| 142 | struct dev_exception_item *ex, *tmp; |
| 143 | |
| 144 | list_for_each_entry_safe(ex, tmp, &dev_cgroup->exceptions, list) { |
| 145 | list_del_rcu(&ex->list); |
| 146 | kfree_rcu(ex, rcu); |
| 147 | } |
| 148 | } |
| 149 | |
Aristeu Rozanski | 868539a | 2012-10-04 17:15:15 -0700 | [diff] [blame] | 150 | /** |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 151 | * dev_exception_clean - frees all entries of the exception list |
| 152 | * @dev_cgroup: dev_cgroup with the exception list to be cleaned |
Aristeu Rozanski | 868539a | 2012-10-04 17:15:15 -0700 | [diff] [blame] | 153 | * |
| 154 | * called under devcgroup_mutex |
| 155 | */ |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 156 | static void dev_exception_clean(struct dev_cgroup *dev_cgroup) |
Aristeu Rozanski | 868539a | 2012-10-04 17:15:15 -0700 | [diff] [blame] | 157 | { |
Tejun Heo | 4b1c784 | 2012-11-06 09:16:53 -0800 | [diff] [blame] | 158 | lockdep_assert_held(&devcgroup_mutex); |
| 159 | |
Jerry Snitselaar | 53eb8c8 | 2013-02-21 16:41:31 -0800 | [diff] [blame] | 160 | __dev_exception_clean(dev_cgroup); |
Aristeu Rozanski | 868539a | 2012-10-04 17:15:15 -0700 | [diff] [blame] | 161 | } |
| 162 | |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 163 | static inline bool is_devcg_online(const struct dev_cgroup *devcg) |
| 164 | { |
| 165 | return (devcg->behavior != DEVCG_DEFAULT_NONE); |
| 166 | } |
| 167 | |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 168 | /** |
| 169 | * devcgroup_online - initializes devcgroup's behavior and exceptions based on |
| 170 | * parent's |
Tejun Heo | eb95419 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 171 | * @css: css getting online |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 172 | * returns 0 in case of success, error code otherwise |
| 173 | */ |
Tejun Heo | eb95419 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 174 | static int devcgroup_online(struct cgroup_subsys_state *css) |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 175 | { |
Tejun Heo | eb95419 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 176 | struct dev_cgroup *dev_cgroup = css_to_devcgroup(css); |
Tejun Heo | 5c9d535 | 2014-05-16 13:22:48 -0400 | [diff] [blame] | 177 | struct dev_cgroup *parent_dev_cgroup = css_to_devcgroup(css->parent); |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 178 | int ret = 0; |
| 179 | |
| 180 | mutex_lock(&devcgroup_mutex); |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 181 | |
| 182 | if (parent_dev_cgroup == NULL) |
| 183 | dev_cgroup->behavior = DEVCG_DEFAULT_ALLOW; |
| 184 | else { |
| 185 | ret = dev_exceptions_copy(&dev_cgroup->exceptions, |
| 186 | &parent_dev_cgroup->exceptions); |
| 187 | if (!ret) |
| 188 | dev_cgroup->behavior = parent_dev_cgroup->behavior; |
| 189 | } |
| 190 | mutex_unlock(&devcgroup_mutex); |
| 191 | |
| 192 | return ret; |
| 193 | } |
| 194 | |
Tejun Heo | eb95419 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 195 | static void devcgroup_offline(struct cgroup_subsys_state *css) |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 196 | { |
Tejun Heo | eb95419 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 197 | struct dev_cgroup *dev_cgroup = css_to_devcgroup(css); |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 198 | |
| 199 | mutex_lock(&devcgroup_mutex); |
| 200 | dev_cgroup->behavior = DEVCG_DEFAULT_NONE; |
| 201 | mutex_unlock(&devcgroup_mutex); |
| 202 | } |
| 203 | |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 204 | /* |
| 205 | * called from kernel/cgroup.c with cgroup_lock() held. |
| 206 | */ |
Tejun Heo | eb95419 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 207 | static struct cgroup_subsys_state * |
| 208 | devcgroup_css_alloc(struct cgroup_subsys_state *parent_css) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 209 | { |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 210 | struct dev_cgroup *dev_cgroup; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 211 | |
| 212 | dev_cgroup = kzalloc(sizeof(*dev_cgroup), GFP_KERNEL); |
| 213 | if (!dev_cgroup) |
| 214 | return ERR_PTR(-ENOMEM); |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 215 | INIT_LIST_HEAD(&dev_cgroup->exceptions); |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 216 | dev_cgroup->behavior = DEVCG_DEFAULT_NONE; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 217 | |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 218 | return &dev_cgroup->css; |
| 219 | } |
| 220 | |
Tejun Heo | eb95419 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 221 | static void devcgroup_css_free(struct cgroup_subsys_state *css) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 222 | { |
Tejun Heo | eb95419 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 223 | struct dev_cgroup *dev_cgroup = css_to_devcgroup(css); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 224 | |
Jerry Snitselaar | 53eb8c8 | 2013-02-21 16:41:31 -0800 | [diff] [blame] | 225 | __dev_exception_clean(dev_cgroup); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 226 | kfree(dev_cgroup); |
| 227 | } |
| 228 | |
| 229 | #define DEVCG_ALLOW 1 |
| 230 | #define DEVCG_DENY 2 |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 231 | #define DEVCG_LIST 3 |
| 232 | |
Li Zefan | 17d213f | 2008-07-13 12:14:02 -0700 | [diff] [blame] | 233 | #define MAJMINLEN 13 |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 234 | #define ACCLEN 4 |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 235 | |
| 236 | static void set_access(char *acc, short access) |
| 237 | { |
| 238 | int idx = 0; |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 239 | memset(acc, 0, ACCLEN); |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 240 | if (access & DEVCG_ACC_READ) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 241 | acc[idx++] = 'r'; |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 242 | if (access & DEVCG_ACC_WRITE) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 243 | acc[idx++] = 'w'; |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 244 | if (access & DEVCG_ACC_MKNOD) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 245 | acc[idx++] = 'm'; |
| 246 | } |
| 247 | |
| 248 | static char type_to_char(short type) |
| 249 | { |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 250 | if (type == DEVCG_DEV_ALL) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 251 | return 'a'; |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 252 | if (type == DEVCG_DEV_CHAR) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 253 | return 'c'; |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 254 | if (type == DEVCG_DEV_BLOCK) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 255 | return 'b'; |
| 256 | return 'X'; |
| 257 | } |
| 258 | |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 259 | static void set_majmin(char *str, unsigned m) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 260 | { |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 261 | if (m == ~0) |
Li Zefan | 7759fc9 | 2008-07-25 01:47:08 -0700 | [diff] [blame] | 262 | strcpy(str, "*"); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 263 | else |
Li Zefan | 7759fc9 | 2008-07-25 01:47:08 -0700 | [diff] [blame] | 264 | sprintf(str, "%u", m); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 265 | } |
| 266 | |
Tejun Heo | 2da8ca8 | 2013-12-05 12:28:04 -0500 | [diff] [blame] | 267 | static int devcgroup_seq_show(struct seq_file *m, void *v) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 268 | { |
Tejun Heo | 2da8ca8 | 2013-12-05 12:28:04 -0500 | [diff] [blame] | 269 | struct dev_cgroup *devcgroup = css_to_devcgroup(seq_css(m)); |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 270 | struct dev_exception_item *ex; |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 271 | char maj[MAJMINLEN], min[MAJMINLEN], acc[ACCLEN]; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 272 | |
Pavel Emelyanov | 4efd1a1 | 2008-07-25 01:47:07 -0700 | [diff] [blame] | 273 | rcu_read_lock(); |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 274 | /* |
| 275 | * To preserve the compatibility: |
| 276 | * - Only show the "all devices" when the default policy is to allow |
| 277 | * - List the exceptions in case the default policy is to deny |
| 278 | * This way, the file remains as a "whitelist of devices" |
| 279 | */ |
Aristeu Rozanski | 5b7aa7d | 2012-10-25 13:37:38 -0700 | [diff] [blame] | 280 | if (devcgroup->behavior == DEVCG_DEFAULT_ALLOW) { |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 281 | set_access(acc, DEVCG_ACC_MASK); |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 282 | set_majmin(maj, ~0); |
| 283 | set_majmin(min, ~0); |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 284 | seq_printf(m, "%c %s:%s %s\n", type_to_char(DEVCG_DEV_ALL), |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 285 | maj, min, acc); |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 286 | } else { |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 287 | list_for_each_entry_rcu(ex, &devcgroup->exceptions, list) { |
| 288 | set_access(acc, ex->access); |
| 289 | set_majmin(maj, ex->major); |
| 290 | set_majmin(min, ex->minor); |
| 291 | seq_printf(m, "%c %s:%s %s\n", type_to_char(ex->type), |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 292 | maj, min, acc); |
| 293 | } |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 294 | } |
Pavel Emelyanov | 4efd1a1 | 2008-07-25 01:47:07 -0700 | [diff] [blame] | 295 | rcu_read_unlock(); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 296 | |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 297 | return 0; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 298 | } |
| 299 | |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 300 | /** |
Aristeu Rozanski | f5f3cf6f | 2014-04-24 15:33:21 -0400 | [diff] [blame] | 301 | * match_exception - iterates the exception list trying to find a complete match |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 302 | * @exceptions: list of exceptions |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 303 | * @type: device type (DEVCG_DEV_BLOCK or DEVCG_DEV_CHAR) |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 304 | * @major: device file major number, ~0 to match all |
| 305 | * @minor: device file minor number, ~0 to match all |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 306 | * @access: permission mask (DEVCG_ACC_READ, DEVCG_ACC_WRITE, DEVCG_ACC_MKNOD) |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 307 | * |
Aristeu Rozanski | f5f3cf6f | 2014-04-24 15:33:21 -0400 | [diff] [blame] | 308 | * It is considered a complete match if an exception is found that will |
| 309 | * contain the entire range of provided parameters. |
| 310 | * |
| 311 | * Return: true in case it matches an exception completely |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 312 | */ |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 313 | static bool match_exception(struct list_head *exceptions, short type, |
| 314 | u32 major, u32 minor, short access) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 315 | { |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 316 | struct dev_exception_item *ex; |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 317 | |
| 318 | list_for_each_entry_rcu(ex, exceptions, list) { |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 319 | if ((type & DEVCG_DEV_BLOCK) && !(ex->type & DEVCG_DEV_BLOCK)) |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 320 | continue; |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 321 | if ((type & DEVCG_DEV_CHAR) && !(ex->type & DEVCG_DEV_CHAR)) |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 322 | continue; |
| 323 | if (ex->major != ~0 && ex->major != major) |
| 324 | continue; |
| 325 | if (ex->minor != ~0 && ex->minor != minor) |
| 326 | continue; |
| 327 | /* provided access cannot have more than the exception rule */ |
| 328 | if (access & (~ex->access)) |
| 329 | continue; |
| 330 | return true; |
| 331 | } |
| 332 | return false; |
| 333 | } |
| 334 | |
| 335 | /** |
Aristeu Rozanski | f5f3cf6f | 2014-04-24 15:33:21 -0400 | [diff] [blame] | 336 | * match_exception_partial - iterates the exception list trying to find a partial match |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 337 | * @exceptions: list of exceptions |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 338 | * @type: device type (DEVCG_DEV_BLOCK or DEVCG_DEV_CHAR) |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 339 | * @major: device file major number, ~0 to match all |
| 340 | * @minor: device file minor number, ~0 to match all |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 341 | * @access: permission mask (DEVCG_ACC_READ, DEVCG_ACC_WRITE, DEVCG_ACC_MKNOD) |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 342 | * |
Aristeu Rozanski | f5f3cf6f | 2014-04-24 15:33:21 -0400 | [diff] [blame] | 343 | * It is considered a partial match if an exception's range is found to |
| 344 | * contain *any* of the devices specified by provided parameters. This is |
| 345 | * used to make sure no extra access is being granted that is forbidden by |
| 346 | * any of the exception list. |
| 347 | * |
| 348 | * Return: true in case the provided range mat matches an exception completely |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 349 | */ |
| 350 | static bool match_exception_partial(struct list_head *exceptions, short type, |
| 351 | u32 major, u32 minor, short access) |
| 352 | { |
| 353 | struct dev_exception_item *ex; |
| 354 | |
| 355 | list_for_each_entry_rcu(ex, exceptions, list) { |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 356 | if ((type & DEVCG_DEV_BLOCK) && !(ex->type & DEVCG_DEV_BLOCK)) |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 357 | continue; |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 358 | if ((type & DEVCG_DEV_CHAR) && !(ex->type & DEVCG_DEV_CHAR)) |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 359 | continue; |
| 360 | /* |
| 361 | * We must be sure that both the exception and the provided |
| 362 | * range aren't masking all devices |
| 363 | */ |
| 364 | if (ex->major != ~0 && major != ~0 && ex->major != major) |
| 365 | continue; |
| 366 | if (ex->minor != ~0 && minor != ~0 && ex->minor != minor) |
| 367 | continue; |
| 368 | /* |
| 369 | * In order to make sure the provided range isn't matching |
| 370 | * an exception, all its access bits shouldn't match the |
| 371 | * exception's access bits |
| 372 | */ |
| 373 | if (!(access & ex->access)) |
| 374 | continue; |
| 375 | return true; |
| 376 | } |
| 377 | return false; |
| 378 | } |
| 379 | |
| 380 | /** |
Aristeu Rozanski | f5f3cf6f | 2014-04-24 15:33:21 -0400 | [diff] [blame] | 381 | * verify_new_ex - verifies if a new exception is allowed by parent cgroup's permissions |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 382 | * @dev_cgroup: dev cgroup to be tested against |
| 383 | * @refex: new exception |
| 384 | * @behavior: behavior of the exception's dev_cgroup |
Aristeu Rozanski | f5f3cf6f | 2014-04-24 15:33:21 -0400 | [diff] [blame] | 385 | * |
| 386 | * This is used to make sure a child cgroup won't have more privileges |
| 387 | * than its parent |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 388 | */ |
| 389 | static bool verify_new_ex(struct dev_cgroup *dev_cgroup, |
| 390 | struct dev_exception_item *refex, |
| 391 | enum devcg_behavior behavior) |
| 392 | { |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 393 | bool match = false; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 394 | |
Paul E. McKenney | f78f5b9 | 2015-06-18 15:50:02 -0700 | [diff] [blame] | 395 | RCU_LOCKDEP_WARN(!rcu_read_lock_held() && |
Paul E. McKenney | dc3a04d | 2015-09-02 17:11:22 -0700 | [diff] [blame] | 396 | !lockdep_is_held(&devcgroup_mutex), |
Paul E. McKenney | f78f5b9 | 2015-06-18 15:50:02 -0700 | [diff] [blame] | 397 | "device_cgroup:verify_new_ex called without proper synchronization"); |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 398 | |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 399 | if (dev_cgroup->behavior == DEVCG_DEFAULT_ALLOW) { |
| 400 | if (behavior == DEVCG_DEFAULT_ALLOW) { |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 401 | /* |
| 402 | * new exception in the child doesn't matter, only |
| 403 | * adding extra restrictions |
| 404 | */ |
Aristeu Rozanski | 26898fd | 2013-02-15 11:55:44 -0500 | [diff] [blame] | 405 | return true; |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 406 | } else { |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 407 | /* |
| 408 | * new exception in the child will add more devices |
| 409 | * that can be acessed, so it can't match any of |
| 410 | * parent's exceptions, even slightly |
| 411 | */ |
| 412 | match = match_exception_partial(&dev_cgroup->exceptions, |
| 413 | refex->type, |
| 414 | refex->major, |
| 415 | refex->minor, |
| 416 | refex->access); |
| 417 | |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 418 | if (match) |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 419 | return false; |
| 420 | return true; |
| 421 | } |
Aristeu Rozanski | 26898fd | 2013-02-15 11:55:44 -0500 | [diff] [blame] | 422 | } else { |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 423 | /* |
| 424 | * Only behavior == DEVCG_DEFAULT_DENY allowed here, therefore |
| 425 | * the new exception will add access to more devices and must |
| 426 | * be contained completely in an parent's exception to be |
| 427 | * allowed |
| 428 | */ |
| 429 | match = match_exception(&dev_cgroup->exceptions, refex->type, |
| 430 | refex->major, refex->minor, |
| 431 | refex->access); |
| 432 | |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 433 | if (match) |
| 434 | /* parent has an exception that matches the proposed */ |
Aristeu Rozanski | 26898fd | 2013-02-15 11:55:44 -0500 | [diff] [blame] | 435 | return true; |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 436 | else |
| 437 | return false; |
Aristeu Rozanski | 26898fd | 2013-02-15 11:55:44 -0500 | [diff] [blame] | 438 | } |
| 439 | return false; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 440 | } |
| 441 | |
| 442 | /* |
| 443 | * parent_has_perm: |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 444 | * when adding a new allow rule to a device exception list, the rule |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 445 | * must be allowed in the parent device |
| 446 | */ |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 447 | static int parent_has_perm(struct dev_cgroup *childcg, |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 448 | struct dev_exception_item *ex) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 449 | { |
Tejun Heo | 5c9d535 | 2014-05-16 13:22:48 -0400 | [diff] [blame] | 450 | struct dev_cgroup *parent = css_to_devcgroup(childcg->css.parent); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 451 | |
Tejun Heo | 6387698 | 2013-08-08 20:11:23 -0400 | [diff] [blame] | 452 | if (!parent) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 453 | return 1; |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 454 | return verify_new_ex(parent, ex, childcg->behavior); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 455 | } |
| 456 | |
Aristeu Rozanski | 4cef729 | 2012-10-25 13:37:45 -0700 | [diff] [blame] | 457 | /** |
Aristeu Rozanski | d2c2b11 | 2014-05-05 11:18:59 -0400 | [diff] [blame] | 458 | * parent_allows_removal - verify if it's ok to remove an exception |
| 459 | * @childcg: child cgroup from where the exception will be removed |
| 460 | * @ex: exception being removed |
| 461 | * |
| 462 | * When removing an exception in cgroups with default ALLOW policy, it must |
| 463 | * be checked if removing it will give the child cgroup more access than the |
| 464 | * parent. |
| 465 | * |
| 466 | * Return: true if it's ok to remove exception, false otherwise |
| 467 | */ |
| 468 | static bool parent_allows_removal(struct dev_cgroup *childcg, |
| 469 | struct dev_exception_item *ex) |
| 470 | { |
Tejun Heo | 5c9d535 | 2014-05-16 13:22:48 -0400 | [diff] [blame] | 471 | struct dev_cgroup *parent = css_to_devcgroup(childcg->css.parent); |
Aristeu Rozanski | d2c2b11 | 2014-05-05 11:18:59 -0400 | [diff] [blame] | 472 | |
| 473 | if (!parent) |
| 474 | return true; |
| 475 | |
| 476 | /* It's always allowed to remove access to devices */ |
| 477 | if (childcg->behavior == DEVCG_DEFAULT_DENY) |
| 478 | return true; |
| 479 | |
| 480 | /* |
| 481 | * Make sure you're not removing part or a whole exception existing in |
| 482 | * the parent cgroup |
| 483 | */ |
| 484 | return !match_exception_partial(&parent->exceptions, ex->type, |
| 485 | ex->major, ex->minor, ex->access); |
| 486 | } |
| 487 | |
| 488 | /** |
Aristeu Rozanski | 4cef729 | 2012-10-25 13:37:45 -0700 | [diff] [blame] | 489 | * may_allow_all - checks if it's possible to change the behavior to |
| 490 | * allow based on parent's rules. |
| 491 | * @parent: device cgroup's parent |
| 492 | * returns: != 0 in case it's allowed, 0 otherwise |
| 493 | */ |
| 494 | static inline int may_allow_all(struct dev_cgroup *parent) |
| 495 | { |
Aristeu Rozanski | 64e1047 | 2012-11-06 07:25:04 -0800 | [diff] [blame] | 496 | if (!parent) |
| 497 | return 1; |
Aristeu Rozanski | 4cef729 | 2012-10-25 13:37:45 -0700 | [diff] [blame] | 498 | return parent->behavior == DEVCG_DEFAULT_ALLOW; |
| 499 | } |
| 500 | |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 501 | /** |
| 502 | * revalidate_active_exceptions - walks through the active exception list and |
| 503 | * revalidates the exceptions based on parent's |
| 504 | * behavior and exceptions. The exceptions that |
| 505 | * are no longer valid will be removed. |
| 506 | * Called with devcgroup_mutex held. |
| 507 | * @devcg: cgroup which exceptions will be checked |
| 508 | * |
| 509 | * This is one of the three key functions for hierarchy implementation. |
| 510 | * This function is responsible for re-evaluating all the cgroup's active |
| 511 | * exceptions due to a parent's exception change. |
Mauro Carvalho Chehab | ec15872 | 2018-05-08 18:54:36 -0300 | [diff] [blame] | 512 | * Refer to Documentation/cgroup-v1/devices.txt for more details. |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 513 | */ |
| 514 | static void revalidate_active_exceptions(struct dev_cgroup *devcg) |
| 515 | { |
| 516 | struct dev_exception_item *ex; |
| 517 | struct list_head *this, *tmp; |
| 518 | |
| 519 | list_for_each_safe(this, tmp, &devcg->exceptions) { |
| 520 | ex = container_of(this, struct dev_exception_item, list); |
| 521 | if (!parent_has_perm(devcg, ex)) |
| 522 | dev_exception_rm(devcg, ex); |
| 523 | } |
| 524 | } |
| 525 | |
| 526 | /** |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 527 | * propagate_exception - propagates a new exception to the children |
| 528 | * @devcg_root: device cgroup that added a new exception |
| 529 | * @ex: new exception to be propagated |
| 530 | * |
| 531 | * returns: 0 in case of success, != 0 in case of error |
| 532 | */ |
| 533 | static int propagate_exception(struct dev_cgroup *devcg_root, |
| 534 | struct dev_exception_item *ex) |
| 535 | { |
Tejun Heo | 492eb21 | 2013-08-08 20:11:25 -0400 | [diff] [blame] | 536 | struct cgroup_subsys_state *pos; |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 537 | int rc = 0; |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 538 | |
Tejun Heo | d591fb5 | 2013-05-24 10:55:38 +0900 | [diff] [blame] | 539 | rcu_read_lock(); |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 540 | |
Tejun Heo | 492eb21 | 2013-08-08 20:11:25 -0400 | [diff] [blame] | 541 | css_for_each_descendant_pre(pos, &devcg_root->css) { |
| 542 | struct dev_cgroup *devcg = css_to_devcgroup(pos); |
Tejun Heo | d591fb5 | 2013-05-24 10:55:38 +0900 | [diff] [blame] | 543 | |
| 544 | /* |
| 545 | * Because devcgroup_mutex is held, no devcg will become |
| 546 | * online or offline during the tree walk (see on/offline |
| 547 | * methods), and online ones are safe to access outside RCU |
| 548 | * read lock without bumping refcnt. |
| 549 | */ |
Tejun Heo | bd8815a | 2013-08-08 20:11:27 -0400 | [diff] [blame] | 550 | if (pos == &devcg_root->css || !is_devcg_online(devcg)) |
Tejun Heo | d591fb5 | 2013-05-24 10:55:38 +0900 | [diff] [blame] | 551 | continue; |
| 552 | |
| 553 | rcu_read_unlock(); |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 554 | |
| 555 | /* |
| 556 | * in case both root's behavior and devcg is allow, a new |
| 557 | * restriction means adding to the exception list |
| 558 | */ |
| 559 | if (devcg_root->behavior == DEVCG_DEFAULT_ALLOW && |
| 560 | devcg->behavior == DEVCG_DEFAULT_ALLOW) { |
| 561 | rc = dev_exception_add(devcg, ex); |
| 562 | if (rc) |
| 563 | break; |
| 564 | } else { |
| 565 | /* |
| 566 | * in the other possible cases: |
| 567 | * root's behavior: allow, devcg's: deny |
| 568 | * root's behavior: deny, devcg's: deny |
| 569 | * the exception will be removed |
| 570 | */ |
| 571 | dev_exception_rm(devcg, ex); |
| 572 | } |
| 573 | revalidate_active_exceptions(devcg); |
| 574 | |
Tejun Heo | d591fb5 | 2013-05-24 10:55:38 +0900 | [diff] [blame] | 575 | rcu_read_lock(); |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 576 | } |
Tejun Heo | d591fb5 | 2013-05-24 10:55:38 +0900 | [diff] [blame] | 577 | |
| 578 | rcu_read_unlock(); |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 579 | return rc; |
| 580 | } |
| 581 | |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 582 | /* |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 583 | * Modify the exception list using allow/deny rules. |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 584 | * CAP_SYS_ADMIN is needed for this. It's at least separate from CAP_MKNOD |
| 585 | * so we can give a container CAP_MKNOD to let it create devices but not |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 586 | * modify the exception list. |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 587 | * It seems likely we'll want to add a CAP_CONTAINER capability to allow |
| 588 | * us to also grant CAP_SYS_ADMIN to containers without giving away the |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 589 | * device exception list controls, but for now we'll stick with CAP_SYS_ADMIN |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 590 | * |
| 591 | * Taking rules away is always allowed (given CAP_SYS_ADMIN). Granting |
| 592 | * new access is only allowed if you're in the top-level cgroup, or your |
| 593 | * parent cgroup has the access you're asking for. |
| 594 | */ |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 595 | static int devcgroup_update_access(struct dev_cgroup *devcgroup, |
Tejun Heo | 4d3bb51 | 2014-03-19 10:23:54 -0400 | [diff] [blame] | 596 | int filetype, char *buffer) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 597 | { |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 598 | const char *b; |
Aristeu Rozanski | 26fd840 | 2012-10-25 13:37:41 -0700 | [diff] [blame] | 599 | char temp[12]; /* 11 + 1 characters needed for a u32 */ |
Aristeu Rozanski | c39a2a3 | 2013-02-15 11:55:45 -0500 | [diff] [blame] | 600 | int count, rc = 0; |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 601 | struct dev_exception_item ex; |
Tejun Heo | 5c9d535 | 2014-05-16 13:22:48 -0400 | [diff] [blame] | 602 | struct dev_cgroup *parent = css_to_devcgroup(devcgroup->css.parent); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 603 | |
| 604 | if (!capable(CAP_SYS_ADMIN)) |
| 605 | return -EPERM; |
| 606 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 607 | memset(&ex, 0, sizeof(ex)); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 608 | b = buffer; |
| 609 | |
| 610 | switch (*b) { |
| 611 | case 'a': |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 612 | switch (filetype) { |
| 613 | case DEVCG_ALLOW: |
Tejun Heo | 7a3bb24 | 2014-05-16 13:22:52 -0400 | [diff] [blame] | 614 | if (css_has_online_children(&devcgroup->css)) |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 615 | return -EINVAL; |
| 616 | |
Aristeu Rozanski | 4cef729 | 2012-10-25 13:37:45 -0700 | [diff] [blame] | 617 | if (!may_allow_all(parent)) |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 618 | return -EPERM; |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 619 | dev_exception_clean(devcgroup); |
Aristeu Rozanski | 64e1047 | 2012-11-06 07:25:04 -0800 | [diff] [blame] | 620 | devcgroup->behavior = DEVCG_DEFAULT_ALLOW; |
| 621 | if (!parent) |
| 622 | break; |
| 623 | |
Aristeu Rozanski | 4cef729 | 2012-10-25 13:37:45 -0700 | [diff] [blame] | 624 | rc = dev_exceptions_copy(&devcgroup->exceptions, |
| 625 | &parent->exceptions); |
| 626 | if (rc) |
| 627 | return rc; |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 628 | break; |
| 629 | case DEVCG_DENY: |
Tejun Heo | 7a3bb24 | 2014-05-16 13:22:52 -0400 | [diff] [blame] | 630 | if (css_has_online_children(&devcgroup->css)) |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 631 | return -EINVAL; |
| 632 | |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 633 | dev_exception_clean(devcgroup); |
Aristeu Rozanski | 5b7aa7d | 2012-10-25 13:37:38 -0700 | [diff] [blame] | 634 | devcgroup->behavior = DEVCG_DEFAULT_DENY; |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 635 | break; |
| 636 | default: |
| 637 | return -EINVAL; |
| 638 | } |
| 639 | return 0; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 640 | case 'b': |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 641 | ex.type = DEVCG_DEV_BLOCK; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 642 | break; |
| 643 | case 'c': |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 644 | ex.type = DEVCG_DEV_CHAR; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 645 | break; |
| 646 | default: |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 647 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 648 | } |
| 649 | b++; |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 650 | if (!isspace(*b)) |
| 651 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 652 | b++; |
| 653 | if (*b == '*') { |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 654 | ex.major = ~0; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 655 | b++; |
| 656 | } else if (isdigit(*b)) { |
Aristeu Rozanski | 26fd840 | 2012-10-25 13:37:41 -0700 | [diff] [blame] | 657 | memset(temp, 0, sizeof(temp)); |
| 658 | for (count = 0; count < sizeof(temp) - 1; count++) { |
| 659 | temp[count] = *b; |
| 660 | b++; |
| 661 | if (!isdigit(*b)) |
| 662 | break; |
| 663 | } |
| 664 | rc = kstrtou32(temp, 10, &ex.major); |
| 665 | if (rc) |
| 666 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 667 | } else { |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 668 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 669 | } |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 670 | if (*b != ':') |
| 671 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 672 | b++; |
| 673 | |
| 674 | /* read minor */ |
| 675 | if (*b == '*') { |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 676 | ex.minor = ~0; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 677 | b++; |
| 678 | } else if (isdigit(*b)) { |
Aristeu Rozanski | 26fd840 | 2012-10-25 13:37:41 -0700 | [diff] [blame] | 679 | memset(temp, 0, sizeof(temp)); |
| 680 | for (count = 0; count < sizeof(temp) - 1; count++) { |
| 681 | temp[count] = *b; |
| 682 | b++; |
| 683 | if (!isdigit(*b)) |
| 684 | break; |
| 685 | } |
| 686 | rc = kstrtou32(temp, 10, &ex.minor); |
| 687 | if (rc) |
| 688 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 689 | } else { |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 690 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 691 | } |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 692 | if (!isspace(*b)) |
| 693 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 694 | for (b++, count = 0; count < 3; count++, b++) { |
| 695 | switch (*b) { |
| 696 | case 'r': |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 697 | ex.access |= DEVCG_ACC_READ; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 698 | break; |
| 699 | case 'w': |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 700 | ex.access |= DEVCG_ACC_WRITE; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 701 | break; |
| 702 | case 'm': |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 703 | ex.access |= DEVCG_ACC_MKNOD; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 704 | break; |
| 705 | case '\n': |
| 706 | case '\0': |
| 707 | count = 3; |
| 708 | break; |
| 709 | default: |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 710 | return -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 711 | } |
| 712 | } |
| 713 | |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 714 | switch (filetype) { |
| 715 | case DEVCG_ALLOW: |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 716 | /* |
| 717 | * If the default policy is to allow by default, try to remove |
| 718 | * an matching exception instead. And be silent about it: we |
| 719 | * don't want to break compatibility |
| 720 | */ |
Aristeu Rozanski | 5b7aa7d | 2012-10-25 13:37:38 -0700 | [diff] [blame] | 721 | if (devcgroup->behavior == DEVCG_DEFAULT_ALLOW) { |
Aristeu Rozanski | d2c2b11 | 2014-05-05 11:18:59 -0400 | [diff] [blame] | 722 | /* Check if the parent allows removing it first */ |
| 723 | if (!parent_allows_removal(devcgroup, &ex)) |
| 724 | return -EPERM; |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 725 | dev_exception_rm(devcgroup, &ex); |
Aristeu Rozanski | d2c2b11 | 2014-05-05 11:18:59 -0400 | [diff] [blame] | 726 | break; |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 727 | } |
Aristeu Rozanski | d2c2b11 | 2014-05-05 11:18:59 -0400 | [diff] [blame] | 728 | |
| 729 | if (!parent_has_perm(devcgroup, &ex)) |
| 730 | return -EPERM; |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 731 | rc = dev_exception_add(devcgroup, &ex); |
| 732 | break; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 733 | case DEVCG_DENY: |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 734 | /* |
| 735 | * If the default policy is to deny by default, try to remove |
| 736 | * an matching exception instead. And be silent about it: we |
| 737 | * don't want to break compatibility |
| 738 | */ |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 739 | if (devcgroup->behavior == DEVCG_DEFAULT_DENY) |
Aristeu Rozanski | db9aeca | 2012-10-04 17:15:20 -0700 | [diff] [blame] | 740 | dev_exception_rm(devcgroup, &ex); |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 741 | else |
| 742 | rc = dev_exception_add(devcgroup, &ex); |
| 743 | |
| 744 | if (rc) |
| 745 | break; |
| 746 | /* we only propagate new restrictions */ |
| 747 | rc = propagate_exception(devcgroup, &ex); |
| 748 | break; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 749 | default: |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 750 | rc = -EINVAL; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 751 | } |
Aristeu Rozanski | bd2953e | 2013-02-15 11:55:47 -0500 | [diff] [blame] | 752 | return rc; |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 753 | } |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 754 | |
Tejun Heo | 451af50 | 2014-05-13 12:16:21 -0400 | [diff] [blame] | 755 | static ssize_t devcgroup_access_write(struct kernfs_open_file *of, |
| 756 | char *buf, size_t nbytes, loff_t off) |
Paul Menage | f92523e | 2008-07-25 01:47:03 -0700 | [diff] [blame] | 757 | { |
| 758 | int retval; |
Li Zefan | b4046f0 | 2009-04-02 16:57:32 -0700 | [diff] [blame] | 759 | |
| 760 | mutex_lock(&devcgroup_mutex); |
Tejun Heo | 451af50 | 2014-05-13 12:16:21 -0400 | [diff] [blame] | 761 | retval = devcgroup_update_access(css_to_devcgroup(of_css(of)), |
| 762 | of_cft(of)->private, strstrip(buf)); |
Li Zefan | b4046f0 | 2009-04-02 16:57:32 -0700 | [diff] [blame] | 763 | mutex_unlock(&devcgroup_mutex); |
Tejun Heo | 451af50 | 2014-05-13 12:16:21 -0400 | [diff] [blame] | 764 | return retval ?: nbytes; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 765 | } |
| 766 | |
| 767 | static struct cftype dev_cgroup_files[] = { |
| 768 | { |
| 769 | .name = "allow", |
Tejun Heo | 451af50 | 2014-05-13 12:16:21 -0400 | [diff] [blame] | 770 | .write = devcgroup_access_write, |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 771 | .private = DEVCG_ALLOW, |
| 772 | }, |
| 773 | { |
| 774 | .name = "deny", |
Tejun Heo | 451af50 | 2014-05-13 12:16:21 -0400 | [diff] [blame] | 775 | .write = devcgroup_access_write, |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 776 | .private = DEVCG_DENY, |
| 777 | }, |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 778 | { |
| 779 | .name = "list", |
Tejun Heo | 2da8ca8 | 2013-12-05 12:28:04 -0500 | [diff] [blame] | 780 | .seq_show = devcgroup_seq_show, |
Serge E. Hallyn | 29486df | 2008-04-29 01:00:14 -0700 | [diff] [blame] | 781 | .private = DEVCG_LIST, |
| 782 | }, |
Tejun Heo | 4baf6e3 | 2012-04-01 12:09:55 -0700 | [diff] [blame] | 783 | { } /* terminate */ |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 784 | }; |
| 785 | |
Tejun Heo | 073219e | 2014-02-08 10:36:58 -0500 | [diff] [blame] | 786 | struct cgroup_subsys devices_cgrp_subsys = { |
Tejun Heo | 92fb974 | 2012-11-19 08:13:38 -0800 | [diff] [blame] | 787 | .css_alloc = devcgroup_css_alloc, |
| 788 | .css_free = devcgroup_css_free, |
Aristeu Rozanski | 1909554c | 2013-02-15 11:55:46 -0500 | [diff] [blame] | 789 | .css_online = devcgroup_online, |
| 790 | .css_offline = devcgroup_offline, |
Tejun Heo | 5577964 | 2014-07-15 11:05:09 -0400 | [diff] [blame] | 791 | .legacy_cftypes = dev_cgroup_files, |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 792 | }; |
| 793 | |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 794 | /** |
| 795 | * __devcgroup_check_permission - checks if an inode operation is permitted |
| 796 | * @dev_cgroup: the dev cgroup to be tested against |
| 797 | * @type: device type |
| 798 | * @major: device major number |
| 799 | * @minor: device minor number |
Roman Gushchin | 67e306f | 2017-11-05 08:15:30 -0500 | [diff] [blame] | 800 | * @access: combination of DEVCG_ACC_WRITE, DEVCG_ACC_READ and DEVCG_ACC_MKNOD |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 801 | * |
| 802 | * returns 0 on success, -EPERM case the operation is not permitted |
| 803 | */ |
Roman Gushchin | ecf8fec | 2017-11-05 08:15:31 -0500 | [diff] [blame] | 804 | int __devcgroup_check_permission(short type, u32 major, u32 minor, |
| 805 | short access) |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 806 | { |
Jiri Slaby | 8c9506d | 2012-10-25 13:37:34 -0700 | [diff] [blame] | 807 | struct dev_cgroup *dev_cgroup; |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 808 | bool rc; |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 809 | |
Pavel Emelyanov | 4efd1a1 | 2008-07-25 01:47:07 -0700 | [diff] [blame] | 810 | rcu_read_lock(); |
Jiri Slaby | 8c9506d | 2012-10-25 13:37:34 -0700 | [diff] [blame] | 811 | dev_cgroup = task_devcgroup(current); |
Aristeu Rozanski | 79d7197 | 2014-04-21 12:13:03 -0400 | [diff] [blame] | 812 | if (dev_cgroup->behavior == DEVCG_DEFAULT_ALLOW) |
| 813 | /* Can't match any of the exceptions, even partially */ |
| 814 | rc = !match_exception_partial(&dev_cgroup->exceptions, |
| 815 | type, major, minor, access); |
| 816 | else |
| 817 | /* Need to match completely one exception to be allowed */ |
| 818 | rc = match_exception(&dev_cgroup->exceptions, type, major, |
| 819 | minor, access); |
Pavel Emelyanov | 4efd1a1 | 2008-07-25 01:47:07 -0700 | [diff] [blame] | 820 | rcu_read_unlock(); |
Serge E. Hallyn | 08ce5f1 | 2008-04-29 01:00:10 -0700 | [diff] [blame] | 821 | |
Aristeu Rozanski | ad67607 | 2012-10-04 17:15:17 -0700 | [diff] [blame] | 822 | if (!rc) |
| 823 | return -EPERM; |
| 824 | |
| 825 | return 0; |
| 826 | } |