diff options
| author |   millert <millert@openbsd.org> | 2004-02-05 22:56:10 +0000 |
|---|---|---|
| committer | millert <millert@openbsd.org> | 2004-02-05 22:56:10 +0000 |
| commit | e481f8e1b51bd77f2720bb49e35575a5ea1e0c8a (patch) | |
| tree | 57443b35396b2e8aa1c90e23e3de56712ad58f54 /sys/kern/kern_exec.c | |
| parent | sync (diff) | |
| download | wireguard-openbsd-e481f8e1b51bd77f2720bb49e35575a5ea1e0c8a.tar.xz wireguard-openbsd-e481f8e1b51bd77f2720bb49e35575a5ea1e0c8a.zip | |
Decrement reference count in error case if uvm_map(9) fails and we
had previously increased the reference count. Should not happen, but...
From dhartmei@; ok deraadt@
Diffstat (limited to 'sys/kern/kern_exec.c')
| -rw-r--r-- | sys/kern/kern_exec.c | 4 |
1 files changed, 3 insertions, 1 deletions
diff --git a/sys/kern/kern_exec.c b/sys/kern/kern_exec.c index c8c4b2e14bb..68f856db795 100644 --- a/sys/kern/kern_exec.c +++ b/sys/kern/kern_exec.c @@ -1,4 +1,4 @@ -/* $OpenBSD: kern_exec.c,v 1.82 2003/09/01 18:06:03 henning Exp $ */ +/* $OpenBSD: kern_exec.c,v 1.83 2004/02/05 22:56:10 millert Exp $ */ /* $NetBSD: kern_exec.c,v 1.75 1996/02/09 18:59:28 christos Exp $ */ /*- @@ -738,6 +738,7 @@ exec_sigcode_map(struct proc *p, struct emul *e) 0, 0, UVM_MAPFLAG(UVM_PROT_RW, UVM_PROT_RW, UVM_INH_SHARE, UVM_ADV_RANDOM, 0)))) { printf("kernel mapping failed %d\n", r); + uao_detach(e->e_sigobject); return (ENOMEM); } memcpy((void *)va, e->e_sigcode, sz); @@ -751,6 +752,7 @@ exec_sigcode_map(struct proc *p, struct emul *e) e->e_sigobject, 0, 0, UVM_MAPFLAG(UVM_PROT_RX, UVM_PROT_RX, UVM_INH_SHARE, UVM_ADV_RANDOM, 0))) { printf("user mapping failed\n"); + uao_detach(e->e_sigobject); return (ENOMEM); } |