carry on the copyargs() and exit1() changes from kern_exec.c

This commit is contained in:
christos 2001-07-29 21:28:20 +00:00
parent 781411d393
commit b474b9ac65
1 changed files with 7 additions and 4 deletions

View File

@ -1,4 +1,4 @@
/* $NetBSD: netbsd32_execve.c,v 1.4 2001/06/15 17:24:20 thorpej Exp $ */ /* $NetBSD: netbsd32_execve.c,v 1.5 2001/07/29 21:28:20 christos Exp $ */
/* /*
* Copyright (c) 1998, 2001 Matthew R. Green * Copyright (c) 1998, 2001 Matthew R. Green
@ -308,12 +308,16 @@ netbsd32_execve2(p, uap, retval)
stack = (char *) (vm->vm_minsaddr - len); stack = (char *) (vm->vm_minsaddr - len);
/* Now copy argc, args & environ to new stack */ /* Now copy argc, args & environ to new stack */
if (!(*pack.ep_es->es_copyargs)(&pack, &arginfo, stack, argp)) { error = (*pack.ep_es->es_copyargs)(&pack, &arginfo,
&stack, argp);
if (error) {
#ifdef DEBUG #ifdef DEBUG
printf("netbsd32_execve: copyargs failed\n"); printf("netbsd32_execve: copyargs failed\n");
#endif #endif
goto exec_abort; goto exec_abort;
} }
/* restore the stack back to its original point */
stack = (char *) (vm->vm_minsaddr - len);
/* fill process ps_strings info */ /* fill process ps_strings info */
p->p_psstr = (struct ps_strings *)(stack - sizeof(struct ps_strings)); p->p_psstr = (struct ps_strings *)(stack - sizeof(struct ps_strings));
@ -493,8 +497,7 @@ exec_abort:
vput(pack.ep_vp); vput(pack.ep_vp);
uvm_km_free_wakeup(exec_map, (vaddr_t) argp, NCARGS); uvm_km_free_wakeup(exec_map, (vaddr_t) argp, NCARGS);
free(pack.ep_hdr, M_EXEC); free(pack.ep_hdr, M_EXEC);
exit1(p, W_EXITCODE(0, SIGABRT)); exit1(p, W_EXITCODE(error, SIGABRT));
exit1(p, -1);
/* NOTREACHED */ /* NOTREACHED */
return 0; return 0;