Fix: report bytecode_push failure when pushing symbol

[lttng-tools.git] / src / lib / lttng-ctl / filter / filter-visitor-generate-bytecode.c

diff --git a/src/lib/lttng-ctl/filter/filter-visitor-generate-bytecode.c b/src/lib/lttng-ctl/filter/filter-visitor-generate-bytecode.c
index 1ab98138e61e16f4120f3be7ae508b518dac65cc..0e778562f99ab606f7f988dd4a16673a9c666e81 100644 (file)
--- a/src/lib/lttng-ctl/filter/filter-visitor-generate-bytecode.c
+++ b/src/lib/lttng-ctl/filter/filter-visitor-generate-bytecode.c
@@ -295,6 +295,9 @@ int visit_node_load_expression_legacy(struct filter_parser_ctx *ctx,
        }
        ret = bytecode_push(&ctx->bytecode_reloc, symbol,
                                1, strlen(symbol) + 1);
+       if (ret) {
+               goto end;
+       }
        ret = 1;        /* legacy */
 end:
        free(insn);
@@ -319,6 +322,10 @@ int visit_node_load_expression(struct filter_parser_ctx *ctx,
                return -EINVAL;
        }
 
+       /*
+        * TODO: if we remove legacy load for application contexts, we
+        * need to update session bytecode parser as well.
+        */
        ret = visit_node_load_expression_legacy(ctx, exp, op);
        if (ret < 0) {
                return ret;