diff options
| author | Peter Wemm <peter@FreeBSD.org> | 2002-09-07 22:31:44 +0000 |
|---|---|---|
| committer | Peter Wemm <peter@FreeBSD.org> | 2002-09-07 22:31:44 +0000 |
| commit | a9f9df5daf5321bc3a8d627dfdad70b10a30133d (patch) | |
| tree | fdac2ae32654c934bc9874ffbac7b508aeca7737 /sys/kern/imgact_aout.c | |
| parent | 1ed8cb4870638d11955c201f052967dc8f7ff8b7 (diff) | |
Tidy up some loose ends that bde pointed out. caddr_t bad, ok?
Move fill_kinfo_proc to before we copy the results instead of after
the copy and too late.
There is still more to do here.
Notes
Notes:
svn path=/head/; revision=103086
Diffstat (limited to 'sys/kern/imgact_aout.c')
| -rw-r--r-- | sys/kern/imgact_aout.c | 14 |
1 files changed, 7 insertions, 7 deletions
diff --git a/sys/kern/imgact_aout.c b/sys/kern/imgact_aout.c index e21358a91298..4ed589dd0ade 100644 --- a/sys/kern/imgact_aout.c +++ b/sys/kern/imgact_aout.c @@ -272,7 +272,7 @@ aout_coredump(td, vp, limit) struct proc *p = td->td_proc; register struct ucred *cred = td->td_ucred; register struct vmspace *vm = p->p_vmspace; - caddr_t tempuser; + char *tempuser; int error; if (ctob((uarea_pages + kstack_pages) @@ -282,15 +282,15 @@ aout_coredump(td, vp, limit) M_WAITOK | M_ZERO); if (tempuser == NULL) return (ENOMEM); + PROC_LOCK(p); + fill_kinfo_proc(p, &p->p_uarea->u_kproc); + PROC_UNLOCK(p); bcopy(p->p_uarea, tempuser, sizeof(struct user)); bcopy(td->td_frame, tempuser + ctob(uarea_pages) + - ((caddr_t) td->td_frame - (caddr_t) td->td_kstack), + ((caddr_t)td->td_frame - (caddr_t)td->td_kstack), sizeof(struct trapframe)); - PROC_LOCK(p); - fill_kinfo_proc(p, &p->p_uarea->u_kproc); - PROC_UNLOCK(p); - error = vn_rdwr(UIO_WRITE, vp, (caddr_t) tempuser, + error = vn_rdwr(UIO_WRITE, vp, (caddr_t)tempuser, ctob(uarea_pages + kstack_pages), (off_t)0, UIO_SYSSPACE, IO_UNIT, cred, NOCRED, (int *)NULL, td); @@ -302,7 +302,7 @@ aout_coredump(td, vp, limit) IO_UNIT | IO_DIRECT, cred, NOCRED, (int *) NULL, td); if (error == 0) error = vn_rdwr_inchunks(UIO_WRITE, vp, - (caddr_t) trunc_page(USRSTACK - ctob(vm->vm_ssize)), + (caddr_t)trunc_page(USRSTACK - ctob(vm->vm_ssize)), round_page(ctob(vm->vm_ssize)), (off_t)ctob(uarea_pages + kstack_pages) + ctob(vm->vm_dsize), UIO_USERSPACE, |