Commit84da009

mhiramat

authored and

suryasaimadhu

committed

x86/sev-es: Use new for_each_insn_prefix() macro to loop over prefixes bytes

Since insn.prefixes.nbytes can be bigger than the size ofinsn.prefixes.bytes[] when a prefix is repeated, the propercheck must be: insn.prefixes.bytes[i] != 0 and i < 4instead of using insn.prefixes.nbytes. Use the newfor_each_insn_prefix() macro which does it correctly.Debugged by Kees Cook <keescook@chromium.org>. [ bp: Massage commit message. ]Fixes:25189d0 ("x86/sev-es: Add support for handling IOIO exceptions")Reported-by: syzbot+9b64b619f10f19d19a7c@syzkaller.appspotmail.comSigned-off-by: Masami Hiramatsu <mhiramat@kernel.org>Signed-off-by: Borislav Petkov <bp@suse.de>Link:https://lkml.kernel.org/r/160697106089.3146288.2052422845039649176.stgit@devnote2

1 parent12cb908 commit84da009Copy full SHA for 84da009

File tree

1 file changed

-3

lines changed

arch/x86/boot/compressed
- sev-es.c

1 file changed

-3

lines changed

`‎arch/x86/boot/compressed/sev-es.c‎`

Lines changed: 2 additions & 3 deletions

Original file line number	Diff line number	Diff line change
`@@ -32,13 +32,12 @@ struct ghcb *boot_ghcb;`
`32`	`32`	`*/`
`33`	`33`	`staticboolinsn_has_rep_prefix(structinsn*insn)`
`34`	`34`	`{`
	`35`	`+insn_byte_tp;`
`35`	`36`	`inti;`
`36`	`37`
`37`	`38`	`insn_get_prefixes(insn);`
`38`	`39`
`39`		`-for (i=0;i<insn->prefixes.nbytes;i++) {`
`40`		`-insn_byte_tp=insn->prefixes.bytes[i];`
`41`		`-`
	`40`	`+for_each_insn_prefix(insn,i,p) {`
`42`	`41`	`if (p==0xf2\|\|p==0xf3)`
`43`	`42`	`return true;`
`44`	`43`	`}`

0 commit comments

Comments

(0)

Movatterモバイル変換

Navigation Menu

Search code, repositories, users, issues, pull requests...

Provide feedback

Saved searches

Use saved searches to filter your results more quickly

Uh oh!

Commit84da009

File tree

1 file changed

1 file changed

`‎arch/x86/boot/compressed/sev-es.c‎`

0 commit comments