do_last(): don't let a bogus return value from ->open() et.al. to confuse us
authorAl Viro <viro@zeniv.linux.org.uk>
Sun, 28 Feb 2016 00:17:33 +0000 (19:17 -0500)
committerSasha Levin <sasha.levin@oracle.com>
Wed, 9 Mar 2016 18:15:12 +0000 (13:15 -0500)
commit3960cde3e356057bd60adce1b625a7d178b9581c
tree1ff7adf66b38e6302d0690478281792c5d979efa
parent6cb69cb2840d3c18a469d6c42dbdb060bdf2bb04
do_last(): don't let a bogus return value from ->open() et.al. to confuse us

[ Upstream commit c80567c82ae4814a41287618e315a60ecf513be6 ]

... into returning a positive to path_openat(), which would interpret that
as "symlink had been encountered" and proceed to corrupt memory, etc.
It can only happen due to a bug in some ->open() instance or in some LSM
hook, etc., so we report any such event *and* make sure it doesn't trick
us into further unpleasantness.

Cc: stable@vger.kernel.org # v3.6+, at least
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
fs/namei.c