Do not leak oldvmspace if image activation failed

[kib]

Summary

and current address space is already destroyed, so kern_execve() terminates the process.

While there, clean up some internals of post_execve() inlined in init_main.

[markj]

sys/kern/kern_exec.c

981 ↗

(On Diff #77376)

Why is it necessary to do this here? There is one caller where failure == true and in that case the following exit1() call handles it, if I understand correctly.

[kib]

sys/kern/kern_exec.c

981 ↗

(On Diff #77376)

I suspect that exit1() really does not handle it. exit1() does thread_single(SINGLE_EXIT) which silently overrides previous single-threading mode, because p_singlethread == curthread.

I can move drop failure case from exec_cleanup(), leaving only the handling of TDP_EXECVMSPC there, if you prefer it this way.

[kib]

sys/kern/kern_exec.c

981 ↗

(On Diff #77376)

Hmm, it is always compatible. I will update the patch in half a hour.

[kib]

Previous single-threading mode is always SINGLE_BOUNDARY, if any. Drop code in exec_cleanup() that duplicates exit1().