x86: properly initialize temp insn buffer for paravirt patching
authorChris Wright <chrisw@sous-sol.org>
Sat, 18 Aug 2007 21:31:41 +0000 (14:31 -0700)
committerLinus Torvalds <torvalds@woody.linux-foundation.org>
Sat, 18 Aug 2007 22:15:54 +0000 (15:15 -0700)
commitd34fda4a84c18402640a1a2342d6e6d9829e6db7
treec6da4cdd6c63be258a32635e87b239c570d5f664
parent18115f45374d19ada218fc013aa5308baf5d283e
x86: properly initialize temp insn buffer for paravirt patching

With commit ab144f5ec64c42218a555ec1dbde6b60cf2982d6 the patching code
now collects the complete new instruction stream into a temp buffer
before finally patching in the new insns.  In some cases the paravirt
patchers will choose to leave the patch site unpatched (length mismatch,
clobbers mismatch, etc).

This causes the new patching code to copy an uninitialized temp buffer,
i.e.  garbage, to the callsite.  Simply make sure to always initialize
the buffer with the original instruction stream.  A better fix is to
audit all the patchers and return proper length so that apply_paravirt()
can skip copies when we leave the patch site untouched.

Signed-off-by: Chris Wright <chrisw@sous-sol.org>
Signed-off-by: Linus Torvalds <torvalds@linux-foundation.org>
arch/i386/kernel/alternative.c