On Mon, Aug 25, 2025 at 03:29:33PM +0200, Christian Brauner wrote: > > - mnt = vfs_create_mount(fc); > > + struct vfsmount *mnt __free(mntput) = vfs_create_mount(fc); > > Ugh, can we please not start declaring variables in the middle of a > scope. Seeing that it *is* the beginning of its scope, what do you suggest? Declaring it above, initializing with NULL and reassigning here? That's actually just as wrong, if not more so - any assignment added to it at earlier point and you've got a silent leak, so verifying correctness would be harder that way.
