From: Michal Sekletar <msekleta@redhat.com>
Date: Wed, 25 Feb 2026 18:45:55 +0000 (+0100)
Subject: core: cleanup unit's dropin directories from global cache
X-Git-Tag: v260-rc3~81
X-Git-Url: http://git.ipfire.org/cgi-bin/gitweb.cgi?a=commitdiff_plain;h=0c98e432d1def1e8428dbead50dc629ed0645366;p=thirdparty%2Fsystemd.git

core: cleanup unit's dropin directories from global cache

When user creates dropin files via API (e.g. systemctl set-property ...)
we put the dropin directory path into unit_path_cache. Drop those
directories from the cache in unit_free() and prevent memory leak.

Follow-up for fce94c5c563b8f6ede2b8f7f283d2d2faff4e062.
---

diff --git a/src/core/unit.c b/src/core/unit.c
index bb3430186ca..ab0db256878 100644
--- a/src/core/unit.c
+++ b/src/core/unit.c
@@ -677,6 +677,8 @@ static void unit_remove_transient(Unit *u) {
         if (!u->transient)
                 return;
 
+        const char *dropin_directory = strjoina(u->id, ".d");
+
         STRV_FOREACH(i, u->dropin_paths) {
                 _cleanup_free_ char *p = NULL, *pp = NULL;
 
@@ -690,6 +692,10 @@ static void unit_remove_transient(Unit *u) {
                 if (!path_equal(u->manager->lookup_paths.transient, pp))
                         continue;
 
+                /* Drop the transient drop-in directory also from unit path cache. */
+                if (path_equal(last_path_component(p), dropin_directory))
+                        free(set_remove(u->manager->unit_path_cache, p));
+
                 (void) unlink(*i);
                 (void) rmdir(p);
         }