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