diff options
| author | Jukka Rissanen <jukka.rissanen@linux.intel.com> | 2013-02-13 17:28:59 +0200 |
|---|---|---|
| committer | Patrik Flykt <patrik.flykt@linux.intel.com> | 2013-02-19 12:51:39 +0200 |
| commit | 0973548ef45ccc4c51bf1d091b725647c916f358 (patch) | |
| tree | e712800778bec6dd979d1c68780dfe786bcd7922 /vpn/vpn-provider.c | |
| parent | db653221c8fe867f05b5e05d47d07fe1e0e6e22a (diff) | |
| download | connman-0973548ef45ccc4c51bf1d091b725647c916f358.tar.gz connman-0973548ef45ccc4c51bf1d091b725647c916f358.tar.bz2 connman-0973548ef45ccc4c51bf1d091b725647c916f358.zip | |
vpn-provider: Avoid printing password to log files
Add a variant to vpn_provider_set_string() that does not print
sensitive data like password in clear text to log files.
Diffstat (limited to 'vpn/vpn-provider.c')
| -rw-r--r-- | vpn/vpn-provider.c | 19 |
1 files changed, 16 insertions, 3 deletions
diff --git a/vpn/vpn-provider.c b/vpn/vpn-provider.c index 36db938b..8b68dba3 100644 --- a/vpn/vpn-provider.c +++ b/vpn/vpn-provider.c @@ -2007,10 +2007,11 @@ const char * __vpn_provider_get_ident(struct vpn_provider *provider) return provider->identifier; } -int vpn_provider_set_string(struct vpn_provider *provider, - const char *key, const char *value) +static int set_string(struct vpn_provider *provider, + const char *key, const char *value, gboolean hide_value) { - DBG("provider %p key %s value %s", provider, key, value); + DBG("provider %p key %s value %s", provider, key, + hide_value ? "<not printed>" : value); if (g_str_equal(key, "Type") == TRUE) { g_free(provider->type); @@ -2030,6 +2031,18 @@ int vpn_provider_set_string(struct vpn_provider *provider, return 0; } +int vpn_provider_set_string(struct vpn_provider *provider, + const char *key, const char *value) +{ + return set_string(provider, key, value, FALSE); +} + +int vpn_provider_set_string_hide_value(struct vpn_provider *provider, + const char *key, const char *value) +{ + return set_string(provider, key, value, TRUE); +} + const char *vpn_provider_get_string(struct vpn_provider *provider, const char *key) { |