From fbaf3b23ae4aa79110ebd37aada70ce6a044c692 Mon Sep 17 00:00:00 2001 From: Yu Watanabe Date: Sat, 12 Aug 2023 15:25:03 +0900 Subject: [PATCH] core: set $CREDENTIALS_DIRECTORY only when we set up credentials It is not necessary to set the environment variable on an execution step we do not support credentials. --- src/core/execute.c | 4 +++- 1 file changed, 3 insertions(+), 1 deletion(-) diff --git a/src/core/execute.c b/src/core/execute.c index 14e196d4c7a..78d8988e8b1 100644 --- a/src/core/execute.c +++ b/src/core/execute.c @@ -2041,7 +2041,9 @@ static int build_environment( our_env[n_env++] = x; } - if (exec_context_has_credentials(c) && p->prefix[EXEC_DIRECTORY_RUNTIME]) { + if (exec_context_has_credentials(c) && + p->prefix[EXEC_DIRECTORY_RUNTIME] && + FLAGS_SET(p->flags, EXEC_WRITE_CREDENTIALS)) { x = strjoin("CREDENTIALS_DIRECTORY=", p->prefix[EXEC_DIRECTORY_RUNTIME], "/credentials/", u->id); if (!x) return -ENOMEM; -- 2.47.3