svn commit: r241143 - in head: lib/libc/rpc sys/rpc
Pedro F. Giffuni
pfg at FreeBSD.org
Tue Oct 2 19:10:20 UTC 2012
Author: pfg
Date: Tue Oct 2 19:10:19 2012
New Revision: 241143
URL: http://svn.freebsd.org/changeset/base/241143
Log:
libtirpc: be sure to free cl_netid and cl_tp
When creating a client with clnt_tli_create, it uses strdup to copy
strings for these fields if nconf is passed in. clnt_dg_destroy frees
these strings already. Make sure clnt_vc_destroy frees them in the same
way.
This change matches the reference (OpenSolaris) implementation.
Tested by: David Wolfskill
Obtained from: Bull GNU/Linux NFSv4 Project (libtirpc)
MFC after: 2 weeks
Modified:
head/lib/libc/rpc/clnt_vc.c
head/sys/rpc/clnt_vc.c
Modified: head/lib/libc/rpc/clnt_vc.c
==============================================================================
--- head/lib/libc/rpc/clnt_vc.c Tue Oct 2 19:03:21 2012 (r241142)
+++ head/lib/libc/rpc/clnt_vc.c Tue Oct 2 19:10:19 2012 (r241143)
@@ -672,6 +672,10 @@ clnt_vc_destroy(cl)
if (ct->ct_addr.buf)
free(ct->ct_addr.buf);
mem_free(ct, sizeof(struct ct_data));
+ if (cl->cl_netid && cl->cl_netid[0])
+ mem_free(cl->cl_netid, strlen(cl->cl_netid) +1);
+ if (cl->cl_tp && cl->cl_tp[0])
+ mem_free(cl->cl_tp, strlen(cl->cl_tp) +1);
mem_free(cl, sizeof(CLIENT));
mutex_unlock(&clnt_fd_lock);
thr_sigsetmask(SIG_SETMASK, &(mask), NULL);
Modified: head/sys/rpc/clnt_vc.c
==============================================================================
--- head/sys/rpc/clnt_vc.c Tue Oct 2 19:03:21 2012 (r241142)
+++ head/sys/rpc/clnt_vc.c Tue Oct 2 19:10:19 2012 (r241143)
@@ -836,6 +836,10 @@ clnt_vc_destroy(CLIENT *cl)
soclose(so);
}
mem_free(ct, sizeof(struct ct_data));
+ if (cl->cl_netid && cl->cl_netid[0])
+ mem_free(cl->cl_netid, strlen(cl->cl_netid) +1);
+ if (cl->cl_tp && cl->cl_tp[0])
+ mem_free(cl->cl_tp, strlen(cl->cl_tp) +1);
mem_free(cl, sizeof(CLIENT));
}
More information about the svn-src-all
mailing list