Skip site navigation (1)Skip section navigation (2)
Date:      Wed, 11 Mar 2009 19:45:52 +0000 (UTC)
From:      Konstantin Belousov <kib@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-head@freebsd.org
Subject:   svn commit: r189706 - head/sys/ufs/ffs
Message-ID:  <200903111945.n2BJjrHZ005166@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kib
Date: Wed Mar 11 19:45:52 2009
New Revision: 189706
URL: http://svn.freebsd.org/changeset/base/189706

Log:
  Do not double-free the struct inode when insmntque failed. Default
  insmntque destructor reclaims the vnode, and ufs_reclaim frees the memory.
  
  Reviewed by:	tegge
  MFC after:	3 days

Modified:
  head/sys/ufs/ffs/ffs_vfsops.c

Modified: head/sys/ufs/ffs/ffs_vfsops.c
==============================================================================
--- head/sys/ufs/ffs/ffs_vfsops.c	Wed Mar 11 17:46:01 2009	(r189705)
+++ head/sys/ufs/ffs/ffs_vfsops.c	Wed Mar 11 19:45:52 2009	(r189706)
@@ -1464,7 +1464,6 @@ ffs_vgetf(mp, ino, flags, vpp, ffs_flags
 		vp->v_vflag |= VV_FORCEINSMQ;
 	error = insmntque(vp, mp);
 	if (error != 0) {
-		uma_zfree(uma_inode, ip);
 		*vpp = NULL;
 		return (error);
 	}



Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?200903111945.n2BJjrHZ005166>