From Frank Filz <ffilzlnx(a)mindspring.com&gt;: Frank Filz has uploaded this change for review. ( https://review.gerrithub.io/428246 Change subject: nfs4_Compound.c: Document requirements for free_res funcs and sa_cachethis ...................................................................... nfs4_Compound.c: Document requirements for free_res funcs and sa_cachethis We got caught out by a free_res function assuming it would only be called with a valid dynamic data pointer. To hopefully prevent this in the future add some documentation of how the free_res function is used. Change-Id: I883d9c71ca98d68ae06c6070ee6e47d7d4f8ca03 Signed-off-by: Frank S. Filz <ffilzlnx(a)mindspring.com&gt; --- M src/Protocols/NFS/nfs4_Compound.c 1 file changed, 56 insertions(+), 0 deletions(-) git pull ssh://review.gerrithub.io:29418/ffilz/nfs-ganesha refs/changes/46/428246/1 -- To view, visit https://review.gerrithub.io/428246 To unsubscribe, or for help writing mail filters, visit https://review.gerrithub.io/settings Gerrit-Project: ffilz/nfs-ganesha Gerrit-Branch: next Gerrit-MessageType: newchange Gerrit-Change-Id: I883d9c71ca98d68ae06c6070ee6e47d7d4f8ca03 Gerrit-Change-Number: 428246 Gerrit-PatchSet: 1 Gerrit-Owner: Frank Filz <ffilzlnx(a)mindspring.com&gt;