On 2016年10月08日 20:07, Li Qiang wrote:
From: Li Qiang <liqiang...@360.cn>

The exit dispatch of eepro100 network card device doesn't free
the 's->vmstate' field which was allocated in device realize thus
leading a host memory leak. This patch avoid this.

Signed-off-by: Li Qiang <liqiang...@360.cn>
---
  hw/net/eepro100.c | 1 +
  1 file changed, 1 insertion(+)

diff --git a/hw/net/eepro100.c b/hw/net/eepro100.c
index bab4dbf..4bf71f2 100644
--- a/hw/net/eepro100.c
+++ b/hw/net/eepro100.c
@@ -1843,6 +1843,7 @@ static void pci_nic_uninit(PCIDevice *pci_dev)
      EEPRO100State *s = DO_UPCAST(EEPRO100State, dev, pci_dev);
vmstate_unregister(&pci_dev->qdev, s->vmstate, s);
+    g_free(s->vmstate);
      eeprom93xx_free(&pci_dev->qdev, s->eeprom);
      qemu_del_nic(s->nic);
  }

Applied, thanks.

We may want to switch to use dc->vmsd instead of this in the future.

Reply via email to