jit: Fix accidentally-harmless type confusion

In 2a0faed9d702, which added JIT compilation support for expressions, I
accidentally used sizeof(LLVMBasicBlockRef *) instead of
sizeof(LLVMBasicBlockRef) as part of computing the size of an allocation. That
turns out to have no real negative consequences due to LLVMBasicBlockRef being
a pointer itself (and thus having the same size). It still is wrong and
confusing, so fix it.

Reported by coverity.

Backpatch-through: 13

diff --git a/src/backend/jit/llvm/llvmjit_expr.c b/src/backend/jit/llvm/llvmjit_expr.c
index f567356ed199cd6c77a3a8d0050a8ff960ff5d94..2e286adfb5bfbc70d2cdd9d6846aaf6c55d1c55a 100644 (file)
--- a/src/backend/jit/llvm/llvmjit_expr.c
+++ b/src/backend/jit/llvm/llvmjit_expr.c
@@ -2065,7 +2065,7 @@ llvm_compile_expr(ExprState *state)
                                        v_nullsp = l_ptr_const(nulls, l_ptr(TypeStorageBool));
 
                                        /* create blocks for checking args */
-                                       b_checknulls = palloc(sizeof(LLVMBasicBlockRef *) * nargs);
+                                       b_checknulls = palloc(sizeof(LLVMBasicBlockRef) * nargs);
                                        for (int argno = 0; argno < nargs; argno++)
                                        {
                                                b_checknulls[argno] =