From: Jukka Rissanen <jukka.rissanen@linux.intel.com>
Date: Wed, 22 Aug 2012 08:28:01 +0000 (+0300)
Subject: vpn: Accept also "true" or "1" as a boolean value
X-Git-Tag: 1.7~72
X-Git-Url: http://review.tizen.org/git/?a=commitdiff_plain;h=b852e57ba2f0c69688755aa1df97ec51f5988e3f;p=platform%2Fupstream%2Fconnman.git

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".
---

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));