xdr_free(caps, j * sizeof(Capabilities));

This is a bad example; in the patch in question, caps is the array of Capabilities structures allocated for rx_multi. It was allocated using afs_osi_Alloc(), not by XDR, and so should be freed using afs_osi_Free(), as Frank's patch does.

What's more - the "tricky" part is actually freeing the individual Capability_vals as those are what's spit out by the stub. Those must be freed individually, with xdr_free() as was suggested, but not using the above snippet, that's true.

Cheers
 - Felix
_______________________________________________
OpenAFS-devel mailing list
[email protected]
https://lists.openafs.org/mailman/listinfo/openafs-devel

Reply via email to