chore(plugins/container): avoid useless req.set_value of empty string.

FedeDP · poiana · commit 298b671a66d7 · 2025-05-13T14:29:29.000+02:00
Signed-off-by: Federico Di Pierro &lt;nierro92@gmail.com&gt;
diff --git a/plugins/container/src/caps/extract/extract.cpp b/plugins/container/src/caps/extract/extract.cpp
@@ -612,11 +612,6 @@ bool my_plugin::extract(const falcosecurity::extract_fields_input &in)
             {
                 // Can't return anything but those fields without containers
                 // metadata.
-
-                // Set an empty extracted value to let `return true` below work
-                // See
-                // https://github.com/falcosecurity/libs/blob/0d94d2bc55d4ccde870bc0c076423a1e13814079/userspace/libsinsp/plugin_filtercheck.cpp#L172
-                req.set_value("");
                 return true; // go on to extract other fields if needed, perhaps
                              // they'll be one of the above
             }

Original file line number	Diff line number	Diff line change
`@@ -612,11 +612,6 @@ bool my_plugin::extract(const falcosecurity::extract_fields_input &in)`
`612`	`612`	`{`
`613`	`613`	`// Can't return anything but those fields without containers`
`614`	`614`	`// metadata.`
`615`		`-`
`616`		- // Set an empty extracted value to let `return true` below work
`617`		`- // See`
`618`		`- // https://github.com/falcosecurity/libs/blob/0d94d2bc55d4ccde870bc0c076423a1e13814079/userspace/libsinsp/plugin_filtercheck.cpp#L172`
`619`		`- req.set_value("");`
`620`	`615`	`return true; // go on to extract other fields if needed, perhaps`
`621`	`616`	`// they'll be one of the above`
`622`	`617`	`}`