create pcre2 compile context once instead of for each compile

Author: Steve Ayre

modules/dialplan/dialplan.c

@@ -86,6 +86,12 @@ static str database_url = {NULL, 0};
 
 void *dp_srg = NULL;
 
+pcre2_general_context *dp_gcontext = NULL;
+pcre2_compile_context *dp_ccontext = NULL;
+
+void * wrap_shm_malloc(PCRE2_SIZE size, void * memory_data);
+void wrap_shm_free(void * p, void * memory_data);
+
 
 static const param_export_t mod_params[]={
 	{ "partition",		STR_PARAM|USE_FUNC_PARAM,
@@ -398,6 +404,18 @@ static int mod_init(void)
 		return -1;
 	}
 
+	dp_gcontext = pcre2_general_context_create(wrap_shm_malloc, wrap_shm_free, NULL);
+	if (!dp_gcontext) {
+		LM_ERR("Unable to create pcre general context\n");
+		return -1;
+	}
+
+	dp_ccontext = pcre2_compile_context_create(dp_gcontext);
+	if (!dp_ccontext) {
+		LM_ERR("Unable to create pcre compile context\n");
+		return -1;
+	}
+
 	return 0;
 }
 
@@ -452,6 +470,18 @@ static void mod_destroy(void)
 	}
 
 	destroy_data();
+
+	if (dp_ccontext)
+	{
+		pcre2_compile_context_free(dp_ccontext);
+		dp_ccontext = NULL;
+	}
+
+	if (dp_gcontext)
+	{
+		pcre2_general_context_free(dp_gcontext);
+		dp_gcontext = NULL;
+	}
 }
 
 
@@ -883,28 +913,11 @@ void wrap_shm_free(void * p, void * memory_data)
 
 pcre2_code * wrap_pcre_compile(char *  pattern, int flags)
 {
-	pcre2_general_context *gcontext;
-	pcre2_compile_context *ccontext;
 	pcre2_code * ret ;
 	int error;
 	PCRE2_SIZE erroffset;
 	int pcre_flags = 0;
 
-	// TODO generate once per worker
-	gcontext = pcre2_general_context_create(wrap_shm_malloc, wrap_shm_free, NULL);
-	if (!gcontext) {
-		LM_ERR("Unable to create pcre general context\n");
-		return NULL;
-	}
-
-	// TODO generate once per worker
-	ccontext = pcre2_compile_context_create(gcontext);
-	if (!ccontext) {
-		LM_ERR("Unable to create pcre compile context\n");
-		pcre2_general_context_free(gcontext);
-		return NULL;
-	}
-
 	if (flags & DP_CASE_INSENSITIVE)
 		pcre_flags |= PCRE2_CASELESS;
 
@@ -914,16 +927,14 @@ pcre2_code * wrap_pcre_compile(char *  pattern, int flags)
 			pcre_flags,			/* default options */
 			&error,				/* for error message */
 			&erroffset,			/* for error offset */
-			ccontext);                      /* compile context, to allocate memory in shm */
-
-	pcre2_compile_context_free(ccontext);
-	pcre2_general_context_free(gcontext);
+			dp_ccontext);                      /* compile context, to allocate memory in shm */
 
 	return ret;
 }
 
 void wrap_pcre_free( pcre2_code* re)
 {
-	// TODO pcre2_code_free
+	// *not* pcre2_code_free
+	// shm_free is used because pcre2_general_context_create overrides malloc with wrap_shm_malloc
 	shm_free(re);
 }