pmqos: Fix bug of 'support' property of pass-pmqos.conf

This patch fixes the bug related to 'support' property of pass-pmqos.conf.

PASS PMQoS has the pass-pmqos.conf which contains the supported scenario
name and the number of supported scenarios. Each scenario has the 'support'
property indicating whether scenario is supported or not on platform.

If the 'support' property value is changed from 'yes' to 'no',
PASS shows the following error log during the init step.
[error log]
 edbus-handler.c: register_method(713) > fail to add method (null)!
 edbus-handler.c: register_edbus_method(862) > fail to register /Org/Tizen/System/Pass/Pmqos method(-22)
 pmqos.c: pmqos_init_done(395) > fail to init edbus method from conf(-22)

Change-Id: Icbb0d29c941345f21c3bcc4ae513ff73095a3aff
Signed-off-by: Chanwoo Choi <cw00.choi@samsung.com>

diff --git a/src/pmqos/pmqos.c b/src/pmqos/pmqos.c
index 26083ecf0174e9faeaef8bc68d5f69fba0c93f38..1869c00fed460875c4ec33b1c978476d204c9d8c 100644 (file)
--- a/src/pmqos/pmqos.c
+++ b/src/pmqos/pmqos.c
@@ -307,7 +307,7 @@ static int get_methods_from_conf(const char *path, struct edbus_method **edbus_m
 {
        struct edbus_method *methods;
        struct pmqos_scenario scenarios = {0,};
-       int i, ret;
+       int i, ret, count = 0;
 
        /* get pmqos table from conf */
        ret = pmqos_get_scenario(path, &scenarios);
@@ -334,20 +334,21 @@ static int get_methods_from_conf(const char *path, struct edbus_method **edbus_m
 
        /* set edbus_methods structure */
        for (i = 0; i < scenarios.num; ++i) {
-
                /* if this scenario does not support */
                if (!scenarios.list[i].support)
                        continue;
 
-               methods[i].member = scenarios.list[i].name;
-               methods[i].signature = "i";
-               methods[i].reply_signature = "i";
-               methods[i].func = dbus_pmqos_handler;
+               methods[count].member = scenarios.list[i].name;
+               methods[count].signature = "i";
+               methods[count].reply_signature = "i";
+               methods[count].func = dbus_pmqos_handler;
+               count++;
+
                _D("support [%s] scenario", scenarios.list[i].name);
        }
 
        *edbus_methods = methods;
-       ret = scenarios.num;
+       ret = count;
        pmqos_put_scenario(&scenarios);
 
        return ret;