On 05/09/2018 11:26 AM, Kevin Wolf wrote:
This moves freeing the Job object and its fields from block_job_unref()
to job_delete().

Signed-off-by: Kevin Wolf <kw...@redhat.com>
---
  include/qemu/job.h | 3 +++
  blockjob.c         | 3 +--
  job.c              | 6 ++++++
  3 files changed, 10 insertions(+), 2 deletions(-)


+
+void job_delete(Job *job)
+{
+    g_free(job->id);
+    g_free(job);
+}

Should this be free-like, acting as a no-op when job == NULL on input?

--
Eric Blake, Principal Software Engineer
Red Hat, Inc.           +1-919-301-3266
Virtualization:  qemu.org | libvirt.org

Reply via email to