* [Qemu-devel] [PATCH] usbredir: free vm_change_state_handler in usbredir destroy dispatch
@ 2016-11-08 5:57 Li Qiang
2016-11-08 6:30 ` Marc-André Lureau
0 siblings, 1 reply; 2+ messages in thread
From: Li Qiang @ 2016-11-08 5:57 UTC (permalink / raw)
To: kraxel, qemu-devel; +Cc: Li Qiang
From: Li Qiang <liqiang6-s@360.cn>
In usbredir destroy dispatch function, it doesn't free the vm change
state handler once registered in usbredir_realize function. This will
lead a memory leak issue. This patch avoid this.
Signed-off-by: Li Qiang <liqiang6-s@360.cn>
---
hw/usb/redirect.c | 5 ++++-
1 file changed, 4 insertions(+), 1 deletion(-)
diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c
index 528081e..a657237 100644
--- a/hw/usb/redirect.c
+++ b/hw/usb/redirect.c
@@ -132,6 +132,7 @@ struct USBRedirDevice {
struct usbredirfilter_rule *filter_rules;
int filter_rules_count;
int compatible_speedmask;
+ VMChangeStateEntry *vmstate;
};
#define TYPE_USB_REDIR "usb-redir"
@@ -1411,7 +1412,8 @@ static void usbredir_realize(USBDevice *udev, Error **errp)
usbredir_chardev_read, usbredir_chardev_event,
dev, NULL, true);
- qemu_add_vm_change_state_handler(usbredir_vm_state_change, dev);
+ dev->vmstate =
+ qemu_add_vm_change_state_handler(usbredir_vm_state_change, dev);
}
static void usbredir_cleanup_device_queues(USBRedirDevice *dev)
@@ -1450,6 +1452,7 @@ static void usbredir_handle_destroy(USBDevice *udev)
}
free(dev->filter_rules);
+ qemu_del_vm_change_state_handler(dev->vmstate);
}
static int usbredir_check_filter(USBRedirDevice *dev)
--
1.8.3.1
^ permalink raw reply related [flat|nested] 2+ messages in thread
* Re: [Qemu-devel] [PATCH] usbredir: free vm_change_state_handler in usbredir destroy dispatch
2016-11-08 5:57 [Qemu-devel] [PATCH] usbredir: free vm_change_state_handler in usbredir destroy dispatch Li Qiang
@ 2016-11-08 6:30 ` Marc-André Lureau
0 siblings, 0 replies; 2+ messages in thread
From: Marc-André Lureau @ 2016-11-08 6:30 UTC (permalink / raw)
To: Li Qiang, kraxel, qemu-devel; +Cc: Li Qiang
Hi
On Tue, Nov 8, 2016 at 9:58 AM Li Qiang <liq3ea@gmail.com> wrote:
> From: Li Qiang <liqiang6-s@360.cn>
>
> In usbredir destroy dispatch function, it doesn't free the vm change
> state handler once registered in usbredir_realize function. This will
> lead a memory leak issue. This patch avoid this.
>
> Signed-off-by: Li Qiang <liqiang6-s@360.cn>
>
Reviewed-by: Marc-André Lureau <marcandre.lureau@redhat.com>
> ---
> hw/usb/redirect.c | 5 ++++-
> 1 file changed, 4 insertions(+), 1 deletion(-)
>
> diff --git a/hw/usb/redirect.c b/hw/usb/redirect.c
> index 528081e..a657237 100644
> --- a/hw/usb/redirect.c
> +++ b/hw/usb/redirect.c
> @@ -132,6 +132,7 @@ struct USBRedirDevice {
> struct usbredirfilter_rule *filter_rules;
> int filter_rules_count;
> int compatible_speedmask;
> + VMChangeStateEntry *vmstate;
> };
>
> #define TYPE_USB_REDIR "usb-redir"
> @@ -1411,7 +1412,8 @@ static void usbredir_realize(USBDevice *udev, Error
> **errp)
> usbredir_chardev_read,
> usbredir_chardev_event,
> dev, NULL, true);
>
> - qemu_add_vm_change_state_handler(usbredir_vm_state_change, dev);
> + dev->vmstate =
> + qemu_add_vm_change_state_handler(usbredir_vm_state_change, dev);
> }
>
> static void usbredir_cleanup_device_queues(USBRedirDevice *dev)
> @@ -1450,6 +1452,7 @@ static void usbredir_handle_destroy(USBDevice *udev)
> }
>
> free(dev->filter_rules);
> + qemu_del_vm_change_state_handler(dev->vmstate);
> }
>
> static int usbredir_check_filter(USBRedirDevice *dev)
> --
> 1.8.3.1
>
>
> --
Marc-André Lureau
^ permalink raw reply [flat|nested] 2+ messages in thread
end of thread, other threads:[~2016-11-08 6:30 UTC | newest]
Thread overview: 2+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2016-11-08 5:57 [Qemu-devel] [PATCH] usbredir: free vm_change_state_handler in usbredir destroy dispatch Li Qiang
2016-11-08 6:30 ` Marc-André Lureau
This is an external index of several public inboxes,
see mirroring instructions on how to clone and mirror
all data and code used by this external index.