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. Thanks, Miklos -- 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
