Date: Thu, 21 Oct 2010 19:17:40 +0000 (UTC) From: John Baldwin <jhb@FreeBSD.org> To: cvs-src-old@freebsd.org Subject: cvs commit: src/sys/kern kern_exec.c kern_exit.c kern_fork.c kern_ktrace.c src/sys/sys ktrace.h Message-ID: <201010211918.o9LJIAOB014441@repoman.freebsd.org>
next in thread | raw e-mail | index | archive | help
jhb 2010-10-21 19:17:40 UTC
FreeBSD src repository
Modified files:
sys/kern kern_exec.c kern_exit.c kern_fork.c
kern_ktrace.c
sys/sys ktrace.h
Log:
SVN rev 214158 on 2010-10-21 19:17:40Z by jhb
- When disabling ktracing on a process, free any pending requests that
may be left. This fixes a memory leak that can occur when tracing is
disabled on a process via disabling tracing of a specific file (or if
an I/O error occurs with the tracefile) if the process's next system
call is exit(). The trace disabling code clears p_traceflag, so exit1()
doesn't do any KTRACE-related cleanup leading to the leak. I chose to
make the free'ing of pending records synchronous rather than patching
exit1().
- Move KTRACE-specific logic out of kern_(exec|exit|fork).c and into
kern_ktrace.c instead. Make ktrace_mtx private to kern_ktrace.c as a
result.
MFC after: 1 month
Revision Changes Path
1.359 +2 -10 src/sys/kern/kern_exec.c
1.337 +1 -31 src/sys/kern/kern_exit.c
1.314 +1 -15 src/sys/kern/kern_fork.c
1.136 +102 -24 src/sys/kern/kern_ktrace.c
1.39 +2 -2 src/sys/sys/ktrace.h
Want to link to this message? Use this URL: <https://mail-archive.FreeBSD.org/cgi/mid.cgi?201010211918.o9LJIAOB014441>