From d8118b445fa6ef44e839d9717b10dbf8d095feaa Mon Sep 17 00:00:00 2001 From: Arturo Borrero Date: Wed, 12 Mar 2014 16:53:22 +0100 Subject: src: fix expr_binary_error()-related compilation warnings MIME-Version: 1.0 Content-Type: text/plain; charset=UTF-8 Content-Transfer-Encoding: 8bit The commit e7b43ec0 [expr: make expr_binary_error() usable outside of evaluation] seem to change expr_binary_error() interface. Later, several compilation warning appears. The expr_binary_error() function and expr_error() macro both expect `struct list_head *', so I simply changed callers to send `ctx->msgs'. [...] src/evaluate.c: In function ‘byteorder_conversion’: src/evaluate.c:166:3: warning: passing argument 1 of ‘expr_binary_error’ from incompatible pointer type [enabled by default] In file included from src/evaluate.c:21:0: include/expression.h:275:12: note: expected ‘struct list_head *’ but argument is of type ‘struct eval_ctx *’ src/evaluate.c: In function ‘expr_evaluate_symbol’: src/evaluate.c:204:4: warning: passing argument 1 of ‘expr_binary_error’ from incompatible pointer type [enabled by default] In file included from src/evaluate.c:21:0: include/expression.h:275:12: note: expected ‘struct list_head *’ but argument is of type ‘struct eval_ctx *’ [...] Signed-off-by: Arturo Borrero Gonzalez Signed-off-by: Pablo Neira Ayuso --- src/evaluate.c | 82 +++++++++++++++++++++++++++++++--------------------------- 1 file changed, 44 insertions(+), 38 deletions(-) (limited to 'src') diff --git a/src/evaluate.c b/src/evaluate.c index 0c19c9fc..fff8fef1 100644 --- a/src/evaluate.c +++ b/src/evaluate.c @@ -163,7 +163,7 @@ static int byteorder_conversion(struct eval_ctx *ctx, struct expr **expr, if ((*expr)->byteorder == byteorder) return 0; if (expr_basetype(*expr)->type != TYPE_INTEGER) - return expr_error(ctx, *expr, + return expr_error(ctx->msgs, *expr, "Byteorder mismatch: expected %s, got %s", byteorder_names[byteorder], byteorder_names[(*expr)->byteorder]); @@ -201,7 +201,7 @@ static int expr_evaluate_symbol(struct eval_ctx *ctx, struct expr **expr) case SYMBOL_DEFINE: sym = symbol_lookup((*expr)->scope, (*expr)->identifier); if (sym == NULL) - return expr_error(ctx, *expr, + return expr_error(ctx->msgs, *expr, "undefined identifier '%s'", (*expr)->identifier); new = expr_clone(sym->expr); @@ -228,7 +228,7 @@ static int expr_evaluate_value(struct eval_ctx *ctx, struct expr **expr) case TYPE_INTEGER: mpz_init_bitmask(mask, ctx->ectx.len); if (mpz_cmp((*expr)->value, mask) > 0) { - expr_error(ctx, *expr, + expr_error(ctx->msgs, *expr, "Value %Zu exceeds valid range 0-%Zu", (*expr)->value, mask); mpz_clear(mask); @@ -240,7 +240,7 @@ static int expr_evaluate_value(struct eval_ctx *ctx, struct expr **expr) case TYPE_STRING: if (ctx->ectx.len > 0) { if ((*expr)->len > ctx->ectx.len) - return expr_error(ctx, *expr, + return expr_error(ctx->msgs, *expr, "String exceeds maximum length of %u", ctx->ectx.len / BITS_PER_BYTE); (*expr)->len = ctx->ectx.len; @@ -281,7 +281,7 @@ static int expr_evaluate_payload(struct eval_ctx *ctx, struct expr **expr) return -1; list_add_tail(&nstmt->list, &ctx->stmt->list); } else if (ctx->pctx.protocol[base].desc != payload->payload.desc) - return expr_error(ctx, payload, + return expr_error(ctx->msgs, payload, "conflicting protocols specified: %s vs. %s", ctx->pctx.protocol[base].desc->name, payload->payload.desc->name); @@ -315,17 +315,17 @@ static int expr_evaluate_prefix(struct eval_ctx *ctx, struct expr **expr) base = prefix->prefix; if (!expr_is_constant(base)) - return expr_error(ctx, prefix, + return expr_error(ctx->msgs, prefix, "Prefix expression is undefined for " "non-constant expressions"); if (expr_basetype(base)->type != TYPE_INTEGER) - return expr_error(ctx, prefix, + return expr_error(ctx->msgs, prefix, "Prefix expression is undefined for " "%s types", base->dtype->desc); if (prefix->prefix_len > base->len) - return expr_error(ctx, prefix, + return expr_error(ctx->msgs, prefix, "Prefix length %u is invalid for type " "of %u bits width", prefix->prefix_len, base->len); @@ -360,11 +360,11 @@ static int expr_evaluate_range_expr(struct eval_ctx *ctx, return -1; if (expr_basetype(*expr)->type != TYPE_INTEGER) - return expr_binary_error(ctx, *expr, range, + return expr_binary_error(ctx->msgs, *expr, range, "Range expression is undefined for " "%s types", (*expr)->dtype->desc); if (!expr_is_constant(*expr)) - return expr_binary_error(ctx, *expr, range, + return expr_binary_error(ctx->msgs, *expr, range, "Range is not constant"); return 0; } @@ -382,7 +382,8 @@ static int expr_evaluate_range(struct eval_ctx *ctx, struct expr **expr) right = range->right; if (mpz_cmp(left->value, right->value) >= 0) - return expr_error(ctx, range, "Range has zero or negative size"); + return expr_error(ctx->msgs, range, + "Range has zero or negative size"); range->dtype = left->dtype; range->flags |= EXPR_F_CONSTANT; @@ -487,7 +488,7 @@ static int expr_evaluate_shift(struct eval_ctx *ctx, struct expr **expr) struct expr *op = *expr, *left = op->left, *right = op->right; if (mpz_get_uint32(right->value) >= left->len) - return expr_binary_error(ctx, right, left, + return expr_binary_error(ctx->msgs, right, left, "%s shift of %u bits is undefined " "for type of %u bits width", op->op == OP_LSHIFT ? "Left" : "Right", @@ -548,24 +549,24 @@ static int expr_evaluate_binop(struct eval_ctx *ctx, struct expr **expr) right = op->right; if (expr_basetype(left)->type != TYPE_INTEGER) - return expr_binary_error(ctx, left, op, + return expr_binary_error(ctx->msgs, left, op, "Binary operation (%s) is undefined " "for %s types", sym, left->dtype->desc); if (expr_is_constant(left) && !expr_is_singleton(left)) - return expr_binary_error(ctx, left, op, + return expr_binary_error(ctx->msgs, left, op, "Binary operation (%s) is undefined " "for %s expressions", sym, left->ops->name); if (!expr_is_constant(right)) - return expr_binary_error(ctx, right, op, + return expr_binary_error(ctx->msgs, right, op, "Right hand side of binary operation " "(%s) must be constant", sym); if (!expr_is_singleton(right)) - return expr_binary_error(ctx, left, op, + return expr_binary_error(ctx->msgs, left, op, "Binary operation (%s) is undefined " "for %s expressions", sym, right->ops->name); @@ -610,7 +611,7 @@ static int expr_evaluate_concat(struct eval_ctx *ctx, struct expr **expr) n = 1; list_for_each_entry_safe(i, next, &(*expr)->expressions, list) { if (dtype && off == 0) - return expr_binary_error(ctx, i, *expr, + return expr_binary_error(ctx->msgs, i, *expr, "unexpected concat component, " "expecting %s", dtype->desc); @@ -628,7 +629,7 @@ static int expr_evaluate_concat(struct eval_ctx *ctx, struct expr **expr) (*expr)->dtype = concat_type_alloc(*expr); if (off > 0) - return expr_error(ctx, *expr, + return expr_error(ctx->msgs, *expr, "datatype mismatch, expected %s, " "expression has type %s", dtype->desc, (*expr)->dtype->desc); @@ -648,11 +649,11 @@ static int expr_evaluate_list(struct eval_ctx *ctx, struct expr **expr) if (list_member_evaluate(ctx, &i) < 0) return -1; if (i->ops->type != EXPR_VALUE) - return expr_error(ctx, i, + return expr_error(ctx->msgs, i, "List member must be a constant " "value"); if (i->dtype->basetype->type != TYPE_BITMASK) - return expr_error(ctx, i, + return expr_error(ctx->msgs, i, "Basetype of type %s is not bitmask", i->dtype->desc); mpz_ior(val, val, i->value); @@ -677,7 +678,8 @@ static int expr_evaluate_set(struct eval_ctx *ctx, struct expr **expr) return -1; if (!expr_is_constant(i)) - return expr_error(ctx, i, "Set member is not constant"); + return expr_error(ctx->msgs, i, + "Set member is not constant"); if (i->ops->type == EXPR_SET) { /* Merge recursive set definitions */ @@ -704,7 +706,7 @@ static int expr_evaluate_map(struct eval_ctx *ctx, struct expr **expr) if (expr_evaluate(ctx, &map->map) < 0) return -1; if (expr_is_constant(map->map)) - return expr_error(ctx, map->map, + return expr_error(ctx->msgs, map->map, "Map expression can not be constant"); mappings = map->mappings; @@ -730,7 +732,7 @@ static int expr_evaluate_map(struct eval_ctx *ctx, struct expr **expr) if (expr_evaluate(ctx, &map->mappings) < 0) return -1; if (map->mappings->ops->type != EXPR_SET_REF) - return expr_error(ctx, map->mappings, + return expr_error(ctx->msgs, map->mappings, "Expression is not a map"); break; default: @@ -755,7 +757,8 @@ static int expr_evaluate_mapping(struct eval_ctx *ctx, struct expr **expr) struct set *set = ctx->set; if (set == NULL) - return expr_error(ctx, mapping, "mapping outside of map context"); + return expr_error(ctx->msgs, mapping, + "mapping outside of map context"); if (!(set->flags & SET_F_MAP)) return set_error(ctx, set, "set is not a map"); @@ -763,16 +766,19 @@ static int expr_evaluate_mapping(struct eval_ctx *ctx, struct expr **expr) if (expr_evaluate(ctx, &mapping->left) < 0) return -1; if (!expr_is_constant(mapping->left)) - return expr_error(ctx, mapping->left, "Key must be a constant"); + return expr_error(ctx->msgs, mapping->left, + "Key must be a constant"); mapping->flags |= mapping->left->flags & EXPR_F_SINGLETON; expr_set_context(&ctx->ectx, set->datatype, set->datalen); if (expr_evaluate(ctx, &mapping->right) < 0) return -1; if (!expr_is_constant(mapping->right)) - return expr_error(ctx, mapping->right, "Value must be a constant"); + return expr_error(ctx->msgs, mapping->right, + "Value must be a constant"); if (!expr_is_singleton(mapping->right)) - return expr_error(ctx, mapping->right, "Value must be a singleton"); + return expr_error(ctx->msgs, mapping->right, + "Value must be a singleton"); mapping->flags |= EXPR_F_CONSTANT; return 0; @@ -789,7 +795,7 @@ static int binop_can_transfer(struct eval_ctx *ctx, switch (left->op) { case OP_LSHIFT: if (mpz_scan1(right->value, 0) < mpz_get_uint32(left->right->value)) - return expr_binary_error(ctx, right, left, + return expr_binary_error(ctx->msgs, right, left, "Comparison is always false"); return 1; case OP_XOR: @@ -896,12 +902,12 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) } if (!expr_is_constant(right)) - return expr_binary_error(ctx, right, rel, + return expr_binary_error(ctx->msgs, right, rel, "Right hand side of relational " "expression (%s) must be constant", expr_op_symbols[rel->op]); if (expr_is_constant(left)) - return expr_binary_error(ctx, left, right, + return expr_binary_error(ctx->msgs, left, right, "Relational expression (%s) has " "constant value", expr_op_symbols[rel->op]); @@ -913,7 +919,7 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) right = rel->right = implicit_set_declaration(ctx, left->dtype, left->len, right); else if (!datatype_equal(left->dtype, right->dtype)) - return expr_binary_error(ctx, right, left, + return expr_binary_error(ctx->msgs, right, left, "datatype mismatch, expected %s, " "set has type %s", left->dtype->desc, @@ -928,7 +934,7 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) break; case OP_EQ: if (!datatype_equal(left->dtype, right->dtype)) - return expr_binary_error(ctx, right, left, + return expr_binary_error(ctx->msgs, right, left, "datatype mismatch, expected %s, " "expression has type %s", left->dtype->desc, @@ -948,7 +954,7 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) case OP_NEQ: case OP_FLAGCMP: if (!datatype_equal(left->dtype, right->dtype)) - return expr_binary_error(ctx, right, left, + return expr_binary_error(ctx->msgs, right, left, "datatype mismatch, expected %s, " "expression has type %s", left->dtype->desc, @@ -974,7 +980,7 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) case OP_LTE: case OP_GTE: if (!datatype_equal(left->dtype, right->dtype)) - return expr_binary_error(ctx, right, left, + return expr_binary_error(ctx->msgs, right, left, "datatype mismatch, expected %s, " "expression has type %s", left->dtype->desc, @@ -982,7 +988,7 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) switch (left->ops->type) { case EXPR_CONCAT: - return expr_binary_error(ctx, left, rel, + return expr_binary_error(ctx->msgs, left, rel, "Relational expression (%s) is undefined " "for %s expressions", expr_op_symbols[rel->op], @@ -992,7 +998,7 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) } if (!expr_is_singleton(right)) - return expr_binary_error(ctx, right, rel, + return expr_binary_error(ctx->msgs, right, rel, "Relational expression (%s) is undefined " "for %s expressions", expr_op_symbols[rel->op], @@ -1005,7 +1011,7 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) break; case OP_RANGE: if (!datatype_equal(left->dtype, right->dtype)) - return expr_binary_error(ctx, right, left, + return expr_binary_error(ctx->msgs, right, left, "datatype mismatch, expected %s, " "expression has type %s", left->dtype->desc, @@ -1014,7 +1020,7 @@ static int expr_evaluate_relational(struct eval_ctx *ctx, struct expr **expr) range: switch (left->ops->type) { case EXPR_CONCAT: - return expr_binary_error(ctx, left, rel, + return expr_binary_error(ctx->msgs, left, rel, "Relational expression (%s) is undefined" "for %s expressions", expr_op_symbols[rel->op], -- cgit v1.2.3