diff options
Diffstat (limited to 'results/classifier/105/device/1660035')
| -rw-r--r-- | results/classifier/105/device/1660035 | 51 |
1 files changed, 51 insertions, 0 deletions
diff --git a/results/classifier/105/device/1660035 b/results/classifier/105/device/1660035 new file mode 100644 index 00000000..1d2d39f1 --- /dev/null +++ b/results/classifier/105/device/1660035 @@ -0,0 +1,51 @@ +device: 0.950 +instruction: 0.878 +socket: 0.748 +network: 0.742 +assembly: 0.702 +semantic: 0.691 +vnc: 0.651 +other: 0.593 +mistranslation: 0.577 +graphic: 0.563 +boot: 0.486 +KVM: 0.350 + +hw/timer/altera_timer.c:207: bad size in memset ? + +hw/timer/altera_timer.c:207:5: warning: ‘memset’ used with length equal to number of elements without multiplication by element size [-Wmemset-elt-size] + +Source code is + + memset(t->regs, 0, ARRAY_SIZE(t->regs)); + +Maybe better code + + memset(t->regs, 0, R_MAX * sizeof( uint32_t)); + +On 28 January 2017 at 13:16, dcb <email address hidden> wrote: +> Public bug reported: +> +> hw/timer/altera_timer.c:207:5: warning: ‘memset’ used with length equal +> to number of elements without multiplication by element size [-Wmemset- +> elt-size] +> +> Source code is +> +> memset(t->regs, 0, ARRAY_SIZE(t->regs)); +> +> Maybe better code +> +> memset(t->regs, 0, R_MAX * sizeof( uint32_t)); + +Better still -- just memset(t->regs, 0, sizeof(t->regs)); + +Chris, could you send a patch to fix this, please? + +thanks +-- PMM + + +This problem should have been fixed with QEMU v2.10.0: +https://git.qemu.org/?p=qemu.git;a=commitdiff;h=cc16ee9d4ecec35ea04e + |