fix update time settings, fixes #2895

diff --git a/src/htsmsg.c b/src/htsmsg.c
index c0112a37b256fbb921d8f50d43b328d206fd7fcc..0b7274472af3f093131c85ba4a6845bef00e46ef 100644 (file)
--- a/src/htsmsg.c
+++ b/src/htsmsg.c
@@ -402,6 +402,18 @@ htsmsg_field_get_s64
  *
  */
 
+int
+bool_check(const char *str)
+{
+  if (str &&
+      (!strcmp(str, "yes")  ||
+       !strcmp(str, "true") ||
+       !strcmp(str, "on") ||
+       !strcmp(str, "1")))
+    return 1;
+  return 0;
+}
+
 int
 htsmsg_field_get_bool
   ( htsmsg_field_t *f, int *boolp )
@@ -410,13 +422,7 @@ htsmsg_field_get_bool
   default:
     return HTSMSG_ERR_CONVERSION_IMPOSSIBLE;
   case HMF_STR:
-    if (!strcmp(f->hmf_str, "yes")  ||
-        !strcmp(f->hmf_str, "true") ||
-        !strcmp(f->hmf_str, "on") ||
-        !strcmp(f->hmf_str, "1"))
-      *boolp = 1;
-    else
-      *boolp = 0;
+    *boolp = bool_check(f->hmf_str);
     break;
   case HMF_S64:
     *boolp = f->hmf_s64 ? 1 : 0;

diff --git a/src/htsmsg.h b/src/htsmsg.h
index 993d213dbe73fa5a347527cdec69192ce1b0e7b2..4b768830893c8434bccbe6c29573c6cd3d25ab4d 100644 (file)
--- a/src/htsmsg.h
+++ b/src/htsmsg.h
@@ -231,6 +231,8 @@ int htsmsg_field_get_s64(htsmsg_field_t *f, int64_t *s64p);
  */
 int64_t htsmsg_get_s64_or_default(htsmsg_t *msg, const char *name, int64_t def);
 
+int bool_check(const char *str);
+
 int htsmsg_field_get_bool(htsmsg_field_t *f, int *boolp);
 
 int htsmsg_get_bool(htsmsg_t *msg, const char *name, int *boolp);

diff --git a/src/webui/extjs.c b/src/webui/extjs.c
index 769c8e6c944247251fe9425349a98654f97885df..64aab9fe7f858efa4cf77f6b25c05218d20faa83 100644 (file)
--- a/src/webui/extjs.c
+++ b/src/webui/extjs.c
@@ -518,9 +518,9 @@ extjs_config(http_connection_t *hc, const char *remain, void *opaque)
 
     /* Time */
     str = http_arg_get(&hc->hc_req_args, "tvhtime_update_enabled");
-    tvhtime_set_update_enabled(!!str);
+    tvhtime_set_update_enabled(bool_check(str));
     str = http_arg_get(&hc->hc_req_args, "tvhtime_ntp_enabled");
-    tvhtime_set_ntp_enabled(!!str);
+    tvhtime_set_ntp_enabled(bool_check(str));
     if ((str = http_arg_get(&hc->hc_req_args, "tvhtime_tolerance")))
       tvhtime_set_tolerance(atoi(str));