Messages in this thread Patch in this message |  | | Subject | Re: [PATCH] I2C fixes for 2.6.10-rc2 | | Date | Tue, 30 Nov 2004 16:13:38 -0800 | | From | Greg KH <> |
| |
ChangeSet 1.2223.2.2, 2004/11/24 14:24:32-08:00, johnpol@2ka.mipt.ru
[PATCH] w1: do not stop and oops if netlink socket was not allocated.
Do not panic if netlink socket was not created. This will allow only first device to broadcast it's slave updates. We need kernel connector here.
Signed-off-by: Evgeniy Polyakov <johnpol@2ka.mipt.ru> Signed-off-by: Greg Kroah-Hartman <greg@kroah.com>
drivers/w1/w1_int.c | 9 +++------ drivers/w1/w1_netlink.c | 3 +++ 2 files changed, 6 insertions(+), 6 deletions(-)
diff -Nru a/drivers/w1/w1_int.c b/drivers/w1/w1_int.c --- a/drivers/w1/w1_int.c 2004-11-30 16:01:21 -08:00 +++ b/drivers/w1/w1_int.c 2004-11-30 16:01:21 -08:00 @@ -89,11 +89,8 @@ dev->seq = 1; dev->nls = netlink_kernel_create(NETLINK_NFLOG, NULL); if (!dev->nls) { - printk(KERN_ERR "Failed to create new netlink socket(%u).\n", - NETLINK_NFLOG); - memset(dev, 0, sizeof(struct w1_master)); - kfree(dev); - dev = NULL; + printk(KERN_ERR "Failed to create new netlink socket(%u) for w1 master %s.\n", + NETLINK_NFLOG, dev->dev.bus_id); } err = device_register(&dev->dev); @@ -112,7 +109,7 @@ void w1_free_dev(struct w1_master *dev) { device_unregister(&dev->dev); - if (dev->nls->sk_socket) + if (dev->nls && dev->nls->sk_socket) sock_release(dev->nls->sk_socket); memset(dev, 0, sizeof(struct w1_master) + sizeof(struct w1_bus_master)); kfree(dev); diff -Nru a/drivers/w1/w1_netlink.c b/drivers/w1/w1_netlink.c --- a/drivers/w1/w1_netlink.c 2004-11-30 16:01:21 -08:00 +++ b/drivers/w1/w1_netlink.c 2004-11-30 16:01:21 -08:00 @@ -34,6 +34,9 @@ struct w1_netlink_msg *data; struct nlmsghdr *nlh; + if (!dev->nls) + return; + size = NLMSG_SPACE(sizeof(struct w1_netlink_msg)); skb = alloc_skb(size, GFP_ATOMIC); - To unsubscribe from this list: send the line "unsubscribe linux-kernel" in the body of a message to majordomo@vger.kernel.org More majordomo info at http://vger.kernel.org/majordomo-info.html Please read the FAQ at http://www.tux.org/lkml/
|  |