diff options
| author | Lennart Poettering <lennart@poettering.net> | 2018-12-10 18:52:11 +0100 |
|---|---|---|
| committer | Lennart Poettering <lennart@poettering.net> | 2018-12-12 11:14:26 +0100 |
| commit | 728ba51e98959dfd9ad9f70cf6526b0a84b529a0 (patch) | |
| tree | 90510fb1c908fe4335d9379d25dafd6ac0f37dda /src/core | |
| parent | Merge pull request #11122 from keszybz/tmpfiles-man (diff) | |
| download | systemd-728ba51e98959dfd9ad9f70cf6526b0a84b529a0.tar.xz systemd-728ba51e98959dfd9ad9f70cf6526b0a84b529a0.zip | |
job: update job_free() to follow our usual return-NULL style
Diffstat (limited to 'src/core')
| -rw-r--r-- | src/core/job.c | 4 | ||||
| -rw-r--r-- | src/core/job.h | 2 |
2 files changed, 3 insertions, 3 deletions
diff --git a/src/core/job.c b/src/core/job.c index af5070b8cf..0f224f2691 100644 --- a/src/core/job.c +++ b/src/core/job.c @@ -89,7 +89,7 @@ void job_unlink(Job *j) { j->timer_event_source = sd_event_source_unref(j->timer_event_source); } -void job_free(Job *j) { +Job* job_free(Job *j) { assert(j); assert(!j->installed); assert(!j->transaction_prev); @@ -102,7 +102,7 @@ void job_free(Job *j) { sd_bus_track_unref(j->bus_track); strv_free(j->deserialized_clients); - free(j); + return mfree(j); } static void job_set_state(Job *j, JobState state) { diff --git a/src/core/job.h b/src/core/job.h index 5f9c14012f..1f5bce85a4 100644 --- a/src/core/job.h +++ b/src/core/job.h @@ -162,7 +162,7 @@ struct Job { Job* job_new(Unit *unit, JobType type); Job* job_new_raw(Unit *unit); void job_unlink(Job *job); -void job_free(Job *job); +Job* job_free(Job *job); Job* job_install(Job *j); int job_install_deserialized(Job *j); void job_uninstall(Job *j); |