ld.so should stop calling msyscall(2), since it is fully neutered and

about to be removed.  Please be careful building through this, you need
a kernel at least March 29th or so to build through it, otherwise use
snapshots to cross over.
ok various people

diff --git a/libexec/ld.so/Makefile b/libexec/ld.so/Makefile
index 44ab6b4..7f8f6ef 100644 (file)
--- a/libexec/ld.so/Makefile
+++ b/libexec/ld.so/Makefile
@@ -1,4 +1,4 @@
-#      $OpenBSD: Makefile,v 1.87 2024/01/19 14:16:41 deraadt Exp $
+#      $OpenBSD: Makefile,v 1.88 2024/04/05 13:51:47 deraadt Exp $
 
 SUBDIR=ldconfig ldd
 MAN=   ld.so.1
@@ -28,7 +28,7 @@ SRCS+=        dl_uname.c dl_dirname.c strlcat.c strlen.c trace.c
 SRCS+= malloc.c reallocarray.c tib.c ffs.c
 
 syscall=close exit fstat getdents getentropy getthrid issetugid kbind \
-       mimmutable mmap mprotect munmap msyscall open pledge \
+       mimmutable mmap mprotect munmap open pledge \
        pinsyscalls read __realpath sendsyslog __set_tcb sysctl thrkill \
        utrace write
 

diff --git a/libexec/ld.so/library.c b/libexec/ld.so/library.c
index 8cc305c..405fc97 100644 (file)
--- a/libexec/ld.so/library.c
+++ b/libexec/ld.so/library.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: library.c,v 1.95 2024/01/17 13:00:05 deraadt Exp $ */
+/*     $OpenBSD: library.c,v 1.96 2024/04/05 13:51:47 deraadt Exp $ */
 
 /*
  * Copyright (c) 2002 Dale Rahn
@@ -324,21 +324,10 @@ _dl_tryload_shlib(const char *libname, int type, int flags, int nodelete)
        }
 
        libc = _dl_islibc(dynp, loff);
-       if (libc) {
-               if (syscall_phdp)
-                       _dl_pin(libfile, syscall_phdp, (void *)libaddr,
-                           (size_t)((exec_start + exec_size) - libaddr),
-                           exec_start, exec_size);
-
-               /*
-                * XXX msyscall() can be removed once pinsyscalls()
-                * is fully operational
-                */
-               /* Request permission for system calls in libc.so's text segment */
-               if (_dl_msyscall(exec_start, exec_size) == -1)
-                       _dl_printf("msyscall %lx %lx error\n",
-                           exec_start, exec_size);
-       }
+       if (libc && syscall_phdp)
+               _dl_pin(libfile, syscall_phdp, (void *)libaddr,
+                   (size_t)((exec_start + exec_size) - libaddr),
+                   exec_start, exec_size);
        _dl_close(libfile);
 
        dynp = (Elf_Dyn *)((unsigned long)dynp + loff);

diff --git a/libexec/ld.so/library_mquery.c b/libexec/ld.so/library_mquery.c
index 1129ecd..971d975 100644 (file)
--- a/libexec/ld.so/library_mquery.c
+++ b/libexec/ld.so/library_mquery.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: library_mquery.c,v 1.75 2024/01/17 13:00:05 deraadt Exp $ */
+/*     $OpenBSD: library_mquery.c,v 1.76 2024/04/05 13:51:47 deraadt Exp $ */
 
 /*
  * Copyright (c) 2002 Dale Rahn
@@ -332,21 +332,10 @@ retry:
        }
 
        libc = _dl_islibc(dynp, LOFF);
-       if (libc) {
-               if (syscall_phdp)
-                       _dl_pin(libfile, syscall_phdp, lowld->start,
-                           (size_t)((exec_start + exec_size) - LOFF),
-                           exec_start, exec_size);
-
-               /*
-                * XXX msyscall() can be removed once pinsyscalls()
-                * is fully operational
-                */
-               /* Request permission for system calls in libc.so's text segment */
-               if (_dl_msyscall(exec_start, exec_size) == -1)
-                       _dl_printf("msyscall %lx %lx error\n",
-                           exec_start, exec_size);
-       }
+       if (libc && syscall_phdp)
+               _dl_pin(libfile, syscall_phdp, lowld->start,
+                   (size_t)((exec_start + exec_size) - LOFF),
+                   exec_start, exec_size);
        _dl_close(libfile);
 
        dynp = (Elf_Dyn *)((unsigned long)dynp + LOFF);

diff --git a/libexec/ld.so/syscall.h b/libexec/ld.so/syscall.h
index 59a8789..de08b23 100644 (file)
--- a/libexec/ld.so/syscall.h
+++ b/libexec/ld.so/syscall.h
@@ -1,4 +1,4 @@
-/*     $OpenBSD: syscall.h,v 1.7 2024/01/19 14:16:42 deraadt Exp $ */
+/*     $OpenBSD: syscall.h,v 1.8 2024/04/05 13:51:47 deraadt Exp $ */
 
 /*
  * Copyright (c) 1998 Per Fogelstrom, Opsycon AB
@@ -50,7 +50,6 @@ int   _dl_kbind(const struct __kbind *, size_t, int64_t);
 void   *_dl_mmap(void *, size_t, int, int, int, off_t);
 int    _dl_mprotect(const void *, size_t, int);
 void   *_dl_mquery(void *, size_t, int, int, int, off_t);
-int    _dl_msyscall(void *addr, size_t len);
 int    _dl_pinsyscalls(void *base, size_t len, u_int *pin, int pinlen);
 int    _dl_munmap(const void *, size_t);
 int    _dl_mimmutable(const void *, size_t);