Avoid some GCC 10 warnings

[PATCH 0/3] Avoid some GCC 10 warnings

Boris Fiuczynski posted 3 patches 3 years, 8 months ago

Download series mbox

Test syntax-check failed

Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/libvirt tags/patchew/20200813140346.4284-1-fiuczy@linux.ibm.com

src/qemu/qemu_migration.c                  | 2 +-
src/storage/storage_backend_iscsi_direct.c | 8 ++++----
tools/vsh.c                                | 2 +-
3 files changed, 6 insertions(+), 6 deletions(-)

Expand all Fold all

[PATCH 0/3] Avoid some GCC 10 warnings

Posted by Boris Fiuczynski 3 years, 8 months ago

Caught these when switching to F32 using GCC v10.2.1 on s390x.

Boris Fiuczynski (3):
  qemu: avoid maybe-uninitialized warning by GCC 10
  tools: avoid potential null pointer dereference by GCC 10
  storage: avoid maybe-uninitialized warning by GCC 10

 src/qemu/qemu_migration.c                  | 2 +-
 src/storage/storage_backend_iscsi_direct.c | 8 ++++----
 tools/vsh.c                                | 2 +-
 3 files changed, 6 insertions(+), 6 deletions(-)

-- 
2.25.1

Re: [PATCH 0/3] Avoid some GCC 10 warnings

Posted by Erik Skultety 3 years, 8 months ago

On Thu, Aug 13, 2020 at 04:03:43PM +0200, Boris Fiuczynski wrote:
> Caught these when switching to F32 using GCC v10.2.1 on s390x.
>
> Boris Fiuczynski (3):
>   qemu: avoid maybe-uninitialized warning by GCC 10
>   tools: avoid potential null pointer dereference by GCC 10
>   storage: avoid maybe-uninitialized warning by GCC 10
>
>  src/qemu/qemu_migration.c                  | 2 +-
>  src/storage/storage_backend_iscsi_direct.c | 8 ++++----
>  tools/vsh.c                                | 2 +-
>  3 files changed, 6 insertions(+), 6 deletions(-)
>
> --
> 2.25.1
>

Reviewed-by: Erik Skultety <eskultet@redhat.com>

Re: [PATCH 0/3] Avoid some GCC 10 warnings

Posted by Marc Hartmayer 3 years, 8 months ago

On Mon, Aug 17, 2020 at 08:22 AM +0200, Erik Skultety <eskultet@redhat.com> wrote:
> On Thu, Aug 13, 2020 at 04:03:43PM +0200, Boris Fiuczynski wrote:
>> Caught these when switching to F32 using GCC v10.2.1 on s390x.
>>
>> Boris Fiuczynski (3):
>>   qemu: avoid maybe-uninitialized warning by GCC 10
>>   tools: avoid potential null pointer dereference by GCC 10
>>   storage: avoid maybe-uninitialized warning by GCC 10
>>
>>  src/qemu/qemu_migration.c                  | 2 +-
>>  src/storage/storage_backend_iscsi_direct.c | 8 ++++----
>>  tools/vsh.c                                | 2 +-
>>  3 files changed, 6 insertions(+), 6 deletions(-)
>>
>> --
>> 2.25.1
>>
>
> Reviewed-by: Erik Skultety <eskultet@redhat.com>
>

Thanks.

-- 
Kind regards / Beste Grüße
   Marc Hartmayer

IBM Deutschland Research & Development GmbH
Vorsitzender des Aufsichtsrats: Gregor Pillen 
Geschäftsführung: Dirk Wittkopp
Sitz der Gesellschaft: Böblingen
Registergericht: Amtsgericht Stuttgart, HRB 243294