diff options
author | Haavard Skinnemoen <hskinnemoen@atmel.com> | 2008-02-27 14:04:29 +0100 |
---|---|---|
committer | Haavard Skinnemoen <hskinnemoen@atmel.com> | 2008-02-27 14:23:53 +0100 |
commit | 325d6f5593b40b5a48cf4ade74c01681f2ff6044 (patch) | |
tree | facb22a7deb7b6c9ce2bdaf07bcc4ac3bd8d21b5 /init/version.c | |
parent | d45ad06273f797f6239b97fd9962ecd81eec847f (diff) | |
download | op-kernel-dev-325d6f5593b40b5a48cf4ade74c01681f2ff6044.zip op-kernel-dev-325d6f5593b40b5a48cf4ade74c01681f2ff6044.tar.gz |
avr32: Fix OCD refcounting bug
Iff the parent has TIF_DEBUG set, _and_ clone_flags includes
CLONE_PTRACE we should set the TIF_DEBUG flag for the child and
increment the ocd refcount. Otherwise, the TIF_DEBUG flag must be
unset.
Currently, the child inherits TIF_DEBUG from the parent before
copy_thread is called, so TIF_DEBUG may be already be set before we
determine whether the child is supposed to inherit debugging
capabilities from the parent or not. This means that ocd_enable()
won't increment the refcount, because TIF_DEBUG is already set, and
that TIF_DEBUG will be set for processes that aren't being debugged.
This leads to a refcounting asymmetry, which may show up as
------------[ cut here ]------------
Badness at arch/avr32/kernel/ocd.c:73
PC is at ocd_disable+0x34/0x60
LR is at put_lock_stats+0xa/0x20
as reported by David Brownell. Happens when strace'ing a process that
forks a new child process, e.g. "strace mount -tjffs2 mtd1 /mnt", and
subsequently killing the child process (e.g. "umount /mnt".)
Signed-off-by: Haavard Skinnemoen <hskinnemoen@atmel.com>
Diffstat (limited to 'init/version.c')
0 files changed, 0 insertions, 0 deletions