Re: cachefiles: fix dentry leak in cachefiles_open_file()
From: Markus Elfring
Date: Tue Aug 27 2024 - 04:35:30 EST
>> …
>>> Add the missing dput() to cachefiles_open_file() for a quick fix.
>> I suggest to use a goto chain accordingly.
…
> Thanks for the suggestion, but I think the current solution is simple enough
Yes, of course (in principle).
> that we don't need to add a label to it.
I came along other software design expectations.
https://git.kernel.org/pub/scm/linux/kernel/git/torvalds/linux.git/tree/Documentation/process/coding-style.rst?h=v6.11-rc5#n526
Regards,
Markus