[PATCH] regmap: debugfs: potentially duplicate the name string of the config

2020-09-08 Thread Dafna Hirschfeld
In function regmap_debugfs_init the name of the regmap_config is assigned in a node of regmap_debugfs_early_list to be used later after regmap is initialized. It is unknown how and when the name is allocated and freed. Therefore the name should be copied to the node using 'kstrdup_const'. This fixe

Re: [PATCH] regmap: debugfs: potentially duplicate the name string of the config

2020-09-08 Thread Dafna Hirschfeld
Hi, thank you for the fast review, Am 08.09.20 um 17:35 schrieb Mark Brown: On Tue, Sep 08, 2020 at 05:28:59PM +0200, Dafna Hirschfeld wrote: In function regmap_debugfs_init the name of the regmap_config is assigned in a node of regmap_debugfs_early_list to be used later after regmap is initial

Re: [PATCH] regmap: debugfs: potentially duplicate the name string of the config

2020-09-08 Thread Mark Brown
On Tue, Sep 08, 2020 at 05:28:59PM +0200, Dafna Hirschfeld wrote: > In function regmap_debugfs_init the name of the regmap_config is assigned > in a node of regmap_debugfs_early_list to be used later after regmap > is initialized. It is unknown how and when the name is allocated and freed. > Theref

Re: [PATCH] regmap: debugfs: potentially duplicate the name string of the config

2020-09-08 Thread Mark Brown
On Tue, Sep 08, 2020 at 06:07:56PM +0200, Dafna Hirschfeld wrote: > Am 08.09.20 um 17:35 schrieb Mark Brown: > > It does not follow that the name should be copied, it equally follows > > (and is expected by the API given that the common case is that the name > > is a static string assigned at buil