plugins/fdp: bugfix error check to validate output format

The return value of validate_output_format functions is whether the
functions fails. So I fix to check the err variable.

Signed-off-by: Minsik Jeon <hmi.jeon@samsung.com>

diff --git a/plugins/fdp/fdp.c b/plugins/fdp/fdp.c
index 1efdd76d25d5f3d4090626af8c9ec88797c7b445..cac0b2bd728fd54ffdc14feab29c6821f17d92b1 100644 (file)
--- a/plugins/fdp/fdp.c
+++ b/plugins/fdp/fdp.c
@@ -57,7 +57,7 @@ static int fdp_configs(int argc, char **argv, struct command *cmd,
                return err;
 
        err = validate_output_format(cfg.output_format, &flags);
-       if (flags < 0)
+       if (err < 0)
                goto out;
 
        if (cfg.raw_binary)
@@ -138,7 +138,7 @@ static int fdp_usage(int argc, char **argv, struct command *cmd, struct plugin *
                return err;
 
        err = validate_output_format(cfg.output_format, &flags);
-       if (flags < 0)
+       if (err < 0)
                goto out;
 
        if (cfg.raw_binary)
@@ -209,7 +209,7 @@ static int fdp_stats(int argc, char **argv, struct command *cmd, struct plugin *
                return err;
 
        err = validate_output_format(cfg.output_format, &flags);
-       if (flags < 0)
+       if (err < 0)
                goto out;
 
        if (cfg.raw_binary)
@@ -270,7 +270,7 @@ static int fdp_events(int argc, char **argv, struct command *cmd, struct plugin
                return err;
 
        err = validate_output_format(cfg.output_format, &flags);
-       if (flags < 0)
+       if (err < 0)
                goto out;
 
        if (cfg.raw_binary)
@@ -329,7 +329,7 @@ static int fdp_status(int argc, char **argv, struct command *cmd, struct plugin
                return err;
 
        err = validate_output_format(cfg.output_format, &flags);
-       if (flags < 0)
+       if (err < 0)
                goto out;
 
        if (cfg.raw_binary)