Re: [PATCH] dmi: sanity check BIOS tables

From: Jiri Slaby
Date: Sun Jun 07 2009 - 04:57:42 EST


On 06/07/2009 01:01 AM, Alan Cox wrote:
> @@ -260,6 +276,11 @@ static void __init dmi_save_ipmi_device(const struct dmi_header *dm)
> struct dmi_device *dev;
> void * data;
>
> + if (((u8 *)dm) + dm->length > dmi_end) {

I think this should be >= too.

> + printk(KERN_ERR "dmi_save_ipmi_device: table overrun.\n");
> + return;
> + }
> +
--
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
Please read the FAQ at http://www.tux.org/lkml/