diff options
author | Peter Maydell <peter.maydell@linaro.org> | 2018-12-14 13:30:54 +0000 |
---|---|---|
committer | Peter Maydell <peter.maydell@linaro.org> | 2018-12-14 13:30:54 +0000 |
commit | c0983085d127b1efb287337de26679c051abda07 (patch) | |
tree | c1e65a55d8caf91b6a8e471164b4e4f9c9a93579 /tests | |
parent | 7081e9b6b2d695c4367e6b9ed3e852bcb6f42907 (diff) |
hw/sd/sdhci: Don't leak memory region in sdhci_sysbus_realize()
In sdhci_sysbus_realize() we override the initialization of
s->iomem that sdhci_common_realize() performs. However we
don't destroy the old memory region before reinitializing
it, which means that the memory allocated for mr->name in
memory_region_do_init() is leaked.
Since sdhci_initfn() already initializes s->io_ops to
&sdhci_mmio_ops, always use that in sdhci_common_realize()
and remove the now-unnecessary reinitialization of the
MMIO region from sdhci_sysbus_realize().
Spotted by clang's leak sanitizer.
Signed-off-by: Peter Maydell <peter.maydell@linaro.org>
Reviewed-by: Richard Henderson <richard.henderson@linaro.org>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Message-id: 20181204132952.2601-4-peter.maydell@linaro.org
Diffstat (limited to 'tests')
0 files changed, 0 insertions, 0 deletions