Re: [PATCH] hw/i386/fw_cfg: Use g_new() and g_new0() instead of g_malloc()

["Michael S. Tsirkin" <mst@redhat.com>]

On Mon, Apr 27, 2026 at 12:02:57PM +0200, Thomas Huth wrote:
> On 24/02/2026 17.00, Sourish Dutta Sharma wrote:
> > Replace g_malloc() and g_malloc0() calls that calculate the allocation
> > size using sizeof() with the type-safe g_new() and g_new0() macros.
> > 
> > This aligns the code with QEMU's coding style guidelines, improving
> > readability and protecting against potential integer overflow
> > vulnerabilities when allocating arrays.
> > 
> > Signed-off-by: Sourish Duttta Sharma  <sourishduttasharma770@gmail.com>
> > ---
> >   hw/i386/fw_cfg.c | 4 ++--
> >   1 file changed, 2 insertions(+), 2 deletions(-)
> 
>  Hi!
> 
> Thanks for your patch! FWIW, fw_cfg is not really my turf, but since I was
> looking into malloc() related patches today, I noticed your patch which
> seems to have fallen through the cracks (sorry for this!), so I picked it up
> and added it to my latest pull request.
> 
>  Regards,
>   Thomas

Thanks for doing this. FWIW

Reviewed-by: Michael S. Tsirkin <mst@redhat.com>