ipc-rules: Only allow subscriptions for valid signal names (#2656)

soreau · web-flow · commit 7091f03c95dd · 2025-06-14T19:03:51.000+02:00
diff --git a/plugins/ipc-rules/ipc-events.hpp b/plugins/ipc-rules/ipc-events.hpp
@@ -153,6 +153,9 @@ class ipc_rules_events_methods_t : public wf::per_output_tracker_mixin_t<>
                 if (signal_map.count(sub))
                 {
                     subscribed_to.insert(sub);
+                } else
+                {
+                    return wf::ipc::json_error("Event not found: \"" + sub.as_string() + "\"");
                 }
             }
         } else

Original file line number	Diff line number	Diff line change
`@@ -153,6 +153,9 @@ class ipc_rules_events_methods_t : public wf::per_output_tracker_mixin_t<>`
`153`	`153`	`if (signal_map.count(sub))`
`154`	`154`	`{`
`155`	`155`	`subscribed_to.insert(sub);`
	`156`	`+ } else`
	`157`	`+ {`
	`158`	`+ return wf::ipc::json_error("Event not found: \"" + sub.as_string() + "\"");`
`156`	`159`	`}`
`157`	`160`	`}`
`158`	`161`	`} else`