[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [Qemu-devel] [Bug 1577841] [NEW] target-mips/helper.c:542: bad sizeo
From: |
Aurelien Jarno |
Subject: |
Re: [Qemu-devel] [Bug 1577841] [NEW] target-mips/helper.c:542: bad sizeof ? |
Date: |
Tue, 3 May 2016 23:11:31 +0200 |
User-agent: |
Mutt/1.5.24 (2015-08-30) |
On 2016-05-03 20:11, Stefan Weil wrote:
> Am 03.05.2016 um 17:58 schrieb dcb:
> > Public bug reported:
> >
> > Recent versions of gcc say this:
> >
> > qemu/target-mips/helper.c:542:9: warning: ‘memset’ used with length
> > equal to number of elements without multiplication by element size
> > [-Wmemset-elt-size]
> >
> > Source code is
> >
> > memset(env->CP0_WatchLo, 0, sizeof(*env->CP0_WatchLo));
> >
> > Maybe better code
> >
> > memset(env->CP0_WatchLo, 0, 8 * sizeof(target_ulong));
> >
> > ** Affects: qemu
> > Importance: Undecided
> > Status: New
>
> This might be an error. I think it should be
>
> memset(env->CP0_WatchLo, 0, sizeof(env->CP0_WatchLo));
>
I confirm this is the correct version of the code. Someone is sending a
patch, i should I do that?
Aurelien
--
Aurelien Jarno GPG: 4096R/1DDD8C9B
address@hidden http://www.aurel32.net