On 21 February 2015 at 05:15, NeilBrown <[email protected]> wrote:
>
> From: NeilBrown <[email protected]>
>
> The current error-path code (when gpiod_get_index() reports
> an error) can never free pwrseq->reset_gpios[0], but might
> try to tree pwrseq->reset_gpios[-1], which has unfortunate
> consequences.
>
> Signed-off-by: NeilBrown <[email protected]>
> Fixes: 934f1f48330ed695927a51fa068dc5d673f2da19
Applied, thanks!
Kind regards
Uffe
>
>
> diff --git a/drivers/mmc/core/pwrseq_simple.c
> b/drivers/mmc/core/pwrseq_simple.c
> index e9f1d8d84613..c53f14a7ce54 100644
> --- a/drivers/mmc/core/pwrseq_simple.c
> +++ b/drivers/mmc/core/pwrseq_simple.c
> @@ -124,7 +124,7 @@ int mmc_pwrseq_simple_alloc(struct mmc_host *host, struct
> device *dev)
> PTR_ERR(pwrseq->reset_gpios[i]) != -ENOSYS) {
> ret = PTR_ERR(pwrseq->reset_gpios[i]);
>
> - while (--i)
> + while (i--)
> gpiod_put(pwrseq->reset_gpios[i]);
>
> goto clk_put;
--
To unsubscribe from this list: send the line "unsubscribe linux-mmc" in
the body of a message to [email protected]
More majordomo info at http://vger.kernel.org/majordomo-info.html
