>From <jlay...@redhat.com>: jlay...@redhat.com has uploaded a new change for review.
https://review.gerrithub.io/298665 Change subject: ceph: pass in correct temporary fd variable to ceph_ll_create ...................................................................... ceph: pass in correct temporary fd variable to ceph_ll_create In the UNCHECKED create case, if we get back -EEXIST on the first ceph_ll_create call, we'll call it again without O_EXCL. The second call however has the return Fh pointer set to &my_fd->fd, which gets clobbered just afterward if that call succeeds. I think this is wrong -- we want to pass in &fd there so that my_fd gets set up properly. Change-Id: Ic2167c1664280ea8de2a3dc2ad89769534ce4088 Signed-off-by: Jeff Layton <jlay...@redhat.com> --- M src/FSAL/FSAL_CEPH/handle.c 1 file changed, 1 insertion(+), 1 deletion(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/65/298665/1 -- To view, visit https://review.gerrithub.io/298665 To unsubscribe, visit https://review.gerrithub.io/settings Gerrit-MessageType: newchange Gerrit-Change-Id: Ic2167c1664280ea8de2a3dc2ad89769534ce4088 Gerrit-PatchSet: 1 Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Owner: jlay...@redhat.com ------------------------------------------------------------------------------ Check out the vibrant tech community on one of the world's most engaging tech sites, SlashDot.org! http://sdm.link/slashdot _______________________________________________ Nfs-ganesha-devel mailing list Nfs-ganesha-devel@lists.sourceforge.net https://lists.sourceforge.net/lists/listinfo/nfs-ganesha-devel
