Re: [PATCH] kobject: Access kobject name with caution if state is not initialized

2018-08-20 Thread Srikar Dronamraju
* Dmitry Torokhov [2018-08-20 12:33:50]: > On Mon, Aug 20, 2018 at 12:22 PM Greg Kroah-Hartman > wrote: > > > > On Mon, Aug 20, 2018 at 10:39:47PM +0530, Srikar Dronamraju wrote: > > Lots of stupid modules can do dumb things. Just don't do that. The > > kernel is not built to keep you from doi

Re: [PATCH] kobject: Access kobject name with caution if state is not initialized

2018-08-20 Thread Srikar Dronamraju
* Greg Kroah-Hartman [2018-08-20 21:22:47]: > On Mon, Aug 20, 2018 at 10:39:47PM +0530, Srikar Dronamraju wrote: > > A stupid module test like > > https://github.com/srikard/tests/blob/master/modules/kobject_test.c > > can panic the system. > > Lots of stupid modules can do dumb things. Just do

Re: [PATCH] kobject: Access kobject name with caution if state is not initialized

2018-08-20 Thread Dmitry Torokhov
On Mon, Aug 20, 2018 at 12:22 PM Greg Kroah-Hartman wrote: > > On Mon, Aug 20, 2018 at 10:39:47PM +0530, Srikar Dronamraju wrote: > > If kobject state is not initialized, then its not even certain that > > kobject'name is initialized. Hence when accessing the kobject's name > > tread carefully. >

Re: [PATCH] kobject: Access kobject name with caution if state is not initialized

2018-08-20 Thread Greg Kroah-Hartman
On Mon, Aug 20, 2018 at 10:39:47PM +0530, Srikar Dronamraju wrote: > If kobject state is not initialized, then its not even certain that > kobject'name is initialized. Hence when accessing the kobject's name > tread carefully. > > A stupid module test like > https://github.com/srikard/tests/blob/m

[PATCH] kobject: Access kobject name with caution if state is not initialized

2018-08-20 Thread Srikar Dronamraju
If kobject state is not initialized, then its not even certain that kobject'name is initialized. Hence when accessing the kobject's name tread carefully. A stupid module test like https://github.com/srikard/tests/blob/master/modules/kobject_test.c can panic the system. With patch: We will see the