fix(tmplexec): handle rare case where lastMatcherEvent address change…

… after lock
projectdiscovery · Dec 5, 2024 · 4c8a166 · 4c8a166
1 parent 2549592
commit 4c8a166
Showing 1 changed file with 5 additions and 4 deletions.
diff --git a/pkg/tmplexec/exec.go b/pkg/tmplexec/exec.go
@@ -214,10 +214,11 @@ func (e *TemplateExecuter) Execute(ctx *scan.ScanContext) (bool, error) {
 	ctx.LogError(errx)
 
 	if lastMatcherEvent != nil {
-		lastMatcherEvent.Lock()
-		lastMatcherEvent.InternalEvent["error"] = getErrorCause(ctx.GenerateErrorMessage())
-		lastMatcherEvent.Unlock()
-		writeFailureCallback(lastMatcherEvent, e.options.Options.MatcherStatus)
+		event := lastMatcherEvent
+		event.Lock()
+		event.InternalEvent["error"] = getErrorCause(ctx.GenerateErrorMessage())
+		event.Unlock()
+		writeFailureCallback(event, e.options.Options.MatcherStatus)
 	}
 
 	//TODO: this is a hacky way to handle the case where the callback is not called and matcher-status is true.