Abort file_remove_privs() for non-reg. files
authorAlexander Lochmann <alexander.lochmann@tu-dortmund.de>
Fri, 14 Dec 2018 10:55:52 +0000 (11:55 +0100)
committerAl Viro <viro@zeniv.linux.org.uk>
Mon, 29 Apr 2019 01:46:57 +0000 (21:46 -0400)
commitf69e749a49353d96af1a293f56b5b56de59c668a
treeccf6a64b148cfc4102dade591f0139a597bbbb34
parentee948837d7fa89127373c139766aacf6b02a9225
Abort file_remove_privs() for non-reg. files

file_remove_privs() might be called for non-regular files, e.g.
blkdev inode. There is no reason to do its job on things
like blkdev inodes, pipes, or cdevs. Hence, abort if
file does not refer to a regular inode.

AV: more to the point, for devices there might be any number of
inodes refering to given device.  Which one to strip the permissions
from, even if that made any sense in the first place?  All of them
will be observed with contents modified, after all.

Found by LockDoc (Alexander Lochmann, Horst Schirmeier and Olaf
Spinczyk)

Reviewed-by: Jan Kara <jack@suse.cz>
Signed-off-by: Alexander Lochmann <alexander.lochmann@tu-dortmund.de>
Signed-off-by: Horst Schirmeier <horst.schirmeier@tu-dortmund.de>
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
fs/inode.c