vfio: platform: reset: calxedaxgmac: fix ioaddr leak

In the current code the vfio_platform_region is copied on the stack.
As a consequence the ioaddr address is not iounmapped in the vfio
platform driver (vfio_platform_regions_cleanup). The patch uses the
pointer to the region instead.

Signed-off-by: Eric Auger <eric.auger@linaro.org>
Signed-off-by: Alex Williamson <alex.williamson@redhat.com>

1 files changed, 7 insertions, 7 deletions

diff --git a/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c b/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
index 640f5d87d422..e3d3d948e661 100644
--- a/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
+++ b/drivers/vfio/platform/reset/vfio_platform_calxedaxgmac.c
@@ -59,20 +59,20 @@ static inline void xgmac_mac_disable(void __iomem *ioaddr)
 
 int vfio_platform_calxedaxgmac_reset(struct vfio_platform_device *vdev)
 {
-        struct vfio_platform_region reg = vdev->regions[0];
+        struct vfio_platform_region *reg = &vdev->regions[0];
 
-        if (!reg.ioaddr) {
-                reg.ioaddr =
-                        ioremap_nocache(reg.addr, reg.size);
-                if (!reg.ioaddr)
+        if (!reg->ioaddr) {
+                reg->ioaddr =
+                        ioremap_nocache(reg->addr, reg->size);
+                if (!reg->ioaddr)
                         return -ENOMEM;
         }
 
         /* disable IRQ */
-        writel(0, reg.ioaddr + XGMAC_DMA_INTR_ENA);
+        writel(0, reg->ioaddr + XGMAC_DMA_INTR_ENA);
 
         /* Disable the MAC core */
-        xgmac_mac_disable(reg.ioaddr);
+        xgmac_mac_disable(reg->ioaddr);
 
         return 0;
 }