]> www.infradead.org Git - users/jedix/linux-maple.git/commit
fix d_walk()/non-delayed __d_free() race
authorAl Viro <viro@zeniv.linux.org.uk>
Wed, 8 Jun 2016 01:26:55 +0000 (21:26 -0400)
committerDhaval Giani <dhaval.giani@oracle.com>
Fri, 20 Jan 2017 22:21:55 +0000 (17:21 -0500)
commitee427030b288ef69e88a8accaf8cac491806e70d
treec74c63f6f20566fd60bb63cef2e0ff45b38b2c5c
parent545578e8d3d863026da4d1274e0ce34c34e2fdf0
fix d_walk()/non-delayed __d_free() race

Orabug: 25256943

[ Upstream commit 3d56c25e3bb0726a5c5e16fc2d9e38f8ed763085 ]

Ascend-to-parent logics in d_walk() depends on all encountered child
dentries not getting freed without an RCU delay.  Unfortunately, in
quite a few cases it is not true, with hard-to-hit oopsable race as
the result.

Fortunately, the fix is simiple; right now the rule is "if it ever
been hashed, freeing must be delayed" and changing it to "if it
ever had a parent, freeing must be delayed" closes that hole and
covers all cases the old rule used to cover.  Moreover, pipes and
sockets remain _not_ covered, so we do not introduce RCU delay in
the cases which are the reason for having that delay conditional
in the first place.

Cc: stable@vger.kernel.org # v3.2+ (and watch out for __d_materialise_dentry())
Signed-off-by: Al Viro <viro@zeniv.linux.org.uk>
Signed-off-by: Sasha Levin <sasha.levin@oracle.com>
(cherry picked from commit 7296467cfbe04536b8ccec5345a1fd0a2cab1cdc)
Signed-off-by: Dhaval Giani <dhaval.giani@oracle.com>
fs/dcache.c