--- fvwm/fvwm/builtins.c +++ fvwm/fvwm/builtins.c @@ -2665,7 +2665,7 @@ void CMD_Echo(F_CMD_ARGS) action[len-1]='\0'; } } - fvwm_msg(ECHO,"Echo",action); + fvwm_msg(ECHO,"Echo","%s",action); return; } --- fvwm/fvwm/move_resize.c +++ fvwm/fvwm/move_resize.c @@ -2964,7 +2964,7 @@ void CMD_SnapAttraction(F_CMD_ARGS) OLD, "CMD_SnapAttraction", "The command SnapAttraction is obsolete. Please use the" " following command instead:"); - fvwm_msg(OLD, "", cmd); + fvwm_msg(OLD, "", "%s", cmd); execute_function( cond_rc, exc, cmd, FUNC_DONT_REPEAT | FUNC_DONT_EXPAND_COMMAND); @@ -2986,7 +2986,7 @@ void CMD_SnapGrid(F_CMD_ARGS) OLD, "CMD_SnapGrid", "The command SnapGrid is obsolete. Please use the following" " command instead:"); - fvwm_msg(OLD, "", cmd); + fvwm_msg(OLD, "", "%s", cmd); execute_function( cond_rc, exc, cmd, FUNC_DONT_REPEAT | FUNC_DONT_EXPAND_COMMAND); --- fvwm/fvwm/virtual.c +++ fvwm/fvwm/virtual.c @@ -2023,9 +2023,9 @@ void CMD_EdgeResistance(F_CMD_ARGS) "The command EdgeResistance with three arguments is" " obsolete. Please use the following commands" " instead:"); - fvwm_msg(OLD, "", cmd); - fvwm_msg(OLD, "", stylecmd); - fvwm_msg(OLD, "", stylecmd2); + fvwm_msg(OLD, "", "%s", cmd); + fvwm_msg(OLD, "", "%s", stylecmd); + fvwm_msg(OLD, "", "%s", stylecmd2); execute_function( cond_rc, exc, cmd, FUNC_DONT_REPEAT | FUNC_DONT_EXPAND_COMMAND); --- fvwm/libs/FRenderInit.c +++ fvwm/libs/FRenderInit.c @@ -129,7 +129,7 @@ Bool FRenderGetErrorText(int code, char *msg) code <= FRenderErrorBase + (sizeof(error_names) / sizeof(char *)) -1) { - sprintf(msg, error_names[code - FRenderErrorBase]); + sprintf(msg, "%s", error_names[code - FRenderErrorBase]); return 1; } } --- fvwm/modules/FvwmIconMan/functions.c +++ fvwm/modules/FvwmIconMan/functions.c @@ -523,20 +523,13 @@ int builtin_ret(int numargs, BuiltinArg *args) int builtin_print(int numargs, BuiltinArg *args) { - char *s, buf[256]; - - ConsoleDebug(FUNCTIONS, "print: %s\n", args[0].value.string_value); + char *s; s = args[0].value.string_value; - if (strlen(s) > 250) - { - ConsoleMessage("String too long\n"); - } - else - { - sprintf(buf, "%s\n", s); - ConsoleMessage(buf); - } + + ConsoleDebug(FUNCTIONS, "print: %s\n", s); + + ConsoleMessage("%s\n", s); return 0; }