svn commit: r189706 - head/sys/ufs/ffs

Konstantin Belousov kib at FreeBSD.org
Wed Mar 11 12:45:54 PDT 2009


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);
 	}


More information about the svn-src-all mailing list