On Fri, Jun 10, 2022 at 4:04 PM Philippe Mathieu-Daudé <f4...@amsat.org>
wrote:

> On 5/6/22 17:19, Bernhard Beschow wrote:
> > Fixes compilation due to false positives with -Werror:
> >
> >    In file included from /usr/include/glib-2.0/glib.h:114,
> >                     from qemu/src/include/glib-compat.h:32,
> >                     from qemu/src/include/qemu/osdep.h:144,
> >                     from ../src/hw/mips/boston.c:20:
> >    In function ‘g_autoptr_cleanup_generic_gfree’,
> >        inlined from ‘boston_mach_init’ at ../src/hw/mips/boston.c:790:52:
> >    /usr/include/glib-2.0/glib/glib-autocleanups.h:28:3: error:
> ‘dtb_load_data’ may be used uninitialized [-Werror=maybe-uninitialized]
> >       28 |   g_free (*pp);
> >          |   ^~~~~~~~~~~~
> >    ../src/hw/mips/boston.c: In function ‘boston_mach_init’:
> >    ../src/hw/mips/boston.c:790:52: note: ‘dtb_load_data’ was declared
> here
> >      790 |             g_autofree const void *dtb_file_data,
> *dtb_load_data;
> >          |
> ^~~~~~~~~~~~~
> >    In function ‘g_autoptr_cleanup_generic_gfree’,
> >      inlined from ‘boston_mach_init’ at ../src/hw/mips/boston.c:790:36:
> >    /usr/include/glib-2.0/glib/glib-autocleanups.h:28:3: error:
> ‘dtb_file_data’ may be used uninitialized [-Werror=maybe-uninitialized]
> >       28 |   g_free (*pp);
> >          |   ^~~~~~~~~~~~
> >    ../src/hw/mips/boston.c: In function ‘boston_mach_init’:
> >    ../src/hw/mips/boston.c:790:36: note: ‘dtb_file_data’ was declared
> here
> >      790 |             g_autofree const void *dtb_file_data,
> *dtb_load_data;
> >          |                                    ^~~~~~~~~~~~~
> >    cc1: all warnings being treated as errors
> >
> > Signed-off-by: Bernhard Beschow <shen...@gmail.com>
> > ---
> >   hw/mips/boston.c | 3 ++-
> >   1 file changed, 2 insertions(+), 1 deletion(-)
>
> Queued via mips-next.
>

Hi Phil,

shouldn't this patch be visible in your pull request then?
https://patchew.org/QEMU/20220611103312.67773-1-philippe.mathieu.da...@gmail.com/

Best regards,
Bernhard

Reply via email to