On Thu, May 18, 2017 at 5:13 PM, Miklos Szeredi <miklos@xxxxxxxxxx> wrote: > On Thu, May 18, 2017 at 3:57 PM, Amir Goldstein <amir73il@xxxxxxxxx> wrote: >> On Thu, May 18, 2017 at 4:34 PM, Arnd Bergmann <arnd@xxxxxxxx> wrote: >>> We try to clean up two objects before they got initialized: >>> >>> fs/overlayfs/dir.c: In function 'ovl_rename': >>> fs/overlayfs/dir.c:1060:2: error: 'newdentry' may be used uninitialized in this function [-Werror=maybe-uninitialized] >>> fs/overlayfs/dir.c:891:17: note: 'newdentry' was declared here >>> fs/overlayfs/dir.c:1062:2: error: 'olddentry' may be used uninitialized in this function [-Werror=maybe-uninitialized] >>> fs/overlayfs/dir.c:890:17: note: 'olddentry' was declared here >>> >>> Assuming that no special cleanup is required for the new code, the correct >>> branch target should be "out_revert_creds" to clean up everything that >>> got initialized up to then. >>> >> >> Yep, this is correct. You are the 4th to send this patch :) >> Thanks > > Pushed to overlayfs-next together with other fixes. Sorry for the delay. > Heh! do you have "undo protection" on your git push ;) ? It's taking the bits time to travel... -- To unsubscribe from this list: send the line "unsubscribe linux-unionfs" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
