Date: Fri, 7 Mar 2014 23:01:35 +0000 (UTC) From: Alan Somers <asomers@FreeBSD.org> To: src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org Subject: svn commit: r262912 - head/cddl/contrib/opensolaris/lib/libuutil/common Message-ID: <201403072301.s27N1ZF3096726@svn.freebsd.org>
next in thread | raw e-mail | index | archive | help
Author: asomers Date: Fri Mar 7 23:01:35 2014 New Revision: 262912 URL: http://svnweb.freebsd.org/changeset/base/262912 Log: cddl/contrib/opensolaris/lib/libuutil/common/uu_avl.c Fix a memory leak in uu_avl_pool_create: pthread_mutex_init without a corresponding pthread_mutex_destroy. It shows up, among other places, when doing "zfs list". MFC after: 3 weeks Sponsored by: Spectra Logic Corporation Modified: head/cddl/contrib/opensolaris/lib/libuutil/common/uu_avl.c Modified: head/cddl/contrib/opensolaris/lib/libuutil/common/uu_avl.c ============================================================================== --- head/cddl/contrib/opensolaris/lib/libuutil/common/uu_avl.c Fri Mar 7 22:53:29 2014 (r262911) +++ head/cddl/contrib/opensolaris/lib/libuutil/common/uu_avl.c Fri Mar 7 23:01:35 2014 (r262912) @@ -128,6 +128,7 @@ uu_avl_pool_destroy(uu_avl_pool_t *pp) pp->uap_next->uap_prev = pp->uap_prev; pp->uap_prev->uap_next = pp->uap_next; (void) pthread_mutex_unlock(&uu_apool_list_lock); + (void) pthread_mutex_destroy(&pp->uap_lock); pp->uap_prev = NULL; pp->uap_next = NULL; uu_free(pp);
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201403072301.s27N1ZF3096726>