Commit e2e82109 authored by Douglas Anderson's avatar Douglas Anderson Committed by Daniel Thompson
Browse files

kdb: Use the passed prompt in kdb_position_cursor() 



The function kdb_position_cursor() takes in a "prompt" parameter but
never uses it. This doesn't _really_ matter since all current callers
of the function pass the same value and it's a global variable, but
it's a bit ugly. Let's clean it up.

Found by code inspection. This patch is expected to functionally be a
no-op.

Fixes: 09b35989 ("kdb: Use format-strings rather than '\0' injection in kdb_read()")
Signed-off-by: default avatarDouglas Anderson <dianders@chromium.org>
Link: https://lore.kernel.org/r/20240528071144.1.I0feb49839c6b6f4f2c4bf34764f5e95de3f55a66@changeid


Signed-off-by: default avatarDaniel Thompson <daniel.thompson@linaro.org>
parent 70867efa

kernel/debug/kdb/kdb_io.c

+1 −1
Original line number Diff line number Diff line
@@ -206,7 +206,7 @@ char kdb_getchar(void) 
 */

static void kdb_position_cursor(char *prompt, char *buffer, char *cp)

{

	kdb_printf("\r%s", kdb_prompt_str);

	kdb_printf("\r%s", prompt);

	if (cp > buffer)

		kdb_printf("%.*s", (int)(cp - buffer), buffer);

}