Skip site navigation (1)Skip section navigation (2)
Date:      Sat, 28 Dec 2019 16:40:44 +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: r356150 - head/sys/vm
Message-ID:  <201912281640.xBSGeieI075214@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: kib
Date: Sat Dec 28 16:40:44 2019
New Revision: 356150
URL: https://svnweb.freebsd.org/changeset/base/356150

Log:
  vm_object_shadow(): fix object reference leak.
  
  In r355270 by me, vm_object_shadow() was changed to handle the
  reference counting for the shared case, but the extra reference that
  was done in vmspace_fork() for the shared/need_copy case was not
  removed.
  
  Submitted by:	jeff

Modified:
  head/sys/vm/vm_map.c

Modified: head/sys/vm/vm_map.c
==============================================================================
--- head/sys/vm/vm_map.c	Sat Dec 28 15:30:50 2019	(r356149)
+++ head/sys/vm/vm_map.c	Sat Dec 28 16:40:44 2019	(r356150)
@@ -4127,8 +4127,6 @@ vmspace_fork(struct vmspace *vm1, vm_ooffset_t *fork_c
 				    true);
 				old_entry->eflags &= ~MAP_ENTRY_NEEDS_COPY;
 				old_entry->cred = NULL;
-				vm_object_reference(
-				    old_entry->object.vm_object);
 
 				/*
 				 * As in vm_map_merged_neighbor_dispose(),



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