linux-kernel - Re: [lm-sensors] [PATCH v2] hwmon: (gpio-fan) Fix fan_ctrl_init error path

Open Source and information security mailing list archives

Message-ID: <20101109094132.GD32648@kw.sim.vm.gnt>
Date:	Tue, 9 Nov 2010 09:41:32 +0000
From:	Simon Guinot <simon@...uanux.org>
To:	Axel Lin <axel.lin@...il.com>
Cc:	linux-kernel <linux-kernel@...r.kernel.org>,
	Simon Guinot <sguinot@...ie.com>, lm-sensors@...sensors.org
Subject: Re: [lm-sensors] [PATCH v2] hwmon: (gpio-fan) Fix fan_ctrl_init
 error path

On Tue, Nov 09, 2010 at 04:41:48PM +0800, Axel Lin wrote:
> In current implementation, the sysfs entries is not removed before return -ENODEV.
> 
> Creating the sysfs attribute should be the last thing done by the function,
> after all the rest has been successful.
> Otherwise there is a small window during which user-space can access the attribute
> but the driver isn't ready to deal with the requests.
> 
> Fix it by moving sysfs_create_group to be the last thing done by the function.
> 
> Signed-off-by: Axel Lin <axel.lin@...il.com>

Thanks for the patch.

Acked-by: Simon Guinot <sguinot@...ie.com>

> ---
>  drivers/hwmon/gpio-fan.c |    8 ++++----
>  1 files changed, 4 insertions(+), 4 deletions(-)
> 
> diff --git a/drivers/hwmon/gpio-fan.c b/drivers/hwmon/gpio-fan.c
> index aa701a1..f141a1d 100644
> --- a/drivers/hwmon/gpio-fan.c
> +++ b/drivers/hwmon/gpio-fan.c
> @@ -376,10 +376,6 @@ static int fan_ctrl_init(struct gpio_fan_data *fan_data,
>  		}
>  	}
>  
> -	err = sysfs_create_group(&pdev->dev.kobj, &gpio_fan_ctrl_group);
> -	if (err)
> -		goto err_free_gpio;
> -
>  	fan_data->num_ctrl = num_ctrl;
>  	fan_data->ctrl = ctrl;
>  	fan_data->num_speed = pdata->num_speed;
> @@ -391,6 +387,10 @@ static int fan_ctrl_init(struct gpio_fan_data *fan_data,
>  		goto err_free_gpio;
>  	}
>  
> +	err = sysfs_create_group(&pdev->dev.kobj, &gpio_fan_ctrl_group);
> +	if (err)
> +		goto err_free_gpio;
> +
>  	return 0;
>  
>  err_free_gpio:
> -- 
> 1.7.2
> 
> 
> 
> 
> _______________________________________________
> lm-sensors mailing list
> lm-sensors@...sensors.org
> http://lists.lm-sensors.org/mailman/listinfo/lm-sensors

Download attachment "signature.asc" of type "application/pgp-signature" (199 bytes)

lists.openwall.net		lists / announce owl-users owl-dev john-users john-dev passwdqc-users yescrypt popa3d-users / oss-security kernel-hardening musl sabotage tlsify passwords / crypt-dev xvendor / Bugtraq Full-Disclosure linux-kernel linux-netdev linux-ext4 linux-hardening linux-cve-announce PHC
Open Source and information security mailing list archives