kvm.vger.kernel.org archive mirror
 help / color / mirror / Atom feed
From: Antoine Martin <antoine@nagafix.co.uk>
To: "kvm@vger.kernel.org" <kvm@vger.kernel.org>,
	Roland McGrath <roland@redhat.com>
Subject: kvm ptrace 32bit DoS bug - bisected
Date: Sat, 05 Sep 2009 20:41:26 +0700	[thread overview]
Message-ID: <4AA26A86.8060908@nagafix.co.uk> (raw)

-----BEGIN PGP SIGNED MESSAGE-----
Hash: SHA512

Hi,

I reported this bug a while ago but no-one picked up on it.
Just launch any UML 32-bit kernel on a 64-bit KVM guest:

test $ ./kernel32-2.6.16.62
Checking that ptrace can change system call numbers...OK
Checking syscall emulation patch for ptrace...OK
Trace/breakpoint trap
test@localhost ~ $ Kernel panic - not syncing: Attempted to kill init!
Kernel panic - not syncing: Attempted to kill init!


You can find some pre-built binaries here:
http://uml.devloop.org.uk/kernels.html

Since then, I've bisected it down to:
d4d67150165df8bf1cc05e532f6efca96f907cab is first bad commit
Author: Roland McGrath <roland@redhat.com>
Date:   Wed Jul 9 02:38:07 2008 -0700
Subject: x86 ptrace: unify syscall tracing

It looks exploitable at first sight (ptrace generally is), but this is
beyond me (I am not a kernel hacker)

QEMU without KVM is not affected.

I've added some printf in a test UML kernel to see more precisely where
it dies in arch/um/os-Linux/startup.c: in check_sysemu():
		non_fatal("Before singlestep\n");
                if (ptrace(PTRACE_SYSEMU_SINGLESTEP, pid, 0, 0) < 0)
                        goto fail;
                non_fatal("Before waitpid\n");
(also added a non_fatal() in fail)

It prints these two statements 30 times from the while(1) loop and stops on:
Before singlestep

Whatever the fix is, this should be queued for stable too.

Antoine
-----BEGIN PGP SIGNATURE-----
Version: GnuPG v1.4.9 (GNU/Linux)
Comment: Using GnuPG with Mozilla - http://enigmail.mozdev.org

iEYEAREKAAYFAkqiaoUACgkQGK2zHPGK1rt1cwCfWgGeuTrD+rpfa9SsUc7/h3eL
+DEAn1LgzrhOjbyEss2zRez+0dk0smZv
=MUXh
-----END PGP SIGNATURE-----

             reply	other threads:[~2009-09-05 13:41 UTC|newest]

Thread overview: 8+ messages / expand[flat|nested]  mbox.gz  Atom feed  top
2009-09-05 13:41 Antoine Martin [this message]
2009-09-05 20:43 ` kvm ptrace 32bit DoS bug - bisected Marcelo Tosatti
2009-09-06  7:37   ` Antoine Martin
2009-09-06  7:50     ` Antoine Martin
2009-09-08 16:33       ` Marcelo Tosatti
2009-09-08 16:53         ` Jan Kiszka
2009-10-17 13:24           ` Antoine Martin
2009-10-18  6:50             ` Avi Kivity

Reply instructions:

You may reply publicly to this message via plain-text email
using any one of the following methods:

* Save the following mbox file, import it into your mail client,
  and reply-to-all from there: mbox

  Avoid top-posting and favor interleaved quoting:
  https://en.wikipedia.org/wiki/Posting_style#Interleaved_style

* Reply using the --to, --cc, and --in-reply-to
  switches of git-send-email(1):

  git send-email \
    --in-reply-to=4AA26A86.8060908@nagafix.co.uk \
    --to=antoine@nagafix.co.uk \
    --cc=kvm@vger.kernel.org \
    --cc=roland@redhat.com \
    /path/to/YOUR_REPLY

  https://kernel.org/pub/software/scm/git/docs/git-send-email.html

* If your mail client supports setting the In-Reply-To header
  via mailto: links, try the mailto: link
Be sure your reply has a Subject: header at the top and a blank line before the message body.
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for NNTP newsgroup(s).