diff options
| author | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2013-04-22 23:12:15 -0400 |
|---|---|---|
| committer | Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl> | 2013-04-24 00:25:04 -0400 |
| commit | ef42202ac8ed27e7ff1fc90ef8bc2590046dff25 (patch) | |
| tree | b1646eeb8ef5070337ae701ffb4abf0e398640ad /src/core/manager.c | |
| parent | c79bb9e4e2e5b96b2ae2c432bf8b0ff9674fce60 (diff) | |
Add set_consume which always takes ownership
Freeing in error path is the common pattern with set_put().
Diffstat (limited to 'src/core/manager.c')
| -rw-r--r-- | src/core/manager.c | 6 |
1 files changed, 2 insertions, 4 deletions
diff --git a/src/core/manager.c b/src/core/manager.c index 208b240ba1..b1a22890fb 100644 --- a/src/core/manager.c +++ b/src/core/manager.c @@ -806,11 +806,9 @@ static void manager_build_unit_path_cache(Manager *m) { goto fail; } - r = set_put(m->unit_path_cache, p); - if (r < 0) { - free(p); + r = set_consume(m->unit_path_cache, p); + if (r < 0) goto fail; - } } closedir(d); |