Module Name: src Committed By: njoly Date: Tue Jun 15 17:23:31 UTC 2010
Modified Files: src/sys/rump/librump/rumpvfs: rumpfs.c Log Message: Fix some rump_etfs_register/rump_etfs_remove memory leaks. To generate a diff of this commit: cvs rdiff -u -r1.51 -r1.52 src/sys/rump/librump/rumpvfs/rumpfs.c Please note that diffs are not public domain; they are subject to the copyright notices on the relevant files.
Modified files: Index: src/sys/rump/librump/rumpvfs/rumpfs.c diff -u src/sys/rump/librump/rumpvfs/rumpfs.c:1.51 src/sys/rump/librump/rumpvfs/rumpfs.c:1.52 --- src/sys/rump/librump/rumpvfs/rumpfs.c:1.51 Mon Jun 14 13:40:25 2010 +++ src/sys/rump/librump/rumpvfs/rumpfs.c Tue Jun 15 17:23:31 2010 @@ -1,4 +1,4 @@ -/* $NetBSD: rumpfs.c,v 1.51 2010/06/14 13:40:25 njoly Exp $ */ +/* $NetBSD: rumpfs.c,v 1.52 2010/06/15 17:23:31 njoly Exp $ */ /* * Copyright (c) 2009 Antti Kantee. All Rights Reserved. @@ -26,7 +26,7 @@ */ #include <sys/cdefs.h> -__KERNEL_RCSID(0, "$NetBSD: rumpfs.c,v 1.51 2010/06/14 13:40:25 njoly Exp $"); +__KERNEL_RCSID(0, "$NetBSD: rumpfs.c,v 1.52 2010/06/15 17:23:31 njoly Exp $"); #include <sys/param.h> #include <sys/atomic.h> @@ -325,6 +325,9 @@ mutex_enter(&etfs_lock); if (etfs_find(key, NULL, REGDIR(ftype))) { mutex_exit(&etfs_lock); + if (et->et_rn->rn_hostpath != NULL) + free(et->et_rn->rn_hostpath, M_TEMP); + kmem_free(et->et_rn, sizeof(*et->et_rn)); kmem_free(et, sizeof(*et)); /* XXX: rumpblk_deregister(hostpath); */ return EEXIST; @@ -373,6 +376,9 @@ LIST_FOREACH(et, &etfs_list, et_entries) { if (keylen == et->et_keylen && strcmp(et->et_key, key) == 0) { LIST_REMOVE(et, et_entries); + if (et->et_rn->rn_hostpath != NULL) + free(et->et_rn->rn_hostpath, M_TEMP); + kmem_free(et->et_rn, sizeof(*et->et_rn)); kmem_free(et, sizeof(*et)); break; }