Bug: Calls to the *printf wrapper functions, and in some cases to the *Printf functions are being made improperly.
Found by: Druid
Fixed by: Druid
When using any printf function you should be inputting three arguments at the very least. The data variable, the format string, and the data that fits into the format string.
For example, the below usages.
sprintf( buf, "Data here!" );
sprintf( buf, "%s", "Data here!" );
The reason for this is when those functions are used (ch_printf and pager_printf, etc) along with user input they can slip in some format tokens (%s %c %d) causing major havoc with the game.
If you want to test it, have someone send you a tell with a string of %s tokens tacked together then use the repeat command. This can be used in some places that will cause crashing of the game.
The best thing to do is to go in and fix in code that wraps into a printf function to use the proper number of arguments. The bug and log function calls are particularly bad about it. Unless you need to input data into a string, use the send_to_char and sent_to_pager function, instead of ch_printf and ch_printf color.