Re: [Qemu-devel] [PATCH] target-i386: cpu: fix potential buffer overrun in get_register_name_32()

2013-06-04 Thread Andreas Färber
Am 03.06.2013 18:23, schrieb Igor Mammedov: > spotted by Coverity, > x86_reg_info_32[] is CPU_NB_REGS32 elements long, so accessing > x86_reg_info_32[CPU_NB_REGS32] will be one element off array. > > Signed-off-by: Igor Mammedov > --- > target-i386/cpu.c |2 +- > 1 files changed, 1 insertion

Re: [Qemu-devel] [PATCH] target-i386: cpu: fix potential buffer overrun in get_register_name_32()

2013-06-03 Thread li guang
Reviewed-by: liguang 在 2013-06-03一的 18:23 +0200,Igor Mammedov写道: > spotted by Coverity, > x86_reg_info_32[] is CPU_NB_REGS32 elements long, so accessing > x86_reg_info_32[CPU_NB_REGS32] will be one element off array. Yes, it will. > > Signed-off-by: Igor Mammedov > --- > target-i386/cpu.c |

Re: [Qemu-devel] [PATCH] target-i386: cpu: fix potential buffer overrun in get_register_name_32()

2013-06-03 Thread Jesse Larrew
On 06/03/2013 11:23 AM, Igor Mammedov wrote: > spotted by Coverity, > x86_reg_info_32[] is CPU_NB_REGS32 elements long, so accessing > x86_reg_info_32[CPU_NB_REGS32] will be one element off array. > > Signed-off-by: Igor Mammedov > --- > target-i386/cpu.c |2 +- > 1 files changed, 1 insertio

[Qemu-devel] [PATCH] target-i386: cpu: fix potential buffer overrun in get_register_name_32()

2013-06-03 Thread Igor Mammedov
spotted by Coverity, x86_reg_info_32[] is CPU_NB_REGS32 elements long, so accessing x86_reg_info_32[CPU_NB_REGS32] will be one element off array. Signed-off-by: Igor Mammedov --- target-i386/cpu.c |2 +- 1 files changed, 1 insertions(+), 1 deletions(-) diff --git a/target-i386/cpu.c b/targe