[i2c] [PATCH] [OF] request_module() use format string

maximilian attems max at stro.at
Thu Jun 12 09:22:59 CEST 2008


hello jean,

On Thu, Jun 12, 2008 at 08:27:26AM +0200, Jean Delvare wrote:
> Hi Maximilian,
> 
> On Thu, 12 Jun 2008 07:02:06 +0200, maximilian attems wrote:
> > similar to 326f6a5c9c9e1a62aec37bdc0c3f8d53adabe77b,
> > maybe writable by root thus worth fixing.
> 
> I don't think root can affect the value of the string, but the file it
> is read from could be broken. So yes it seems worth fixing.

thanks for check.
 
> > 
> > Signed-off-by: maximilian attems <max at stro.at>
> > ---
> >  drivers/of/of_i2c.c |    2 +-
> >  1 files changed, 1 insertions(+), 1 deletions(-)
> > 
> > diff --git a/drivers/of/of_i2c.c b/drivers/of/of_i2c.c
> > index b2ccdcb..117de8b 100644
> > --- a/drivers/of/of_i2c.c
> > +++ b/drivers/of/of_i2c.c
> > @@ -100,7 +100,7 @@ void of_register_i2c_devices(struct i2c_adapter *adap,
> >  
> >  		info.addr = *addr;
> >  
> > -		request_module(info.type);
> > +		request_module("%s", info.type);
> >  
> >  		result = i2c_new_device(adap, &info);
> >  		if (result == NULL) {
> 
> Acked-by: Jean Delvare <khali at linux-fr.org>
> 
> I think you want to send this fix to Jochen Friedrich and Paul
> Mackerras (both Cc'd) as they are the ones working on this driver.

will do in a minute, with fixed desc.

-- 
maks



More information about the i2c mailing list