Re: [PATCH 2/2] pinctrl: sprd: fix off by one bugs

[Baolin Wang <baolin.wang@xxxxxxxxxxxxxx>]

On 四,  9月 07, 2017 at 02:12:05下午 +0300, Dan Carpenter wrote:
> info->groups[] has info->ngroups elements so these comparisons should be
> >= instead of >.
> 
> Fixes: 41d32cfce1ae ("pinctrl: sprd: Add Spreadtrum pin control driver")
> Signed-off-by: Dan Carpenter <dan.carpenter@xxxxxxxxxx>

Thanks for fixing my mistakes, please add my tag if it is useful.
Reviewed-by: Baolin Wang <baolin.wang@xxxxxxxxxxxxxx>

> 
> diff --git a/drivers/pinctrl/sprd/pinctrl-sprd.c b/drivers/pinctrl/sprd/pinctrl-sprd.c
> index 673b77bbbc8c..939e1bbf6253 100644
> --- a/drivers/pinctrl/sprd/pinctrl-sprd.c
> +++ b/drivers/pinctrl/sprd/pinctrl-sprd.c
> @@ -400,7 +400,7 @@ static int sprd_pmx_set_mux(struct pinctrl_dev *pctldev,
>  	unsigned long reg;
>  	unsigned int val = 0;
>  
> -	if (group_selector > info->ngroups)
> +	if (group_selector >= info->ngroups)
>  		return -EINVAL;
>  
>  	switch (func_selector) {
> @@ -734,7 +734,7 @@ static int sprd_pinconf_group_get(struct pinctrl_dev *pctldev,
>  	struct sprd_pin_group *grp;
>  	unsigned int pin_id;
>  
> -	if (selector > info->ngroups)
> +	if (selector >= info->ngroups)
>  		return -EINVAL;
>  
>  	grp = &info->groups[selector];
> @@ -753,7 +753,7 @@ static int sprd_pinconf_group_set(struct pinctrl_dev *pctldev,
>  	struct sprd_pin_group *grp;
>  	int ret, i;
>  
> -	if (selector > info->ngroups)
> +	if (selector >= info->ngroups)
>  		return -EINVAL;
>  
>  	grp = &info->groups[selector];
> @@ -813,7 +813,7 @@ static void sprd_pinconf_group_dbg_show(struct pinctrl_dev *pctldev,
>  	const char *name;
>  	int i, ret;
>  
> -	if (selector > info->ngroups)
> +	if (selector >= info->ngroups)
>  		return;
>  
>  	grp = &info->groups[selector];
--
To unsubscribe from this list: send the line "unsubscribe linux-gpio" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at  http://vger.kernel.org/majordomo-info.html