From: Jeffrey Altman <jaltman@secure-endpoints.com>
Date: Wed, 30 Dec 2009 11:32:58 +0000 (+0000)
Subject: Unix CM: Use xdr_free to free memory allocated by XDR
X-Git-Tag: openafs-devel-1_5_69~53
X-Git-Url: https://git.openafs.org/?p=openafs.git;a=commitdiff_plain;h=e4eb19cdc03c4f31ed1d4cefcd01163036ed8c6d

Unix CM: Use xdr_free to free memory allocated by XDR

In src/afs/afs_volume.c VL_GetAddrsU() returns memory allocated by the
xdr package. In 1.4 there was no method of freeing this safely. On the
head there is using the xdr_free() function.

FIXES 124937

Change-Id: I72b8333e02e0e58dd16d25e9ed258a2551cdef9d
Reviewed-on: http://gerrit.openafs.org/1047
Reviewed-by: Derrick Brashear <shadow@dementia.org>
Tested-by: Derrick Brashear <shadow@dementia.org>
---

diff --git a/src/afs/afs_volume.c b/src/afs/afs_volume.c
index 387fd7b..43bf72f 100644
--- a/src/afs/afs_volume.c
+++ b/src/afs/afs_volume.c
@@ -1100,8 +1100,8 @@ InstallUVolumeEntry(struct volume *av, struct uvldbentry *ve, int acell,
 		ts = afs_GetServer(addrp, nentries, acell, cellp->fsport,
 				   WRITE_LOCK, &ve->serverNumber[i],
 				   ve->serverUnique[i]);
-		afs_osi_Free(addrs.bulkaddrs_val,
-			     addrs.bulkaddrs_len * sizeof(*addrp));
+		xdr_free(addrs.bulkaddrs_val,
+			 addrs.bulkaddrs_len * sizeof(*addrp));
 	    }
 	}
 	av->serverHost[j] = ts;