[Qemu-devel] [PATCH] virtio-serial-bus: Delete timer from list before free it

zhanghailiang posted 1 patch 7 years ago
Patches applied successfully (tree, apply log)
git fetch https://github.com/patchew-project/qemu tags/patchew/1488770972-322500-1-git-send-email-zhang.zhanghailiang@huawei.com
Test checkpatch passed
Test docker passed
hw/char/virtio-serial-bus.c | 1 +
1 file changed, 1 insertion(+)
[Qemu-devel] [PATCH] virtio-serial-bus: Delete timer from list before free it
Posted by zhanghailiang 7 years ago
Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
---
 hw/char/virtio-serial-bus.c | 1 +
 1 file changed, 1 insertion(+)

diff --git a/hw/char/virtio-serial-bus.c b/hw/char/virtio-serial-bus.c
index d544cd9..d797a67 100644
--- a/hw/char/virtio-serial-bus.c
+++ b/hw/char/virtio-serial-bus.c
@@ -724,6 +724,7 @@ static void virtio_serial_post_load_timer_cb(void *opaque)
         }
     }
     g_free(s->post_load->connected);
+    timer_del(s->post_load->timer);
     timer_free(s->post_load->timer);
     g_free(s->post_load);
     s->post_load = NULL;
-- 
1.8.3.1



Re: [Qemu-devel] [PATCH] virtio-serial-bus: Delete timer from list before free it
Posted by Paolo Bonzini 7 years ago

On 06/03/2017 04:29, zhanghailiang wrote:
> Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
> ---
>  hw/char/virtio-serial-bus.c | 1 +
>  1 file changed, 1 insertion(+)
> 
> diff --git a/hw/char/virtio-serial-bus.c b/hw/char/virtio-serial-bus.c
> index d544cd9..d797a67 100644
> --- a/hw/char/virtio-serial-bus.c
> +++ b/hw/char/virtio-serial-bus.c
> @@ -724,6 +724,7 @@ static void virtio_serial_post_load_timer_cb(void *opaque)
>          }
>      }
>      g_free(s->post_load->connected);
> +    timer_del(s->post_load->timer);
>      timer_free(s->post_load->timer);
>      g_free(s->post_load);
>      s->post_load = NULL;
> 

Reviewed-by: Paolo Bonzini <pbonzini@redhat.com>
Cc: qemu-stable@nongnu.org

Re: [Qemu-devel] [PATCH] virtio-serial-bus: Delete timer from list before free it
Posted by Amit Shah 7 years ago
On (Mon) 06 Mar 2017 [11:29:31], zhanghailiang wrote:
> Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>

Reviewed-by: Amit Shah <amit@kernel.org>

Michael, please pick this up.

		Amit
-- 
http://log.amitshah.net/

Re: [Qemu-devel] [PATCH] virtio-serial-bus: Delete timer from list before free it
Posted by Hailiang Zhang 7 years ago
ping... ?

On 2017/3/8 4:01, Amit Shah wrote:
> On (Mon) 06 Mar 2017 [11:29:31], zhanghailiang wrote:
>> Signed-off-by: zhanghailiang <zhang.zhanghailiang@huawei.com>
>
> Reviewed-by: Amit Shah <amit@kernel.org>
>
> Michael, please pick this up.
>
> 		Amit
>