Re: [PATCH] ovl: fix ovl_rename error handling

[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]

 



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
--
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



[Index of Archives]     [Linux Filesystems Devel]     [Linux NFS]     [Linux NILFS]     [Linux USB Devel]     [Linux Audio Users]     [Yosemite News]     [Linux Kernel]     [Linux SCSI]

  Powered by Linux