On Tue, Oct 09, 2007 at 06:44:07PM +0200, Roel Kluin wrote: > >> + if (IS_ERR(data->hwmon_dev)) { > >> + printk(KERN_ERR DRVNAME ": Error, unable to register hwmon " > >> + "class device for interface %d\n", > >> + data->interface); > >> + kfree(data); > >> + return; > > don't you want to goto out_user here instead? Err, yes, thank you for catching that. --D