From b852e57ba2f0c69688755aa1df97ec51f5988e3f Mon Sep 17 00:00:00 2001 From: Jukka Rissanen Date: Wed, 22 Aug 2012 11:28:01 +0300 Subject: [PATCH] vpn: Accept also "true" or "1" as a boolean value Currently user is able to set the boolean VPN config value only by setting the variable to "yes". This is not very intuitive so accept also the value of "true" or "1". --- plugins/l2tp.c | 4 +++- plugins/pptp.c | 4 +++- plugins/vpnc.c | 4 +++- 3 files changed, 9 insertions(+), 3 deletions(-) diff --git a/plugins/l2tp.c b/plugins/l2tp.c index 5a655e6..39c9618 100644 --- a/plugins/l2tp.c +++ b/plugins/l2tp.c @@ -285,7 +285,9 @@ static ssize_t l2tp_write_bool_option(int fd, ssize_t ret = 0; if (key != NULL && value != NULL) { - if (strcmp(value, "yes") == 0) { + if (strcasecmp(value, "yes") == 0 || + strcasecmp(value, "true") == 0 || + strcmp(value, "1") == 0) { buf = g_strdup_printf("%s\n", key); ret = full_write(fd, buf, strlen(buf)); diff --git a/plugins/pptp.c b/plugins/pptp.c index 9db1b89..365bde7 100644 --- a/plugins/pptp.c +++ b/plugins/pptp.c @@ -225,7 +225,9 @@ static void pptp_write_bool_option(struct connman_task *task, const char *key, const char *value) { if (key != NULL && value != NULL) { - if (strcmp(value, "yes") == 0) + if (strcasecmp(value, "yes") == 0 || + strcasecmp(value, "true") == 0 || + strcmp(value, "1") == 0) connman_task_add_argument(task, key, NULL); } } diff --git a/plugins/vpnc.c b/plugins/vpnc.c index fea1c15..6b7a02b 100644 --- a/plugins/vpnc.c +++ b/plugins/vpnc.c @@ -194,7 +194,9 @@ static ssize_t write_bool_option(int fd, const char *key, const char *value) ssize_t ret = 0; if (key != NULL && value != NULL) { - if (strcmp(value, "yes") == 0) { + if (strcasecmp(value, "yes") == 0 || + strcasecmp(value, "true") == 0 || + strcmp(value, "1") == 0) { buf = g_strdup_printf("%s\n", key); ret = full_write(fd, buf, strlen(buf)); -- 2.7.4