fix(listener): credconfig leak

emqx · Oct 7, 2023 · b35e2c4 · b35e2c4
1 parent d0f7273
commit b35e2c4
Showing 1 changed file with 6 additions and 0 deletions.
diff --git a/c_src/quicer_listener.c b/c_src/quicer_listener.c
@@ -279,6 +279,7 @@ listen2(ErlNifEnv *env, __unused_parm__ int argc, const ERL_NIF_TERM argv[])
     {
       // TLS opt error not file content error
       free(cacertfile);
+      free_certificate(&CredConfig);
       return ERROR_TUPLE_2(ATOM_CACERTFILE);
     }
 
@@ -288,6 +289,7 @@ listen2(ErlNifEnv *env, __unused_parm__ int argc, const ERL_NIF_TERM argv[])
   if (!l_ctx)
     {
       free(cacertfile);
+      free_certificate(&CredConfig);
       return ERROR_TUPLE_2(ATOM_ERROR_NOT_ENOUGH_MEMORY);
     }
 
@@ -306,6 +308,10 @@ listen2(ErlNifEnv *env, __unused_parm__ int argc, const ERL_NIF_TERM argv[])
           goto exit;
         }
     }
+  else
+    { // since we don't use cacertfile, free it
+      free(cacertfile);
+    }
 
   // Set owner for l_ctx
   if (!enif_self(env, &(l_ctx->listenerPid)))