diff options
Diffstat (limited to '')
-rw-r--r-- | misc.c | 13 | ||||
-rw-r--r-- | misc.h | 3 | ||||
-rw-r--r-- | plugin.c | 5 |
3 files changed, 19 insertions, 2 deletions
@@ -770,7 +770,8 @@ env_set_print (int msglevel, const struct env_set *es) while (e) { - msg (msglevel, "ENV [%d] '%s'", i, e->string); + if (env_safe_to_print (e->string)) + msg (msglevel, "ENV [%d] '%s'", i, e->string); ++i; e = e->next; } @@ -1454,6 +1455,16 @@ safe_print (const char *str, struct gc_arena *gc) return string_mod_const (str, CC_PRINT, CC_CRLF, '.', gc); } +bool +env_safe_to_print (const char *str) +{ +#ifndef UNSAFE_DEBUG + if (strncmp (str, "password", 8) == 0) + return false; +#endif + return true; +} + /* Make arrays of strings */ const char ** @@ -265,6 +265,9 @@ void purge_user_pass (struct user_pass *up, const bool force); */ const char *safe_print (const char *str, struct gc_arena *gc); +/* returns true if environmental variable safe to print to log */ +bool env_safe_to_print (const char *str); + /* * A sleep function that services the management layer for n * seconds rather than doing nothing. @@ -43,7 +43,10 @@ plugin_show_string_array (int msglevel, const char *name, const char *array[]) { int i; for (i = 0; array[i]; ++i) - msg (msglevel, "%s[%d] = '%s'", name, i, array[i]); + { + if (env_safe_to_print (array[i])) + msg (msglevel, "%s[%d] = '%s'", name, i, array[i]); + } } static void |