for dynamic, copy a changed pattern strictly under the mutex double-check

e5b54119 · Geoff Simmons · 02dfa2f9 · e5b54119
Commit e5b54119 authored Sep 13, 2013 by Geoff Simmons
Hide whitespace changes
Inline Side-by-side

Showing with 6 additions and 5 deletions

vmod_re.c src/vmod_re.c +6 -5

No files found.
--- a/src/vmod_re.c
+++ b/src/vmod_re.c
@@ -160,11 +160,12 @@ match(struct sess *sp, struct vmod_priv *priv_vcl, struct vmod_priv *priv_call,
 				    erroffset);
 			else
 				priv_call->free = VRT_re_fini;
-		}
-		if (dynamic) {
-			if (ov->pattern != NULL)
-				free(ov->pattern);
-			ov->pattern = strdup(pattern);
+			
+			if (dynamic) {
+				if (ov->pattern != NULL)
+					free(ov->pattern);
+				ov->pattern = strdup(pattern);
+			}
 		}
 		AZ(pthread_mutex_unlock(&re_mutex));
 	}