Hi list, no need for an array here. therefor no need to worry about possible overflows. seq_printf() can handle this. avoid array in show_tty_range() re, walter signed-off-by: walter harms Signed-off-by: Domen Puncer --- kj-domen/fs/proc/proc_tty.c | 4 +--- 1 files changed, 1 insertion(+), 3 deletions(-) diff -puN fs/proc/proc_tty.c~remove_sprintf-fs_proc_proc_tty.c.bak fs/proc/proc_tty.c --- kj/fs/proc/proc_tty.c~remove_sprintf-fs_proc_proc_tty.c.bak 2004-12-25 01:36:07.000000000 +0100 +++ kj-domen/fs/proc/proc_tty.c 2004-12-25 01:36:07.000000000 +0100 @@ -32,10 +32,8 @@ static void show_tty_range(struct seq_fi seq_printf(m, "%-20s ", p->driver_name ? p->driver_name : "unknown"); seq_printf(m, "/dev/%-8s ", p->name); if (p->num > 1) { - char range[20]; - sprintf(range, "%d-%d", MINOR(from), + seq_printf(m, "%3d %d-%d ", MAJOR(from), MINOR(from), MINOR(from) + num - 1); - seq_printf(m, "%3d %7s ", MAJOR(from), range); } else { seq_printf(m, "%3d %7d ", MAJOR(from), MINOR(from)); } diff -L fs/proc/proc_tty.c.bak -puN /dev/null /dev/null _