Fix: bytecode validator: reject specialized load field/context ref instructions

[lttng-ust.git] / src / lib / lttng-ust / lttng-bytecode.c

diff --git a/src/lib/lttng-ust/lttng-bytecode.c b/src/lib/lttng-ust/lttng-bytecode.c
index 099fc5ca5bab4d444f328fc2fa0ae9991cefe749..92258f168dd34beaeaf62b4fc7a7bdded27f582c 100644 (file)
--- a/src/lib/lttng-ust/lttng-bytecode.c
+++ b/src/lib/lttng-ust/lttng-bytecode.c
@@ -458,6 +458,11 @@ int link_bytecode(const struct lttng_ust_event_desc *event_desc,
        runtime->len = bytecode->bc.reloc_offset;
        /* copy original bytecode */
        memcpy(runtime->code, bytecode->bc.data, runtime->len);
+       /* Validate bytecode load instructions before relocs. */
+       ret = lttng_bytecode_validate_load(runtime);
+       if (ret) {
+               goto link_error;
+       }
        /*
         * apply relocs. Those are a uint16_t (offset in bytecode)
         * followed by a string (field name).