ptrace reads/writes memory using uvm_io, which generates an temporary

alias mapping using uvm_map_extract.  With xonly now operational, this
alias mapping is created with minprot, which for text will be xonly, and
the kernel cannot read it (unless the architecture has implied read for
exec from kernel pov).
Pass UVM_IO_FIXPROT to create the alias with maxprot instead.
ok kettenis

diff --git a/sys/kern/sys_process.c b/sys/kern/sys_process.c
index 5d0df0c..1fdfc65 100644 (file)
--- a/sys/kern/sys_process.c
+++ b/sys/kern/sys_process.c
@@ -1,4 +1,4 @@
-/*     $OpenBSD: sys_process.c,v 1.92 2023/01/02 23:09:48 guenther Exp $       */
+/*     $OpenBSD: sys_process.c,v 1.93 2023/01/24 00:12:03 deraadt Exp $        */
 /*     $NetBSD: sys_process.c,v 1.55 1996/05/15 06:17:47 tls Exp $     */
 
 /*-
@@ -864,8 +864,7 @@ process_domem(struct proc *curp, struct process *tr, struct uio *uio, int req)
 
        uvmspace_addref(vm);
 
-       error = uvm_io(&vm->vm_map, uio,
-           (uio->uio_rw == UIO_WRITE) ? UVM_IO_FIXPROT : 0);
+       error = uvm_io(&vm->vm_map, uio, UVM_IO_FIXPROT);
 
        uvmspace_free(vm);