From b10bfb2acd5dfd24306a62ddb53186a04a1207a9 Mon Sep 17 00:00:00 2001 From: Thomas Adam Date: Fri, 5 Apr 2024 21:39:11 +0100 Subject: [PATCH] Send_ConfigInfo: don't send monitor info When a module requests information about what it needs to know, it does so to fvwm by asking "Send_ConfigInfo". However, if this happens, and fvwm is already sending information to all modules (such as via a DesktopSize command), then an infinite loop can take place. Fixes #997 --- fvwm/modconf.c | 8 +++++++- 1 file changed, 7 insertions(+), 1 deletion(-) diff --git a/fvwm/modconf.c b/fvwm/modconf.c index 8cf0c94ca..77f47666f 100644 --- a/fvwm/modconf.c +++ b/fvwm/modconf.c @@ -397,7 +397,13 @@ void CMD_Send_ConfigInfo(F_CMD_ARGS) int match_len = 0; fmodule *mod = exc->m.module; - send_monitor_list(mod); + /* Don't send the monitor list when a module asks for its + * configuration... in this case, fvwm -> module could get in an + * infinite loop, continually telling that module the same + * information over and over. + * + * send_monitor_list(mod); + */ send_desktop_geometry(mod); /* send ImagePath and ColorLimit first */ send_image_path(mod);