X-Git-Url: http://git.lttng.org/?a=blobdiff_plain;f=lttng-filter-validator.c;h=446fc41bbda9b657a615871ed779b59c2b527227;hb=refs%2Fheads%2Fstable-2.7;hp=71e9a7409167d87684bb11049040f7f0a459d14c;hpb=07dfc1d0e4b093ad02682499a702dc11e54e8302;p=lttng-modules.git diff --git a/lttng-filter-validator.c b/lttng-filter-validator.c index 71e9a740..446fc41b 100644 --- a/lttng-filter-validator.c +++ b/lttng-filter-validator.c @@ -20,10 +20,11 @@ * Foundation, Inc., 51 Franklin Street, Fifth Floor, Boston, MA 02110-1301 USA */ -#include +#include #include #include +#include "wrapper/list.h" #include "lttng-filter.h" #define MERGE_POINT_TABLE_BITS 7 @@ -87,7 +88,7 @@ int merge_point_add_check(struct mp_table *mp_table, unsigned long target_pc, memcpy(&mp_node->stack, stack, sizeof(mp_node->stack)); head = &mp_table->mp_head[hash & (MERGE_POINT_TABLE_SIZE - 1)]; - hlist_for_each_entry(lookup_node, head, node) { + lttng_hlist_for_each_entry(lookup_node, head, node) { if (lttng_hash_match(lookup_node, target_pc)) { found = 1; break; @@ -103,8 +104,9 @@ int merge_point_add_check(struct mp_table *mp_table, unsigned long target_pc, target_pc); return -EINVAL; } + } else { + hlist_add_head(&mp_node->node, head); } - hlist_add_head(&mp_node->node, head); return 0; } @@ -299,6 +301,8 @@ int bytecode_validate_overflow(struct bytecode_runtime *bytecode, } case FILTER_OP_LOAD_FIELD_REF_STRING: case FILTER_OP_LOAD_FIELD_REF_SEQUENCE: + case FILTER_OP_LOAD_FIELD_REF_USER_STRING: + case FILTER_OP_LOAD_FIELD_REF_USER_SEQUENCE: case FILTER_OP_LOAD_FIELD_REF_S64: case FILTER_OP_GET_CONTEXT_REF_STRING: case FILTER_OP_GET_CONTEXT_REF_S64: @@ -367,7 +371,7 @@ unsigned long delete_all_nodes(struct mp_table *mp_table) struct hlist_head *head; head = &mp_table->mp_head[i]; - hlist_for_each_entry_safe(mp_node, tmp, head, node) { + lttng_hlist_for_each_entry_safe(mp_node, tmp, head, node) { kfree(mp_node); nr_nodes++; } @@ -612,6 +616,8 @@ int validate_instruction_context(struct bytecode_runtime *bytecode, } case FILTER_OP_LOAD_FIELD_REF_STRING: case FILTER_OP_LOAD_FIELD_REF_SEQUENCE: + case FILTER_OP_LOAD_FIELD_REF_USER_STRING: + case FILTER_OP_LOAD_FIELD_REF_USER_SEQUENCE: { struct load_op *insn = (struct load_op *) pc; struct field_ref *ref = (struct field_ref *) insn->data; @@ -735,7 +741,7 @@ int validate_instruction_all_contexts(struct bytecode_runtime *bytecode, /* Validate merge points */ hash = jhash_1word(target_pc, 0); head = &mp_table->mp_head[hash & (MERGE_POINT_TABLE_SIZE - 1)]; - hlist_for_each_entry(mp_node, head, node) { + lttng_hlist_for_each_entry(mp_node, head, node) { if (lttng_hash_match(mp_node, target_pc)) { found = 1; break; @@ -932,6 +938,8 @@ int exec_insn(struct bytecode_runtime *bytecode, case FILTER_OP_LOAD_FIELD_REF_STRING: case FILTER_OP_LOAD_FIELD_REF_SEQUENCE: case FILTER_OP_GET_CONTEXT_REF_STRING: + case FILTER_OP_LOAD_FIELD_REF_USER_STRING: + case FILTER_OP_LOAD_FIELD_REF_USER_SEQUENCE: { if (vstack_push(stack)) { ret = -EINVAL;