Subject: Re: Null pointer dereference in bonding driver, kernel 2.6.24

From: Jay Vosburgh <[email-masked]>
Date: Thu, 31 Jan 2008 14:55:29 -0800

Chuck Ebbert <[email-masked]> wrote:

>In bond_main.c:
>
>int bond_create(char *name, struct bond_params *params, struct bonding **newbond)
>{
>...
>        /* Check to see if the bond already exists. */
>        list_for_each_entry_safe(bond, nxt, &bond_dev_list, bond_list)
>                if (strnicmp(bond->dev->name, name, IFNAMSIZ) == 0) {
>                        printk(KERN_ERR DRV_NAME
>                               ": cannot add bond %s; it already exists\n",
>
>
>If 'name' is null we get a null dereference in strnicmp()
>
>The code was added in 2.6.24.

	This is already fixed in netdev-2.6#upstream.

	-J

---
	-Jay Vosburgh, IBM Linux Technology Center, [email-masked]