pass: parser: Clean-up code to remove warning from static code analyzer

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

diff --git a/src/pass/pass-parser.c b/src/pass/pass-parser.c
index acaca3c8054d0e8dcfd90df3716a83f71e9b3999..38c6392c780542ca51d34d66103e66dedcc4fd54 100644 (file)
--- a/src/pass/pass-parser.c
+++ b/src/pass/pass-parser.c
@@ -532,13 +532,11 @@ static int parse_core(struct parse_result *result, void *user_data)
                        return gov_type;
                res->config_data.gov_type = gov_type;
        } else if (MATCH(result->name, "pass_num_levels")) {
-               unsigned int num_levels;
-
-               ret = sys_strtol(result->value);
-               if (ret < 0)
-                       return ret;
-               num_levels = ret;
+               int num_levels;
 
+               num_levels = sys_strtol(result->value);
+               if (num_levels < 0)
+                       return num_levels;
                if (num_levels > MAX_NUM) {
                        _E("cannot parse %s\n", result->name);
                        return -EINVAL;

diff --git a/src/pass/pass.h b/src/pass/pass.h
index 89a5494ff297443ecd13b3e976234b7abf90e128..2bf4a935c71ce1e64c57d94eb3c3485b04769a33 100644 (file)
--- a/src/pass/pass.h
+++ b/src/pass/pass.h
@@ -474,7 +474,7 @@ struct pass_resource_config_data {
        unsigned int default_max_level;
 
        /** The number of pass_levels */
-       unsigned int num_levels;
+       int num_levels;
        /** List of pass_levels */
        struct pass_level *levels;