On Mon, Jan 28, 2019 at 4:39 PM Mattias Jacobsson <2pi@xxxxxx> wrote: > On 2019-01-27, Andy Shevchenko wrote: > > On Tue, Jan 22, 2019 at 10:04 PM Mattias Jacobsson <2pi@xxxxxx> wrote: > > > - struct wmi_driver *wmi_driver = > > > - container_of(driver, struct wmi_driver, driver); > > > > AFAIU this is just a pointer arithmetics, no need to move it. > > That is my understanding too, it seamed backwards to do the NULL check > afterwards, but we still have access to dev and driver. So why not... > > > > > > - struct wmi_block *wblock = dev_to_wblock(dev); > > > > > - const struct wmi_device_id *id = wmi_driver->id_table; > > > + const struct wmi_device_id *id; > > > + struct wmi_block *wblock; > > > + struct wmi_driver *wmi_driver; > > > + > > > > > + if (dev == NULL || driver == NULL) > > > + return 0; > > > > On which circumstances this may ever happen? > > Nothing in particular. If there is a bug in the caller of this function, > then that is when this will come into play. See my earlier mail to > Darren too. So, I suggest in these cases do not touch existing till the actual problem will be discovered and proved. I.o.w. touch only place which you have real problem, and describe this issue in commit message. -- With Best Regards, Andy Shevchenko
