fix(plugin/container): avoid overwriting host container info when loading pre-existing containers.

FedeDP · poiana · commit 89712a554e25 · 2025-04-17T10:42:49.000+02:00
Signed-off-by: Federico Di Pierro &lt;nierro92@gmail.com&gt;
diff --git a/plugins/container/src/caps/async/async.cpp b/plugins/container/src/caps/async/async.cpp
@@ -41,9 +41,9 @@ bool my_plugin::start_async_events(
     free((void *)enabled_engines);
 
     // Merge back pre-existing containers to our cache
-    m_containers = s_preexisting_containers;
-    for(const auto &c : m_containers)
+    for(const auto &c : s_preexisting_containers)
     {
+        m_containers.insert(c);
         m_logger.log(fmt::format("Added pre-existing container: {}", c.first),
                      falcosecurity::_internal::SS_PLUGIN_LOG_SEV_TRACE);
     }

Original file line number	Diff line number	Diff line change
`@@ -41,9 +41,9 @@ bool my_plugin::start_async_events(`
`41`	`41`	`free((void *)enabled_engines);`
`42`	`42`
`43`	`43`	`// Merge back pre-existing containers to our cache`
`44`		`- m_containers = s_preexisting_containers;`
`45`		`- for(const auto &c : m_containers)`
	`44`	`+ for(const auto &c : s_preexisting_containers)`
`46`	`45`	`{`
	`46`	`+ m_containers.insert(c);`
`47`	`47`	`m_logger.log(fmt::format("Added pre-existing container: {}", c.first),`
`48`	`48`	`falcosecurity::_internal::SS_PLUGIN_LOG_SEV_TRACE);`
`49`	`49`	`}`