Correct return code from setting_get_priority().
authorIain Patterson <me@iain.cx>
Sat, 23 Jul 2016 11:01:52 +0000 (12:01 +0100)
committerIain Patterson <me@iain.cx>
Thu, 28 Jul 2016 15:44:19 +0000 (16:44 +0100)
The function was returning 1 when filling the destination buffer with
the default value.  It should return 0 when the value is default and 1
when it has been set by the administrator.

settings.cpp

index c85bab4..f50dcc4 100644 (file)
@@ -452,7 +452,9 @@ static int setting_get_priority(const TCHAR *service_name, void *param, const TC
 \r
   unsigned long constant;\r
   switch (get_number(key, (TCHAR *) name, &constant, false)) {\r
-    case 0: return value_from_string(name, value, (const TCHAR *) default_value);\r
+    case 0:\r
+      if (value_from_string(name, value, (const TCHAR *) default_value) == -1) return -1;\r
+      return 0;\r
     case -1: return -1;\r
   }\r
 \r