![]() | |||||||||||||
Messages in this thread |
On Tue, Sep 26, 2006 at 01:23:34PM -0400, Dmitry Torokhov wrote: > On 9/26/06, Greg KH <greg@kroah.com> wrote: > >From: Andrew Morton <akpm@osdl.org> > > > >Add lots of return-value checking. > > > >+ if (error) > >+ goto out; > >+ error = sysfs_create_link(&bus->devices.kobj, > >+ &dev->kobj, dev->bus_id); > >+ if (error) > >+ goto out; > >+ error = sysfs_create_link(&dev->kobj, > >+ &dev->bus->subsys.kset.kobj, "subsystem"); > >+ if (error) > >+ goto out; > >+ error = sysfs_create_link(&dev->kobj, > >+ &dev->bus->subsys.kset.kobj, "bus"); > > } > >+out: > > return error; > > What about removing the links that were created if one of these calls fails? Yes, that would be good, I think I have a patch in my queue that handles that properly... thanks, greg k-h - 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/ | ||||||||||||
| Last update: 2006-09-27 06:43 [from the cache] ©2003-2008 | |||||||||||||