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