From: Kevin Wolf <kwolf@redhat.com>
Date: Thu, 12 Apr 2018 17:06:53 +0000 (+0200)
Subject: job: Add job_delete()
X-Git-Tag: v3.0.0-rc0~138^2~34
X-Git-Url: http://git.ipfire.org/?a=commitdiff_plain;h=fd61a701f1de8e4c1d89b3716ba9ca749cf5c724;p=thirdparty%2Fqemu.git

job: Add job_delete()

This moves freeing the Job object and its fields from block_job_unref()
to job_delete().

Signed-off-by: Kevin Wolf <kwolf@redhat.com>
Reviewed-by: Max Reitz <mreitz@redhat.com>
Reviewed-by: John Snow <jsnow@redhat.com>
---

diff --git a/blockjob.c b/blockjob.c
index ea71ec01298..430a67ba638 100644
--- a/blockjob.c
+++ b/blockjob.c
@@ -261,9 +261,8 @@ void block_job_unref(BlockJob *job)
                                         block_job_detach_aio_context, job);
         blk_unref(job->blk);
         error_free(job->blocker);
-        g_free(job->job.id);
         assert(!timer_pending(&job->sleep_timer));
-        g_free(job);
+        job_delete(&job->job);
     }
 }
 
diff --git a/include/qemu/job.h b/include/qemu/job.h
index 279ce688fda..43dc2e4a7d3 100644
--- a/include/qemu/job.h
+++ b/include/qemu/job.h
@@ -62,6 +62,9 @@ struct JobDriver {
  */
 void *job_create(const char *job_id, const JobDriver *driver, Error **errp);
 
+/** Frees the @job object. */
+void job_delete(Job *job);
+
 /** Returns the JobType of a given Job. */
 JobType job_type(const Job *job);
 
diff --git a/job.c b/job.c
index 83724a43deb..cfdd008c52d 100644
--- a/job.c
+++ b/job.c
@@ -56,3 +56,9 @@ void *job_create(const char *job_id, const JobDriver *driver, Error **errp)
 
     return job;
 }
+
+void job_delete(Job *job)
+{
+    g_free(job->id);
+    g_free(job);
+}