Commit f0aeb8bf authored Apr 18, 2011 by Jon Medhurst Committed by Nicolas Pitre Apr 28, 2011

ARM: kprobes: Reject probing of SETEND instructions



The emulation of SETEND was broken as it changed the endianess for
the running kprobes handling code. Rather than adding a new simulation
routine to fix this we'll just reject probing of SETEND as these should
be very rare in the kernel.

Note, the function emulate_none is now unused but it is left in the
source code as future patches will use it.

Signed-off-by: Jon Medhurst <tixy@yxit.co.uk>
Signed-off-by: Nicolas Pitre <nicolas.pitre@linaro.org>

parent ac211c69

arch/arm/kernel/kprobes-decode.c

+0 −5

Original line number	Diff line number	Diff line
		@@ -956,11 +956,6 @@ space_1111(kprobe_opcode_t insn, struct arch_specific_insn *asi)
		}

		/* SETEND : 1111 0001 0000 0001 xxxx xxxx 0000 xxxx */
		if ((insn & 0xffff00f0) == 0xf1010000) {
		asi->insn[0] = insn;
		asi->insn_handler = emulate_none;
		return INSN_GOOD;
		}

		/* Coprocessor instructions... */
		/* MCRR2 : 1111 1100 0100 xxxx xxxx xxxx xxxx xxxx : (Rd != Rn) */