From ef42202ac8ed27e7ff1fc90ef8bc2590046dff25 Mon Sep 17 00:00:00 2001
From: Zbigniew Jędrzejewski-Szmek <zbyszek@in.waw.pl>
Date: Mon, 22 Apr 2013 23:12:15 -0400
Subject: Add set_consume which always takes ownership

Freeing in error path is the common pattern with set_put().
---
 src/core/manager.c | 6 ++----
 1 file changed, 2 insertions(+), 4 deletions(-)

(limited to 'src/core/manager.c')

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);
-- 
cgit v1.2.3-54-g00ecf