Merge "Change requested in launchpad bug #1288352" into 5.0
[packages/centos6/qemu.git] / 0002-target-cris-Fix-buffer-overflow.patch
1 From 8057ac10e8cba3acb89c11c94f04967306e55a9f Mon Sep 17 00:00:00 2001
2 From: Stefan Weil <sw@weilnetz.de>
3 Date: Fri, 7 Sep 2012 22:36:08 +0200
4 Subject: [PATCH] target-cris: Fix buffer overflow
5
6 Report from smatch:
7
8 target-cris/translate.c:3464 cpu_dump_state(32) error:
9  buffer overflow 'env->sregs' 4 <= 255
10
11 sregs is declared 'uint32_t sregs[4][16]', so the first index must be
12 less than 4 or ARRAY_SIZE(env->sregs).
13
14 Signed-off-by: Stefan Weil <sw@weilnetz.de>
15 Signed-off-by: Michael Roth <mdroth@linux.vnet.ibm.com>
16 ---
17  target-cris/translate.c | 2 +-
18  1 file changed, 1 insertion(+), 1 deletion(-)
19
20 diff --git a/target-cris/translate.c b/target-cris/translate.c
21 index 1ad9ec7..ad31877 100644
22 --- a/target-cris/translate.c
23 +++ b/target-cris/translate.c
24 @@ -3458,7 +3458,7 @@ void cpu_dump_state (CPUCRISState *env, FILE *f, fprintf_function cpu_fprintf,
25         }
26         srs = env->pregs[PR_SRS];
27         cpu_fprintf(f, "\nsupport function regs bank %x:\n", srs);
28 -       if (srs < 256) {
29 +       if (srs < ARRAY_SIZE(env->sregs)) {
30                 for (i = 0; i < 16; i++) {
31                         cpu_fprintf(f, "s%2.2d=%8.8x ",
32                                     i, env->sregs[srs][i]);
33 -- 
34 1.7.12.1
35