linux-kernel - Re: [PATCH v3 17/21] tools/objtool: Convert to insn

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives

Hash Suite: Windows password security audit tool. GUI, reports in PDF.

[<prev] [next>] [<thread-prev] [thread-next>] [day] [month] [year] [list]

Date:   Thu, 4 Mar 2021 19:06:52 +0100
From:   Borislav Petkov <bp@...en8.de>
To:     Peter Zijlstra <peterz@...radead.org>
Cc:     X86 ML <x86@...nel.org>, Masami Hiramatsu <mhiramat@...nel.org>,
        LKML <linux-kernel@...r.kernel.org>
Subject: Re: [PATCH v3 17/21] tools/objtool: Convert to insn_decode()

On Thu, Mar 04, 2021 at 06:49:08PM +0100, Peter Zijlstra wrote:
> This is going to have trivial rejects/fuzz against tip/objtool/core.

I was just wondering whether to you show you how I resolved :)

diff --cc tools/objtool/arch/x86/decode.c
index 431bafb881d4,8380d0b1d933..22a53ee322ea
--- a/tools/objtool/arch/x86/decode.c
+++ b/tools/objtool/arch/x86/decode.c
@@@ -104,14 -90,12 +104,14 @@@ int arch_decode_instruction(const struc
  			    struct list_head *ops_list)
  {
  	struct insn insn;
- 	int x86_64;
 -	int x86_64, sign, ret;
 -	unsigned char op1, op2, rex = 0, rex_b = 0, rex_r = 0, rex_w = 0,
 -		      rex_x = 0, modrm = 0, modrm_mod = 0, modrm_rm = 0,
 -		      modrm_reg = 0, sib = 0;
++	int x86_64, ret;
 +	unsigned char op1, op2,
 +		      rex = 0, rex_b = 0, rex_r = 0, rex_w = 0, rex_x = 0,
 +		      modrm = 0, modrm_mod = 0, modrm_rm = 0, modrm_reg = 0,
 +		      sib = 0, /* sib_scale = 0, */ sib_index = 0, sib_base = 0;
  	struct stack_op *op = NULL;
  	struct symbol *sym;
 +	u64 imm;
  
  	x86_64 = is_x86_64(elf);
  	if (x86_64 == -1)

-- 
Regards/Gruss,
    Boris.

https://people.kernel.org/tglx/notes-about-netiquette