diff options
Diffstat (limited to 'fs/overlayfs/inode.c')
-rw-r--r-- | fs/overlayfs/inode.c | 17 |
1 files changed, 4 insertions, 13 deletions
diff --git a/fs/overlayfs/inode.c b/fs/overlayfs/inode.c index 6bcc9dedc342..3b7ed5d2279c 100644 --- a/fs/overlayfs/inode.c +++ b/fs/overlayfs/inode.c | |||
@@ -286,22 +286,13 @@ int ovl_permission(struct inode *inode, int mask) | |||
286 | if (err) | 286 | if (err) |
287 | return err; | 287 | return err; |
288 | 288 | ||
289 | /* No need to do any access on underlying for special files */ | 289 | old_cred = ovl_override_creds(inode->i_sb); |
290 | if (special_file(realinode->i_mode)) | 290 | if (!upperinode && |
291 | return 0; | 291 | !special_file(realinode->i_mode) && mask & MAY_WRITE) { |
292 | |||
293 | /* No need to access underlying for execute */ | ||
294 | mask &= ~MAY_EXEC; | ||
295 | if ((mask & (MAY_READ | MAY_WRITE)) == 0) | ||
296 | return 0; | ||
297 | |||
298 | /* Lower files get copied up, so turn write access into read */ | ||
299 | if (!upperinode && mask & MAY_WRITE) { | ||
300 | mask &= ~(MAY_WRITE | MAY_APPEND); | 292 | mask &= ~(MAY_WRITE | MAY_APPEND); |
293 | /* Make sure mounter can read file for copy up later */ | ||
301 | mask |= MAY_READ; | 294 | mask |= MAY_READ; |
302 | } | 295 | } |
303 | |||
304 | old_cred = ovl_override_creds(inode->i_sb); | ||
305 | err = inode_permission(realinode, mask); | 296 | err = inode_permission(realinode, mask); |
306 | revert_creds(old_cred); | 297 | revert_creds(old_cred); |
307 | 298 | ||