parser, eval: use an ut_op flag to denote postfix access

The current code still abused the JSON value pointer to denote postfix
access for certain operations which led to a crash when freeing the
parser state due to an attempt to put a (void *)1 pointer.

Since we do have the ability to set flags on operations since the AST
rework, use this much cleaner approach and avoid the invalid pointer
hackery.

Signed-off-by: Jo-Philipp Wich <jo@mein.io>

1 files changed, 2 insertions, 2 deletions

diff --git a/eval.c b/eval.c
index 4bcc2b2..d2099dd 100644
--- a/eval.c
+++ b/eval.c
@@ -804,7 +804,7 @@ ut_execute_inc_dec(struct ut_state *state, uint32_t off)
 	ut_putval(ut_setval(scope, key, nval));
 
 	/* postfix inc/dec, return old val */
-	if (op->val)
+	if (op->is_postfix)
 		return val;
 
 	ut_putval(val);
@@ -1292,7 +1292,7 @@ ut_execute_op(struct ut_state *state, uint32_t off)
 
 	case T_LBRACK:
 		/* postfix access */
-		if (op->val) {
+		if (op->is_postfix) {
 			scope = ut_getref_required(state, off, &key);
 			state->ctx = scope;