function_signature should be freed, otherwise 64 bytes of memory is l…

…eaked each time the bytecode is constructed
CopernicaMarketingSoftware · Mar 21, 2019 · 52ca90f · 52ca90f
1 parent dbc1fc8
commit 52ca90f
Show file tree

Hide file tree

Showing 2 changed files with 10 additions and 1 deletion.
diff --git a/src/bytecode.cpp b/src/bytecode.cpp
@@ -88,6 +88,15 @@ Bytecode::Bytecode(const Source& source) :
     jit_exception_set_handler(original_handler);
 }
 
+/**
+ *  Destructor
+ */
+Bytecode::~Bytecode()
+{
+    // if there is a signature, we need to free it
+    if (_function_signature) jit_type_free(_function_signature);
+}
+
 /**
  *  Method to initialize an error label
  *  @param  label

diff --git a/src/bytecode.h b/src/bytecode.h
@@ -284,7 +284,7 @@ class Bytecode : private Generator, public Executor
     /**
      *  Destructor
      */
-    virtual ~Bytecode() {};
+    virtual ~Bytecode();
 
     /**
      *  Execute the template given a certain handler