Skip site navigation (1)Skip section navigation (2)
Date:      Mon, 14 Dec 2009 12:19:21 +0000 (UTC)
From:      Robert Watson <rwatson@FreeBSD.org>
To:        src-committers@freebsd.org, svn-src-all@freebsd.org, svn-src-stable@freebsd.org, svn-src-stable-8@freebsd.org
Subject:   svn commit: r200509 - stable/8/libexec/rtld-elf
Message-ID:  <200912141219.nBECJLfJ026164@svn.freebsd.org>

next in thread | raw e-mail | index | archive | help
Author: rwatson
Date: Mon Dec 14 12:19:21 2009
New Revision: 200509
URL: http://svn.freebsd.org/changeset/base/200509

Log:
  Merge r197808 from head to stable/8:
  
    In rtld's map_object(), use pread(..., 0) rather than read() to read the
    ELF header from the front of the file.  As all other I/O on the binary
    is done using mmap(), this avoids the need for seek privileges on the
    file descriptor during run-time linking.
  
    Sponsored by: Google

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

Modified: stable/8/libexec/rtld-elf/map_object.c
==============================================================================
--- stable/8/libexec/rtld-elf/map_object.c	Mon Dec 14 11:53:02 2009	(r200508)
+++ stable/8/libexec/rtld-elf/map_object.c	Mon Dec 14 12:19:21 2009	(r200509)
@@ -273,7 +273,7 @@ get_elf_header (int fd, const char *path
     } u;
     ssize_t nbytes;
 
-    if ((nbytes = read(fd, u.buf, PAGE_SIZE)) == -1) {
+    if ((nbytes = pread(fd, u.buf, PAGE_SIZE, 0)) == -1) {
 	_rtld_error("%s: read error: %s", path, strerror(errno));
 	return NULL;
     }



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