Re: [PATCH] fs: fix: second lock in function d_prune_aliases().

2020-12-30 Thread Matthew Wilcox
On Wed, Dec 30, 2020 at 08:04:49PM +, Al Viro wrote: > On Wed, Dec 30, 2020 at 03:01:25PM +0800, YANG LI wrote: > > Goto statement jumping will cause lock to be executed again without > > executing unlock, placing the lock statement in front of goto > > label to fix this problem. > > > >

Re: [PATCH] fs: fix: second lock in function d_prune_aliases().

2020-12-30 Thread Al Viro
On Wed, Dec 30, 2020 at 03:01:25PM +0800, YANG LI wrote: > Goto statement jumping will cause lock to be executed again without > executing unlock, placing the lock statement in front of goto > label to fix this problem. > > Signed-off-by: YANG LI > Reported-by: Abaci I am sorry, but have you

Re: [PATCH] fs: fix: second lock in function d_prune_aliases().

2020-12-30 Thread Li, Hao
On 2020/12/30 15:01, YANG LI wrote: > Goto statement jumping will cause lock to be executed again without > executing unlock, placing the lock statement in front of goto > label to fix this problem. > > Signed-off-by: YANG LI > Reported-by: Abaci > --- >  fs/dcache.c | 2 +- >  1 file changed, 1

[PATCH] fs: fix: second lock in function d_prune_aliases().

2020-12-29 Thread YANG LI
Goto statement jumping will cause lock to be executed again without executing unlock, placing the lock statement in front of goto label to fix this problem. Signed-off-by: YANG LI Reported-by: Abaci --- fs/dcache.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/fs/dcache.c