From: Borislav Petkov <b...@suse.de>

Simplify code, no functional changes.

Signed-off-by: Borislav Petkov <b...@suse.de>
---
 arch/x86/kernel/uprobes.c | 8 ++++----
 1 file changed, 4 insertions(+), 4 deletions(-)

diff --git a/arch/x86/kernel/uprobes.c b/arch/x86/kernel/uprobes.c
index a2b413394917..b63cf8f7745e 100644
--- a/arch/x86/kernel/uprobes.c
+++ b/arch/x86/kernel/uprobes.c
@@ -276,12 +276,12 @@ static bool is_prefix_bad(struct insn *insn)
 
 static int uprobe_init_insn(struct arch_uprobe *auprobe, struct insn *insn, 
bool x86_64)
 {
+       enum insn_mode m = x86_64 ? INSN_MODE_64 : INSN_MODE_32;
        u32 volatile *good_insns;
+       int ret;
 
-       insn_init(insn, auprobe->insn, sizeof(auprobe->insn), x86_64);
-       /* has the side-effect of processing the entire instruction */
-       insn_get_length(insn);
-       if (!insn_complete(insn))
+       ret = insn_decode(insn, auprobe->insn, sizeof(auprobe->insn), m);
+       if (ret < 0)
                return -ENOEXEC;
 
        if (is_prefix_bad(insn))
-- 
2.29.2

Reply via email to