commit 7650c8fe520c67c3b36f6962c4ad990f56ad40b8
author Philippe Mathieu-Daudé <philmd@redhat.com> Fri Aug 19 16:39:27 2022 +0100
committer Peter Maydell <peter.maydell@linaro.org> Thu Sep 22 16:38:28 2022 +0100
tree 1e8fceae7f953f303ea6e676f24892651b246531
parent a580fdcd609e1db77ef5a1cbcbfd2af5ca05c939

hw/intc/xics: Avoid dynamic stack allocation

Use autofree heap allocation instead of variable-length
array on the stack.

Signed-off-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Acked-by: David Gibson <david@gibson.dropbear.id.au>
Reviewed-by: Greg Kurz <groug@kaod.org>
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Message-id: 20220819153931.3147384-8-peter.maydell@linaro.org

hw/intc/xics.c

diff --git a/hw/intc/xics.c b/hw/intc/xics.c
index 5b0b4d9..dcd021a 100644
--- a/hw/intc/xics.c
+++ b/hw/intc/xics.c
@@ -567,8 +567,8 @@
 static void ics_reset(DeviceState *dev)
 {
     ICSState *ics = ICS(dev);
+    g_autofree uint8_t *flags = g_malloc(ics->nr_irqs);
     int i;
-    uint8_t flags[ics->nr_irqs];
 
     for (i = 0; i < ics->nr_irqs; i++) {
         flags[i] = ics->irqs[i].flags;