On Wed, Aug 06, 2025 at 08:07:51PM +0100, Al Viro wrote:
On Thu, Aug 07, 2025 at 03:55:23AM +1000, Aleksa Sarai wrote:
goto err_free;- ret = -EFAULT;
- if (copy_to_user(_buf, p, n) != 0)
goto err_free;
- if (copy_to_user(_buf, p, n))
n = -EFAULT;
-err_free: if (need_free) kfree(p); return ret;
Minor nit: seeing that there's only one path to that return, I would rather turn it into return n; and dropped the assignment to ret a few lines above. Anyway, that's trivially done when applying...
Anyway, who's carrying fscontext-related stuff this cycle? I've got a short series in that area, but there won't be much from me around there - a plenty of tree-in-dcache stuff, quite a bit of mount-related work, etc., but not a lot around the options-parsing machinery.
Christian, do you have any plans around that area?
I've got a tree for that already and have applied related stuff there. I've fixed up the comments from this thread.