projects
/
apps
/
pfixtools.git
/ blobdiff
commit
grep
author
committer
pickaxe
?
search:
re
summary
|
shortlog
|
log
|
commit
|
commitdiff
|
tree
raw
|
inline
| side by side
Can disable syslog logging on tests.
[apps/pfixtools.git]
/
postlicyd
/
match.c
diff --git
a/postlicyd/match.c
b/postlicyd/match.c
index
ae4dbe4
..
1b02cc2
100644
(file)
--- a/
postlicyd/match.c
+++ b/
postlicyd/match.c
@@
-54,6
+54,15
@@
typedef struct match_condition_t {
} match_condition_t;
ARRAY(match_condition_t)
} match_condition_t;
ARRAY(match_condition_t)
+static const char *condition_names[] = {
+ "unknown",
+ "equals to",
+ "differs from",
+ "contains",
+ "is contained",
+ "is empty"
+};
+
#define CONDITION_INIT { PTK_UNKNOWN, false, MATCH_UNKNOWN, NULL, 0 }
typedef struct match_config_t {
#define CONDITION_INIT { PTK_UNKNOWN, false, MATCH_UNKNOWN, NULL, 0 }
typedef struct match_config_t {
@@
-86,8
+95,8
@@
static bool match_filter_constructor(filter_t *filter)
#define PARSE_CHECK(Expr, Str, ...) \
if (!(Expr)) { \
#define PARSE_CHECK(Expr, Str, ...) \
if (!(Expr)) { \
-
syslog(LOG_ERR, Str, ##__VA_ARGS__);
\
- match_config_delete(&config); \
+
err(Str, ##__VA_ARGS__);
\
+ match_config_delete(&config);
\
return false; \
}
return false; \
}
@@
-146,7
+155,7
@@
static bool match_filter_constructor(filter_t *filter)
PARSE_CHECK(condition.condition != MATCH_UNKNOWN,
"invalid operator");
if (condition.condition != MATCH_EMPTY) {
PARSE_CHECK(condition.condition != MATCH_UNKNOWN,
"invalid operator");
if (condition.condition != MATCH_EMPTY) {
- p =
m_strnextsp
(n + 1);
+ p =
skipspaces
(n + 1);
PARSE_CHECK(*p, "no value defined to check the condition");
condition.value_len = param->value_len - (p - param->value);
condition.value = p_dupstr(p, condition.value_len);
PARSE_CHECK(*p, "no value defined to check the condition");
condition.value_len = param->value_len - (p - param->value);
condition.value = p_dupstr(p, condition.value_len);
@@
-180,7
+189,9
@@
static inline bool match_condition(const match_condition_t *cond, const query_t
CASE(HELO_NAME, helo_name)
CASE(QUEUE_ID, queue_id)
CASE(SENDER, sender)
CASE(HELO_NAME, helo_name)
CASE(QUEUE_ID, queue_id)
CASE(SENDER, sender)
+ CASE(SENDER_DOMAIN, sender_domain)
CASE(RECIPIENT, recipient)
CASE(RECIPIENT, recipient)
+ CASE(RECIPIENT_DOMAIN, recipient_domain)
CASE(RECIPIENT_COUNT, recipient_count)
CASE(CLIENT_ADDRESS, client_address)
CASE(CLIENT_NAME, client_name)
CASE(RECIPIENT_COUNT, recipient_count)
CASE(CLIENT_ADDRESS, client_address)
CASE(CLIENT_NAME, client_name)
@@
-201,6
+212,10
@@
static inline bool match_condition(const match_condition_t *cond, const query_t
#undef CASE
default: return false;
}
#undef CASE
default: return false;
}
+ debug("running condition: \"%s\" %s %s\"%s\"",
+ field, condition_names[cond->condition],
+ cond->case_sensitive ? "" : "(alternative) ",
+ cond->value ? cond->value : "(none)");
switch (cond->condition) {
case MATCH_EQUAL:
case MATCH_DIFFER:
switch (cond->condition) {
case MATCH_EQUAL:
case MATCH_DIFFER:
@@
-239,7
+254,7
@@
static inline bool match_condition(const match_condition_t *cond, const query_t
break;
case MATCH_EMPTY:
break;
case MATCH_EMPTY:
- return !!(
!!(field == NULL || *field == '\0')) ^ (!!cond->case_sensitive
);
+ return !!(
(field == NULL || *field == '\0') ^ (!cond->case_sensitive)
);
default:
assert(false && "invalid condition type");
default:
assert(false && "invalid condition type");
@@
-253,14
+268,18
@@
static filter_result_t match_filter(const filter_t *filter, const query_t *query
foreach (const match_condition_t *condition, config->conditions) {
bool r = match_condition(condition, query);
if (!r && config->match_all) {
foreach (const match_condition_t *condition, config->conditions) {
bool r = match_condition(condition, query);
if (!r && config->match_all) {
+ debug("condition failed, match_all failed");
return HTK_FAIL;
} else if (r && !(config->match_all)) {
return HTK_FAIL;
} else if (r && !(config->match_all)) {
+ debug("condition succeed, not-match_all succeed");
return HTK_MATCH;
}
}}
if (config->match_all) {
return HTK_MATCH;
}
}}
if (config->match_all) {
+ debug("all conditions matched, match_all succeed");
return HTK_MATCH;
} else {
return HTK_MATCH;
} else {
+ debug("no condition matched, not-match_all failed");
return HTK_FAIL;
}
}
return HTK_FAIL;
}
}