From 5b10ddf3589bdd6ef9cfc63a807ae91dc0e4095f Mon Sep 17 00:00:00 2001 From: Eric Engestrom Date: Sun, 7 Jul 2019 11:40:04 +0100 Subject: [PATCH] util: fix mem leak of program path MIME-Version: 1.0 Content-Type: text/plain; charset=utf8 Content-Transfer-Encoding: 8bit Fixes: 759b94038987bb983398 ("util: Get program name based on path when possible") Signed-off-by: Eric Engestrom Reviewed-by: Tapani Pälli --- src/util/u_process.c | 19 ++++++++++++------- 1 file changed, 12 insertions(+), 7 deletions(-) diff --git a/src/util/u_process.c b/src/util/u_process.c index cbccf48ed01..371335303ec 100644 --- a/src/util/u_process.c +++ b/src/util/u_process.c @@ -33,6 +33,15 @@ #undef GET_PROGRAM_NAME #if defined(__linux__) && defined(HAVE_PROGRAM_INVOCATION_NAME) + +static char *path = NULL; + +static void __freeProgramPath() +{ + free(path); + path = NULL; +} + static const char * __getProgramName() { @@ -45,14 +54,10 @@ __getProgramName() * Strip these arguments out by using the realpath only if it was * a prefix of the invocation name. */ - static char *path; - - if (!path) - /* Note: realpath() allocates memory that we will keep around for - * the lifetime of the app, and then leak as the app closes. - * FIXME: we should find a way to clean this properly - */ + if (!path) { path = realpath("/proc/self/exe", NULL); + atexit(__freeProgramPath); + } if (path && strncmp(path, program_invocation_name, strlen(path)) == 0) { /* This shouldn't be null because path is a a prefix, -- 2.30.2