[Date Prev][Date Next][Thread Prev][Thread Next][Date Index][Thread Index]
Re: [PATCH 3/4] pc-bios/s390-ccw: Silence GCC 11 stringop-overflow warni
From: |
Philippe Mathieu-Daudé |
Subject: |
Re: [PATCH 3/4] pc-bios/s390-ccw: Silence GCC 11 stringop-overflow warning |
Date: |
Mon, 3 May 2021 11:50:39 +0200 |
User-agent: |
Mozilla/5.0 (X11; Linux x86_64; rv:78.0) Gecko/20100101 Thunderbird/78.8.1 |
On 5/3/21 11:31 AM, Thomas Huth wrote:
> On 03/05/2021 11.30, Philippe Mathieu-Daudé wrote:
>> On 5/3/21 11:00 AM, Cornelia Huck wrote:
>>> On Sun, 2 May 2021 19:48:35 +0200
>>> Thomas Huth <thuth@redhat.com> wrote:
>>>
>>>> From: Philippe Mathieu-Daudé <philmd@redhat.com>
>>>>
>>>> When building on Fedora 34 (gcc version 11.0.0 20210210) we get:
>>>>
>>>> In file included from pc-bios/s390-ccw/main.c:11:
>>>> In function ‘memset’,
>>>> inlined from ‘boot_setup’ at pc-bios/s390-ccw/main.c:185:5,
>>>> inlined from ‘main’ at pc-bios/s390-ccw/main.c:288:5:
>>>> pc-bios/s390-ccw/libc.h:28:14: warning: writing 1 byte into a
>>>> region of size 0 [-Wstringop-overflow=]
>>>> 28 | p[i] = c;
>>>> | ~~~~~^~~
>>>>
>>>> The offending code is:
>>>>
>>>> memset((char *)S390EP, 0, 6);
>>>>
>>>> where S390EP is a const address:
>>>>
>>>> #define S390EP 0x10008
>>>>
>>>> The compiler doesn't now how big that pointed area is, so assume its
>>>
>>> s/now/know/
>>> s/assume/it assumes that/
>>
>> Oops, thanks. Thomas, do you want me to repost this patch fixed?
>
> I can fix it up in my tree here, no need to resend.
Great, thank you!
Phil.
- Re: [PATCH 4/4] pc-bios/s390-ccw: Allow building with Clang, too, (continued)
- Re: [PATCH 4/4] pc-bios/s390-ccw: Allow building with Clang, too, Thomas Huth, 2021/05/03
- Re: [PATCH 4/4] pc-bios/s390-ccw: Allow building with Clang, too, Cornelia Huck, 2021/05/03
- Re: [PATCH 4/4] pc-bios/s390-ccw: Allow building with Clang, too, Christian Borntraeger, 2021/05/03
Re: [PATCH 4/4] pc-bios/s390-ccw: Allow building with Clang, too, Cornelia Huck, 2021/05/03
[PATCH 3/4] pc-bios/s390-ccw: Silence GCC 11 stringop-overflow warning, Thomas Huth, 2021/05/02