Change in ...nfs-ganesha[next]: FSAL_GLUSTER: fix double free of caller_garray when nfs3_create fails
Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/520421 ) Change subject: FSAL_GLUSTER: fix double free of caller_garray when nfs3_create fails ...................................................................... FSAL_GLUSTER: fix double free of caller_garray when nfs3_create fails The create is performed using tmp_fd until the fsal_obj_handle is created, at which point, the tmp_fd is copied into globalfd, but if a subsequent error happened, tmp_fd was closed and then the fsal_obj_handle was released which closed globalfd. Since the two shared one caller_garray, double free. The fix is to change my_fd to refer to globalfd once the copy is accomplished and then ignore tmp_fd. Change-Id: I4ee6ce428895546e6b683017ee74ac2702c6c681 Signed-off-by: Frank S. Filz <ffilzlnx@mindspring.com> --- M src/FSAL/FSAL_GLUSTER/handle.c 1 file changed, 24 insertions(+), 15 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/21/520421/1 -- To view, visit https://review.gerrithub.io/c/ffilz/nfs-ganesha/+/520421 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-Change-Id: I4ee6ce428895546e6b683017ee74ac2702c6c681 Gerrit-Change-Number: 520421 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx@mindspring.com> Gerrit-MessageType: newchange
participants (1)
-
Frank Filz (GerritHub)