Skip site navigation (1)Skip section navigation (2)
Date:      Tue, 24 Jan 2017 16:47:07 +0000 (UTC)
From:      Bryan Drewery <bdrewery@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-10@freebsd.org
Subject:   svn commit: r312700 - stable/10/libexec/rtld-elf
Message-ID:  <201701241647.v0OGl7pr009835@repo.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: bdrewery
Date: Tue Jan 24 16:47:06 2017
New Revision: 312700
URL: https://svnweb.freebsd.org/changeset/base/312700

Log:
  MFC r310025:
  
    Take write lock for rtld_bind before modifying obj_list in dl_iterate_phdr().

Modified:
  stable/10/libexec/rtld-elf/rtld.c
Directory Properties:
  stable/10/   (props changed)

Modified: stable/10/libexec/rtld-elf/rtld.c
==============================================================================
--- stable/10/libexec/rtld-elf/rtld.c	Tue Jan 24 16:13:59 2017	(r312699)
+++ stable/10/libexec/rtld-elf/rtld.c	Tue Jan 24 16:47:06 2017	(r312700)
@@ -3470,7 +3470,7 @@ dl_iterate_phdr(__dl_iterate_hdr_callbac
 	error = 0;
 
 	wlock_acquire(rtld_phdr_lock, &phdr_lockstate);
-	rlock_acquire(rtld_bind_lock, &bind_lockstate);
+	wlock_acquire(rtld_bind_lock, &bind_lockstate);
 	for (obj = globallist_curr(TAILQ_FIRST(&obj_list)); obj != NULL;) {
 		TAILQ_INSERT_AFTER(&obj_list, obj, &marker, next);
 		rtld_fill_dl_phdr_info(obj, &phdr_info);
@@ -3478,7 +3478,7 @@ dl_iterate_phdr(__dl_iterate_hdr_callbac
 
 		error = callback(&phdr_info, sizeof phdr_info, param);
 
-		rlock_acquire(rtld_bind_lock, &bind_lockstate);
+		wlock_acquire(rtld_bind_lock, &bind_lockstate);
 		obj = globallist_next(&marker);
 		TAILQ_REMOVE(&obj_list, &marker, next);
 		if (error != 0) {



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