Greg Kroah-Hartman | b244131 | 2017-11-01 15:07:57 +0100 | [diff] [blame] | 1 | // SPDX-License-Identifier: GPL-2.0 |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 2 | /* |
| 3 | * linux/fs/attr.c |
| 4 | * |
| 5 | * Copyright (C) 1991, 1992 Linus Torvalds |
| 6 | * changes by Thomas Schoebel-Theuer |
| 7 | */ |
| 8 | |
Paul Gortmaker | 630d9c4 | 2011-11-16 23:57:37 -0500 | [diff] [blame] | 9 | #include <linux/export.h> |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 10 | #include <linux/time.h> |
| 11 | #include <linux/mm.h> |
| 12 | #include <linux/string.h> |
Ingo Molnar | 3f07c01 | 2017-02-08 18:51:30 +0100 | [diff] [blame] | 13 | #include <linux/sched/signal.h> |
Randy Dunlap | 16f7e0f | 2006-01-11 12:17:46 -0800 | [diff] [blame] | 14 | #include <linux/capability.h> |
Robert Love | 0eeca28 | 2005-07-12 17:06:03 -0400 | [diff] [blame] | 15 | #include <linux/fsnotify.h> |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 16 | #include <linux/fcntl.h> |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 17 | #include <linux/security.h> |
Mimi Zohar | 975d294 | 2011-03-09 14:39:57 -0500 | [diff] [blame] | 18 | #include <linux/evm.h> |
Mimi Zohar | 9957a50 | 2011-03-09 22:57:53 -0500 | [diff] [blame] | 19 | #include <linux/ima.h> |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 20 | |
Eric W. Biederman | 0031181 | 2016-10-15 18:36:59 -0500 | [diff] [blame] | 21 | static bool chown_ok(const struct inode *inode, kuid_t uid) |
| 22 | { |
| 23 | if (uid_eq(current_fsuid(), inode->i_uid) && |
| 24 | uid_eq(uid, inode->i_uid)) |
| 25 | return true; |
| 26 | if (capable_wrt_inode_uidgid(inode, CAP_CHOWN)) |
| 27 | return true; |
| 28 | if (uid_eq(inode->i_uid, INVALID_UID) && |
| 29 | ns_capable(inode->i_sb->s_user_ns, CAP_CHOWN)) |
| 30 | return true; |
| 31 | return false; |
| 32 | } |
| 33 | |
| 34 | static bool chgrp_ok(const struct inode *inode, kgid_t gid) |
| 35 | { |
| 36 | if (uid_eq(current_fsuid(), inode->i_uid) && |
| 37 | (in_group_p(gid) || gid_eq(gid, inode->i_gid))) |
| 38 | return true; |
| 39 | if (capable_wrt_inode_uidgid(inode, CAP_CHOWN)) |
| 40 | return true; |
| 41 | if (gid_eq(inode->i_gid, INVALID_GID) && |
| 42 | ns_capable(inode->i_sb->s_user_ns, CAP_CHOWN)) |
| 43 | return true; |
| 44 | return false; |
| 45 | } |
| 46 | |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 47 | /** |
Jan Kara | 31051c8 | 2016-05-26 16:55:18 +0200 | [diff] [blame] | 48 | * setattr_prepare - check if attribute changes to a dentry are allowed |
| 49 | * @dentry: dentry to check |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 50 | * @attr: attributes to change |
| 51 | * |
| 52 | * Check if we are allowed to change the attributes contained in @attr |
Jan Kara | 31051c8 | 2016-05-26 16:55:18 +0200 | [diff] [blame] | 53 | * in the given dentry. This includes the normal unix access permission |
| 54 | * checks, as well as checks for rlimits and others. The function also clears |
| 55 | * SGID bit from mode if user is not allowed to set it. Also file capabilities |
| 56 | * and IMA extended attributes are cleared if ATTR_KILL_PRIV is set. |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 57 | * |
| 58 | * Should be called as the first thing in ->setattr implementations, |
| 59 | * possibly after taking additional locks. |
| 60 | */ |
Jan Kara | 31051c8 | 2016-05-26 16:55:18 +0200 | [diff] [blame] | 61 | int setattr_prepare(struct dentry *dentry, struct iattr *attr) |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 62 | { |
Jan Kara | 31051c8 | 2016-05-26 16:55:18 +0200 | [diff] [blame] | 63 | struct inode *inode = d_inode(dentry); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 64 | unsigned int ia_valid = attr->ia_valid; |
| 65 | |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 66 | /* |
| 67 | * First check size constraints. These can't be overriden using |
| 68 | * ATTR_FORCE. |
| 69 | */ |
| 70 | if (ia_valid & ATTR_SIZE) { |
| 71 | int error = inode_newsize_ok(inode, attr->ia_size); |
| 72 | if (error) |
| 73 | return error; |
| 74 | } |
| 75 | |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 76 | /* If force is set do it anyway. */ |
| 77 | if (ia_valid & ATTR_FORCE) |
Jan Kara | 030b533 | 2016-05-26 17:21:32 +0200 | [diff] [blame] | 78 | goto kill_priv; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 79 | |
| 80 | /* Make sure a caller can chown. */ |
Eric W. Biederman | 0031181 | 2016-10-15 18:36:59 -0500 | [diff] [blame] | 81 | if ((ia_valid & ATTR_UID) && !chown_ok(inode, attr->ia_uid)) |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 82 | return -EPERM; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 83 | |
| 84 | /* Make sure caller can chgrp. */ |
Eric W. Biederman | 0031181 | 2016-10-15 18:36:59 -0500 | [diff] [blame] | 85 | if ((ia_valid & ATTR_GID) && !chgrp_ok(inode, attr->ia_gid)) |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 86 | return -EPERM; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 87 | |
| 88 | /* Make sure a caller can chmod. */ |
| 89 | if (ia_valid & ATTR_MODE) { |
Serge E. Hallyn | 2e14967 | 2011-03-23 16:43:26 -0700 | [diff] [blame] | 90 | if (!inode_owner_or_capable(inode)) |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 91 | return -EPERM; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 92 | /* Also check the setgid bit! */ |
| 93 | if (!in_group_p((ia_valid & ATTR_GID) ? attr->ia_gid : |
Eric W. Biederman | 7fa294c | 2012-09-02 19:12:51 -0700 | [diff] [blame] | 94 | inode->i_gid) && |
Andy Lutomirski | 23adbe1 | 2014-06-10 12:45:42 -0700 | [diff] [blame] | 95 | !capable_wrt_inode_uidgid(inode, CAP_FSETID)) |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 96 | attr->ia_mode &= ~S_ISGID; |
| 97 | } |
| 98 | |
| 99 | /* Check for setting the inode time. */ |
Miklos Szeredi | 9767d74 | 2008-07-01 15:01:26 +0200 | [diff] [blame] | 100 | if (ia_valid & (ATTR_MTIME_SET | ATTR_ATIME_SET | ATTR_TIMES_SET)) { |
Serge E. Hallyn | 2e14967 | 2011-03-23 16:43:26 -0700 | [diff] [blame] | 101 | if (!inode_owner_or_capable(inode)) |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 102 | return -EPERM; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 103 | } |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 104 | |
Jan Kara | 030b533 | 2016-05-26 17:21:32 +0200 | [diff] [blame] | 105 | kill_priv: |
| 106 | /* User has permission for the change */ |
| 107 | if (ia_valid & ATTR_KILL_PRIV) { |
| 108 | int error; |
| 109 | |
| 110 | error = security_inode_killpriv(dentry); |
| 111 | if (error) |
| 112 | return error; |
| 113 | } |
| 114 | |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 115 | return 0; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 116 | } |
Greg Kroah-Hartman | d483eed | 2020-07-02 12:51:03 +0200 | [diff] [blame] | 117 | EXPORT_SYMBOL_NS(setattr_prepare, ANDROID_GKI_VFS_EXPORT_ONLY); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 118 | |
npiggin@suse.de | 25d9e2d | 2009-08-21 02:35:05 +1000 | [diff] [blame] | 119 | /** |
| 120 | * inode_newsize_ok - may this inode be truncated to a given size |
| 121 | * @inode: the inode to be truncated |
| 122 | * @offset: the new size to assign to the inode |
npiggin@suse.de | 25d9e2d | 2009-08-21 02:35:05 +1000 | [diff] [blame] | 123 | * |
npiggin@suse.de | 7bb46a6 | 2010-05-27 01:05:33 +1000 | [diff] [blame] | 124 | * inode_newsize_ok must be called with i_mutex held. |
| 125 | * |
npiggin@suse.de | 25d9e2d | 2009-08-21 02:35:05 +1000 | [diff] [blame] | 126 | * inode_newsize_ok will check filesystem limits and ulimits to check that the |
| 127 | * new inode size is within limits. inode_newsize_ok will also send SIGXFSZ |
| 128 | * when necessary. Caller must not proceed with inode size change if failure is |
| 129 | * returned. @inode must be a file (not directory), with appropriate |
| 130 | * permissions to allow truncate (inode_newsize_ok does NOT check these |
| 131 | * conditions). |
Matthew Wilcox | 3fae174 | 2018-07-03 08:08:34 -0700 | [diff] [blame] | 132 | * |
| 133 | * Return: 0 on success, -ve errno on failure |
npiggin@suse.de | 25d9e2d | 2009-08-21 02:35:05 +1000 | [diff] [blame] | 134 | */ |
| 135 | int inode_newsize_ok(const struct inode *inode, loff_t offset) |
| 136 | { |
| 137 | if (inode->i_size < offset) { |
| 138 | unsigned long limit; |
| 139 | |
Jiri Slaby | d554ed89 | 2010-03-05 13:42:42 -0800 | [diff] [blame] | 140 | limit = rlimit(RLIMIT_FSIZE); |
npiggin@suse.de | 25d9e2d | 2009-08-21 02:35:05 +1000 | [diff] [blame] | 141 | if (limit != RLIM_INFINITY && offset > limit) |
| 142 | goto out_sig; |
| 143 | if (offset > inode->i_sb->s_maxbytes) |
| 144 | goto out_big; |
| 145 | } else { |
| 146 | /* |
| 147 | * truncation of in-use swapfiles is disallowed - it would |
| 148 | * cause subsequent swapout to scribble on the now-freed |
| 149 | * blocks. |
| 150 | */ |
| 151 | if (IS_SWAPFILE(inode)) |
| 152 | return -ETXTBSY; |
| 153 | } |
| 154 | |
| 155 | return 0; |
| 156 | out_sig: |
| 157 | send_sig(SIGXFSZ, current, 0); |
| 158 | out_big: |
| 159 | return -EFBIG; |
| 160 | } |
Greg Kroah-Hartman | d483eed | 2020-07-02 12:51:03 +0200 | [diff] [blame] | 161 | EXPORT_SYMBOL_NS(inode_newsize_ok, ANDROID_GKI_VFS_EXPORT_ONLY); |
npiggin@suse.de | 25d9e2d | 2009-08-21 02:35:05 +1000 | [diff] [blame] | 162 | |
npiggin@suse.de | 7bb46a6 | 2010-05-27 01:05:33 +1000 | [diff] [blame] | 163 | /** |
Christoph Hellwig | 6a1a90a | 2010-06-04 11:30:00 +0200 | [diff] [blame] | 164 | * setattr_copy - copy simple metadata updates into the generic inode |
npiggin@suse.de | 7bb46a6 | 2010-05-27 01:05:33 +1000 | [diff] [blame] | 165 | * @inode: the inode to be updated |
| 166 | * @attr: the new attributes |
| 167 | * |
Christoph Hellwig | 6a1a90a | 2010-06-04 11:30:00 +0200 | [diff] [blame] | 168 | * setattr_copy must be called with i_mutex held. |
npiggin@suse.de | 7bb46a6 | 2010-05-27 01:05:33 +1000 | [diff] [blame] | 169 | * |
Christoph Hellwig | 6a1a90a | 2010-06-04 11:30:00 +0200 | [diff] [blame] | 170 | * setattr_copy updates the inode's metadata with that specified |
Lucas De Marchi | 25985ed | 2011-03-30 22:57:33 -0300 | [diff] [blame] | 171 | * in attr. Noticeably missing is inode size update, which is more complex |
Christoph Hellwig | 2c27c65 | 2010-06-04 11:30:04 +0200 | [diff] [blame] | 172 | * as it requires pagecache updates. |
npiggin@suse.de | 7bb46a6 | 2010-05-27 01:05:33 +1000 | [diff] [blame] | 173 | * |
| 174 | * The inode is not marked as dirty after this operation. The rationale is |
| 175 | * that for "simple" filesystems, the struct inode is the inode storage. |
| 176 | * The caller is free to mark the inode dirty afterwards if needed. |
| 177 | */ |
Christoph Hellwig | 6a1a90a | 2010-06-04 11:30:00 +0200 | [diff] [blame] | 178 | void setattr_copy(struct inode *inode, const struct iattr *attr) |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 179 | { |
| 180 | unsigned int ia_valid = attr->ia_valid; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 181 | |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 182 | if (ia_valid & ATTR_UID) |
| 183 | inode->i_uid = attr->ia_uid; |
| 184 | if (ia_valid & ATTR_GID) |
| 185 | inode->i_gid = attr->ia_gid; |
Amir Goldstein | eb31e2f | 2019-11-24 21:31:45 +0200 | [diff] [blame] | 186 | if (ia_valid & ATTR_ATIME) |
| 187 | inode->i_atime = attr->ia_atime; |
| 188 | if (ia_valid & ATTR_MTIME) |
| 189 | inode->i_mtime = attr->ia_mtime; |
| 190 | if (ia_valid & ATTR_CTIME) |
| 191 | inode->i_ctime = attr->ia_ctime; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 192 | if (ia_valid & ATTR_MODE) { |
| 193 | umode_t mode = attr->ia_mode; |
| 194 | |
Eric W. Biederman | 7fa294c | 2012-09-02 19:12:51 -0700 | [diff] [blame] | 195 | if (!in_group_p(inode->i_gid) && |
Andy Lutomirski | 23adbe1 | 2014-06-10 12:45:42 -0700 | [diff] [blame] | 196 | !capable_wrt_inode_uidgid(inode, CAP_FSETID)) |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 197 | mode &= ~S_ISGID; |
| 198 | inode->i_mode = mode; |
| 199 | } |
npiggin@suse.de | 7bb46a6 | 2010-05-27 01:05:33 +1000 | [diff] [blame] | 200 | } |
Christoph Hellwig | 6a1a90a | 2010-06-04 11:30:00 +0200 | [diff] [blame] | 201 | EXPORT_SYMBOL(setattr_copy); |
npiggin@suse.de | 7bb46a6 | 2010-05-27 01:05:33 +1000 | [diff] [blame] | 202 | |
J. Bruce Fields | 27ac0ff | 2011-09-20 17:19:26 -0400 | [diff] [blame] | 203 | /** |
| 204 | * notify_change - modify attributes of a filesytem object |
| 205 | * @dentry: object affected |
Matthew Wilcox | 3fae174 | 2018-07-03 08:08:34 -0700 | [diff] [blame] | 206 | * @attr: new attributes |
J. Bruce Fields | 27ac0ff | 2011-09-20 17:19:26 -0400 | [diff] [blame] | 207 | * @delegated_inode: returns inode, if the inode is delegated |
| 208 | * |
| 209 | * The caller must hold the i_mutex on the affected object. |
| 210 | * |
| 211 | * If notify_change discovers a delegation in need of breaking, |
| 212 | * it will return -EWOULDBLOCK and return a reference to the inode in |
| 213 | * delegated_inode. The caller should then break the delegation and |
| 214 | * retry. Because breaking a delegation may take a long time, the |
| 215 | * caller should drop the i_mutex before doing so. |
| 216 | * |
| 217 | * Alternatively, a caller may pass NULL for delegated_inode. This may |
| 218 | * be appropriate for callers that expect the underlying filesystem not |
| 219 | * to be NFS exported. Also, passing NULL is fine for callers holding |
| 220 | * the file open for write, as there can be no conflicting delegation in |
| 221 | * that case. |
| 222 | */ |
Greg Kroah-Hartman | 584c547 | 2020-05-28 16:46:54 +0200 | [diff] [blame] | 223 | int notify_change(struct dentry * dentry, struct iattr * attr, struct inode **delegated_inode) |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 224 | { |
| 225 | struct inode *inode = dentry->d_inode; |
Al Viro | 8d334ac | 2011-07-24 23:21:59 -0400 | [diff] [blame] | 226 | umode_t mode = inode->i_mode; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 227 | int error; |
Deepa Dinamani | 95582b0 | 2018-05-08 19:36:02 -0700 | [diff] [blame] | 228 | struct timespec64 now; |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 229 | unsigned int ia_valid = attr->ia_valid; |
| 230 | |
Al Viro | 5955102 | 2016-01-22 15:40:57 -0500 | [diff] [blame] | 231 | WARN_ON_ONCE(!inode_is_locked(inode)); |
Andrew Morton | c4107b3 | 2012-06-20 09:55:58 +1000 | [diff] [blame] | 232 | |
Miklos Szeredi | beb29e0 | 2008-07-01 15:01:29 +0200 | [diff] [blame] | 233 | if (ia_valid & (ATTR_MODE | ATTR_UID | ATTR_GID | ATTR_TIMES_SET)) { |
| 234 | if (IS_IMMUTABLE(inode) || IS_APPEND(inode)) |
| 235 | return -EPERM; |
| 236 | } |
| 237 | |
Miklos Szeredi | f2b20f6 | 2016-09-16 12:44:20 +0200 | [diff] [blame] | 238 | /* |
| 239 | * If utimes(2) and friends are called with times == NULL (or both |
| 240 | * times are UTIME_NOW), then we need to check for write permission |
| 241 | */ |
| 242 | if (ia_valid & ATTR_TOUCH) { |
| 243 | if (IS_IMMUTABLE(inode)) |
| 244 | return -EPERM; |
| 245 | |
| 246 | if (!inode_owner_or_capable(inode)) { |
Greg Kroah-Hartman | c3271fe | 2020-05-28 16:44:45 +0200 | [diff] [blame] | 247 | error = inode_permission(inode, MAY_WRITE); |
Miklos Szeredi | f2b20f6 | 2016-09-16 12:44:20 +0200 | [diff] [blame] | 248 | if (error) |
| 249 | return error; |
| 250 | } |
| 251 | } |
| 252 | |
Andi Kleen | 69b4573 | 2011-05-28 08:25:51 -0700 | [diff] [blame] | 253 | if ((ia_valid & ATTR_MODE)) { |
Al Viro | 8d334ac | 2011-07-24 23:21:59 -0400 | [diff] [blame] | 254 | umode_t amode = attr->ia_mode; |
Andi Kleen | 69b4573 | 2011-05-28 08:25:51 -0700 | [diff] [blame] | 255 | /* Flag setting protected by i_mutex */ |
| 256 | if (is_sxid(amode)) |
| 257 | inode->i_flags &= ~S_NOSEC; |
| 258 | } |
| 259 | |
Deepa Dinamani | c2050a4 | 2016-09-14 07:48:06 -0700 | [diff] [blame] | 260 | now = current_time(inode); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 261 | |
| 262 | attr->ia_ctime = now; |
| 263 | if (!(ia_valid & ATTR_ATIME_SET)) |
| 264 | attr->ia_atime = now; |
Amir Goldstein | eb31e2f | 2019-11-24 21:31:45 +0200 | [diff] [blame] | 265 | else |
| 266 | attr->ia_atime = timestamp_truncate(attr->ia_atime, inode); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 267 | if (!(ia_valid & ATTR_MTIME_SET)) |
| 268 | attr->ia_mtime = now; |
Amir Goldstein | eb31e2f | 2019-11-24 21:31:45 +0200 | [diff] [blame] | 269 | else |
| 270 | attr->ia_mtime = timestamp_truncate(attr->ia_mtime, inode); |
| 271 | |
Serge E. Hallyn | b537677 | 2007-10-16 23:31:36 -0700 | [diff] [blame] | 272 | if (ia_valid & ATTR_KILL_PRIV) { |
Serge E. Hallyn | b537677 | 2007-10-16 23:31:36 -0700 | [diff] [blame] | 273 | error = security_inode_need_killpriv(dentry); |
Jan Kara | 030b533 | 2016-05-26 17:21:32 +0200 | [diff] [blame] | 274 | if (error < 0) |
Serge E. Hallyn | b537677 | 2007-10-16 23:31:36 -0700 | [diff] [blame] | 275 | return error; |
Jan Kara | 030b533 | 2016-05-26 17:21:32 +0200 | [diff] [blame] | 276 | if (error == 0) |
| 277 | ia_valid = attr->ia_valid &= ~ATTR_KILL_PRIV; |
Serge E. Hallyn | b537677 | 2007-10-16 23:31:36 -0700 | [diff] [blame] | 278 | } |
Jeff Layton | 6de0ec0 | 2007-10-18 03:05:20 -0700 | [diff] [blame] | 279 | |
| 280 | /* |
| 281 | * We now pass ATTR_KILL_S*ID to the lower level setattr function so |
| 282 | * that the function has the ability to reinterpret a mode change |
| 283 | * that's due to these bits. This adds an implicit restriction that |
| 284 | * no function will ever call notify_change with both ATTR_MODE and |
| 285 | * ATTR_KILL_S*ID set. |
| 286 | */ |
| 287 | if ((ia_valid & (ATTR_KILL_SUID|ATTR_KILL_SGID)) && |
| 288 | (ia_valid & ATTR_MODE)) |
| 289 | BUG(); |
| 290 | |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 291 | if (ia_valid & ATTR_KILL_SUID) { |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 292 | if (mode & S_ISUID) { |
Jeff Layton | 6de0ec0 | 2007-10-18 03:05:20 -0700 | [diff] [blame] | 293 | ia_valid = attr->ia_valid |= ATTR_MODE; |
| 294 | attr->ia_mode = (inode->i_mode & ~S_ISUID); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 295 | } |
| 296 | } |
| 297 | if (ia_valid & ATTR_KILL_SGID) { |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 298 | if ((mode & (S_ISGID | S_IXGRP)) == (S_ISGID | S_IXGRP)) { |
| 299 | if (!(ia_valid & ATTR_MODE)) { |
| 300 | ia_valid = attr->ia_valid |= ATTR_MODE; |
| 301 | attr->ia_mode = inode->i_mode; |
| 302 | } |
| 303 | attr->ia_mode &= ~S_ISGID; |
| 304 | } |
| 305 | } |
Jeff Layton | 6de0ec0 | 2007-10-18 03:05:20 -0700 | [diff] [blame] | 306 | if (!(attr->ia_valid & ~(ATTR_KILL_SUID | ATTR_KILL_SGID))) |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 307 | return 0; |
| 308 | |
Seth Forshee | a475acf | 2016-04-26 14:36:25 -0500 | [diff] [blame] | 309 | /* |
| 310 | * Verify that uid/gid changes are valid in the target |
| 311 | * namespace of the superblock. |
| 312 | */ |
| 313 | if (ia_valid & ATTR_UID && |
| 314 | !kuid_has_mapping(inode->i_sb->s_user_ns, attr->ia_uid)) |
| 315 | return -EOVERFLOW; |
| 316 | if (ia_valid & ATTR_GID && |
| 317 | !kgid_has_mapping(inode->i_sb->s_user_ns, attr->ia_gid)) |
| 318 | return -EOVERFLOW; |
| 319 | |
Eric W. Biederman | 0bd23d09 | 2016-06-29 14:54:46 -0500 | [diff] [blame] | 320 | /* Don't allow modifications of files with invalid uids or |
| 321 | * gids unless those uids & gids are being made valid. |
| 322 | */ |
| 323 | if (!(ia_valid & ATTR_UID) && !uid_valid(inode->i_uid)) |
| 324 | return -EOVERFLOW; |
| 325 | if (!(ia_valid & ATTR_GID) && !gid_valid(inode->i_gid)) |
| 326 | return -EOVERFLOW; |
| 327 | |
Miklos Szeredi | a77b72d | 2008-07-30 14:06:22 +0200 | [diff] [blame] | 328 | error = security_inode_setattr(dentry, attr); |
| 329 | if (error) |
| 330 | return error; |
J. Bruce Fields | 27ac0ff | 2011-09-20 17:19:26 -0400 | [diff] [blame] | 331 | error = try_break_deleg(inode, delegated_inode); |
| 332 | if (error) |
| 333 | return error; |
Miklos Szeredi | a77b72d | 2008-07-30 14:06:22 +0200 | [diff] [blame] | 334 | |
Greg Kroah-Hartman | 584c547 | 2020-05-28 16:46:54 +0200 | [diff] [blame] | 335 | if (inode->i_op->setattr) |
Miklos Szeredi | a77b72d | 2008-07-30 14:06:22 +0200 | [diff] [blame] | 336 | error = inode->i_op->setattr(dentry, attr); |
Christoph Hellwig | eef2380 | 2010-06-04 11:30:01 +0200 | [diff] [blame] | 337 | else |
| 338 | error = simple_setattr(dentry, attr); |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 339 | |
Mimi Zohar | 975d294 | 2011-03-09 14:39:57 -0500 | [diff] [blame] | 340 | if (!error) { |
Robert Love | 0eeca28 | 2005-07-12 17:06:03 -0400 | [diff] [blame] | 341 | fsnotify_change(dentry, ia_valid); |
Mimi Zohar | 9957a50 | 2011-03-09 22:57:53 -0500 | [diff] [blame] | 342 | ima_inode_post_setattr(dentry); |
Mimi Zohar | 975d294 | 2011-03-09 14:39:57 -0500 | [diff] [blame] | 343 | evm_inode_post_setattr(dentry, ia_valid); |
| 344 | } |
Robert Love | 0eeca28 | 2005-07-12 17:06:03 -0400 | [diff] [blame] | 345 | |
Linus Torvalds | 1da177e | 2005-04-16 15:20:36 -0700 | [diff] [blame] | 346 | return error; |
| 347 | } |
Greg Kroah-Hartman | d483eed | 2020-07-02 12:51:03 +0200 | [diff] [blame] | 348 | EXPORT_SYMBOL_NS(notify_change, ANDROID_GKI_VFS_EXPORT_ONLY); |