From a9d9537110686054ec6c7a469f976c5b2c7fd51a Mon Sep 17 00:00:00 2001 From: Konstantin Belousov Date: Wed, 11 Mar 2009 19:45:52 +0000 Subject: [PATCH] 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 --- sys/ufs/ffs/ffs_vfsops.c | 1 - 1 file changed, 1 deletion(-) diff --git a/sys/ufs/ffs/ffs_vfsops.c b/sys/ufs/ffs/ffs_vfsops.c index c6de7df58935..73535fca4980 100644 --- a/sys/ufs/ffs/ffs_vfsops.c +++ b/sys/ufs/ffs/ffs_vfsops.c @@ -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); }