MEDIUM: actions: Normalize the return code of the configuration parsers
This patch normalize the return code of the configuration parsers. Before
these changes, the tcp action parser returned -1 if fail and 0 for the
succes. The http action returned 0 if fail and 1 if succes.
The normalisation does:
- ACT_RET_PRS_OK for succes
- ACT_RET_PRS_ERR for failure
diff --git a/include/types/action.h b/include/types/action.h
index b8b4cd4..fd264cb 100644
--- a/include/types/action.h
+++ b/include/types/action.h
@@ -38,6 +38,11 @@
ACT_RET_ERR, /* processing error. */
};
+enum act_parse_ret {
+ ACT_RET_PRS_OK, /* continue processing. */
+ ACT_RET_PRS_ERR, /* abort processing. */
+};
+
enum act_name {
ACT_ACTION_CONT = 0,
ACT_ACTION_STOP,
@@ -134,8 +139,8 @@
struct action_kw {
const char *kw;
- int (*parse)(const char **args, int *cur_arg, struct proxy *px,
- struct act_rule *rule, char **err);
+ enum act_parse_ret (*parse)(const char **args, int *cur_arg, struct proxy *px,
+ struct act_rule *rule, char **err);
int match_pfx;
};