On Thursday 19 October 2017 12:50:08 Mario Limonciello wrote: > + priv->interface_version = buffer[2]; > + priv->size = buffer[3]; > + ret = 0; > + dev_set_drvdata(&wdev->dev, priv); > + list_add_tail(&priv->list, &wmi_list); Still missing lock when changing wmi_list. > + > + dev_dbg(&wdev->dev, "Detected Dell WMI interface version %lu and buffer size %lu\n", > + (unsigned long) priv->interface_version, > + (unsigned long) priv->size); > + > +out: > + kfree(obj); > + return ret; > +} -- Pali Rohár pali.rohar@xxxxxxxxx
