From 549c7ef9e7c8045bed861cd913126cff930ca809 Mon Sep 17 00:00:00 2001 From: Jan Polensky Date: Tue, 4 Mar 2025 15:26:08 +0100 Subject: [PATCH] zkey/kms.c: Use correct variable in warnx() MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The variable filename was freed inside the loop and then incorrectly used in the subsequent warnx() call. Replacing it with directory reflects the indented context and avoids use after free. Fixes: f25aaf32b85d ("zkey: Add support for key management system plugins") Reviewed-by: Ingo Franzki Reviewed-by: Marc Hartmayer Signed-off-by: Jan Polensky Signed-off-by: Jan Höppner --- zkey/kms.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) diff --git a/zkey/kms.c b/zkey/kms.c index 294625b6..4ab82b7c 100644 --- a/zkey/kms.c +++ b/zkey/kms.c @@ -745,7 +745,7 @@ static int remove_directory_recursively(const char *directory) if (rmdir(directory) != 0) { rc = -errno; - warnx("Failed to remove '%s': %s", filename, strerror(-rc)); + warnx("Failed to remove '%s': %s", directory, strerror(-rc)); goto out; }