Re: [PATCH v2] hw/ppc/ppc405_boards: Initialize g_autofree pointer

[Daniel Henrique Barboza]

Queued in gitlab.com/danielhb/qemu/tree/ppc-next. Thanks,


Daniel

On 4/5/22 09:35, Bernhard Beschow wrote:

Resolves the only compiler warning when building a full QEMU under Arch Linux:

   Compiling C object libqemu-ppc-softmmu.fa.p/hw_ppc_ppc405_boards.c.o
   In file included from /usr/include/glib-2.0/glib.h:114,
                    from qemu/include/glib-compat.h:32,
                    from qemu/include/qemu/osdep.h:132,
                    from ../src/hw/ppc/ppc405_boards.c:25:
   ../src/hw/ppc/ppc405_boards.c: In function ‘ref405ep_init’:
   /usr/include/glib-2.0/glib/glib-autocleanups.h:28:3: warning: ‘filename’ may 
be used uninitialized in this function [-Wmaybe-uninitialized]
      28 |   g_free (*pp);
         |   ^~~~~~~~~~~~
   ../src/hw/ppc/ppc405_boards.c:265:26: note: ‘filename’ was declared here
     265 |         g_autofree char *filename;
         |                          ^~~~~~~~

Signed-off-by: Bernhard Beschow <shen...@gmail.com>
---
  hw/ppc/ppc405_boards.c | 4 ++--
  1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/hw/ppc/ppc405_boards.c b/hw/ppc/ppc405_boards.c
index 7e1a4ac955..3bed7002d2 100644
--- a/hw/ppc/ppc405_boards.c
+++ b/hw/ppc/ppc405_boards.c
@@ -262,13 +262,13 @@ static void ref405ep_init(MachineState *machine)
      /* allocate and load BIOS */
      if (machine->firmware) {
          MemoryRegion *bios = g_new(MemoryRegion, 1);
-        g_autofree char *filename;
+        g_autofree char *filename = qemu_find_file(QEMU_FILE_TYPE_BIOS,
+                                                   machine->firmware);
          long bios_size;

memory_region_init_rom(bios, NULL, "ef405ep.bios", BIOS_SIZE,

                                 &error_fatal);

- filename = qemu_find_file(QEMU_FILE_TYPE_BIOS, machine->firmware);

          if (!filename) {
              error_report("Could not find firmware '%s'", machine->firmware);
              exit(1);