Patchwork migration: use qemu_free() instead of free().

login
register
mail settings
Submitter Yoshiaki Tamura
Date June 9, 2010, 5:44 a.m.
Message ID <1276062271-27678-1-git-send-email-tamura.yoshiaki@lab.ntt.co.jp>
Download mbox | patch
Permalink /patch/55051/
State New
Headers show

Comments

Yoshiaki Tamura - June 9, 2010, 5:44 a.m.
Although there is no difference, other migration related code use
qemu_free(), and it should be better to be consistent.

Signed-off-by: Yoshiaki Tamura <tamura.yoshiaki@lab.ntt.co.jp>
---
 migration.c |    2 +-
 1 files changed, 1 insertions(+), 1 deletions(-)
Anthony Liguori - June 14, 2010, 8:58 p.m.
On 06/09/2010 12:44 AM, Yoshiaki Tamura wrote:
> Although there is no difference, other migration related code use
> qemu_free(), and it should be better to be consistent.
>
> Signed-off-by: Yoshiaki Tamura<tamura.yoshiaki@lab.ntt.co.jp>
>    

Applied.  Thanks.

Regards,

Anthony Liguori
> ---
>   migration.c |    2 +-
>   1 files changed, 1 insertions(+), 1 deletions(-)
>
> diff --git a/migration.c b/migration.c
> index fbf2339..64ed36e 100644
> --- a/migration.c
> +++ b/migration.c
> @@ -396,7 +396,7 @@ void migrate_fd_release(MigrationState *mig_state)
>           s->state = MIG_STATE_CANCELLED;
>           migrate_fd_cleanup(s);
>       }
> -    free(s);
> +    qemu_free(s);
>   }
>
>   void migrate_fd_wait_for_unfreeze(void *opaque)
>

Patch

diff --git a/migration.c b/migration.c
index fbf2339..64ed36e 100644
--- a/migration.c
+++ b/migration.c
@@ -396,7 +396,7 @@  void migrate_fd_release(MigrationState *mig_state)
         s->state = MIG_STATE_CANCELLED;
         migrate_fd_cleanup(s);
     }
-    free(s);
+    qemu_free(s);
 }
 
 void migrate_fd_wait_for_unfreeze(void *opaque)