Move guard to a safer position

daniel-romano-DD · daniel-romano-DD · commit b0ba04631e1d · 2026-01-13T11:31:18.000+01:00
diff --git a/tracer/src/Datadog.Tracer.Native/iast/dataflow.cpp b/tracer/src/Datadog.Tracer.Native/iast/dataflow.cpp
@@ -384,10 +384,10 @@ HRESULT Dataflow::AppDomainShutdown(AppDomainID appDomainId)
 
 HRESULT Dataflow::ModuleLoaded(ModuleID moduleId, ModuleInfo** pModuleInfo)
 {
+    CSGUARD(_cs);
     // Retrieve all already modules at once to mimic initialization from creation behavior
     if (_preLoadedModuleIds.size() > 0)
     {
-        CSGUARD(_cs);
         for (auto const& id : _preLoadedModuleIds)
         {
             GetModuleInfo(id);

Original file line number	Diff line number	Diff line change
`@@ -384,10 +384,10 @@ HRESULT Dataflow::AppDomainShutdown(AppDomainID appDomainId)`
`384`	`384`
`385`	`385`	`HRESULT Dataflow::ModuleLoaded(ModuleID moduleId, ModuleInfo** pModuleInfo)`
`386`	`386`	`{`
	`387`	`+ CSGUARD(_cs);`
`387`	`388`	`// Retrieve all already modules at once to mimic initialization from creation behavior`
`388`	`389`	`if (_preLoadedModuleIds.size() > 0)`
`389`	`390`	`{`
`390`		`- CSGUARD(_cs);`
`391`	`391`	`for (auto const& id : _preLoadedModuleIds)`
`392`	`392`	`{`
`393`	`393`	`GetModuleInfo(id);`