From: Cliff Wickman <cpw@xxxxxxx> Security hole in sn2_ptc_proc_write It is possible to overrun a buffer with a write to this /proc file. A little improvement suggested by Russ Anderson: > + if (count > 64) I prefer sizeof(optstr) instead of hardcoding 64 again. That way if optstr changes, it only has to be changed in one place. Diffed against 2.6.26-rc5 Signed-off-by: Cliff Wickman <cpw@xxxxxxx> --- arch/ia64/sn/kernel/sn2/sn2_smp.c | 2 ++ 1 file changed, 2 insertions(+) Index: linux-2.6/arch/ia64/sn/kernel/sn2/sn2_smp.c =================================================================== --- linux-2.6.orig/arch/ia64/sn/kernel/sn2/sn2_smp.c +++ linux-2.6/arch/ia64/sn/kernel/sn2/sn2_smp.c @@ -512,6 +512,8 @@ static ssize_t sn2_ptc_proc_write(struct int cpu; char optstr[64]; + if (count > sizeof(optstr)) + return -EINVAL; if (copy_from_user(optstr, user, count)) return -EFAULT; optstr[count - 1] = '\0'; -- To unsubscribe from this list: send the line "unsubscribe linux-ia64" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html
![]() |