From c250bf69fda3120c4b47e41aa61cbad349b7df7a Mon Sep 17 00:00:00 2001 From: Sahitya Tummala Date: Fri, 30 May 2014 09:22:35 +0530 Subject: [PATCH] mmc: core: Fix NULL pointer dereference issue with mmc_blk_reset() If the mmc_hw_reset() fails, then host->card might be NULL in some cases. Hence, check for reset errors and report it to the caller so that the current request can be aborted and also check for host->card before accessing it so as to prevent NULL pointer dereference issue. Change-Id: Iba0f0be314474e607a40383bc0b28eef66a31d63 Signed-off-by: Sahitya Tummala [subhashj@codeaurora.org: fixed trivial merge conflicts] Signed-off-by: Subhash Jadavani --- drivers/mmc/card/block.c | 9 ++++++++- 1 file changed, 8 insertions(+), 1 deletion(-) diff --git a/drivers/mmc/card/block.c b/drivers/mmc/card/block.c index 27c3cb27d395..32b413b61cda 100644 --- a/drivers/mmc/card/block.c +++ b/drivers/mmc/card/block.c @@ -1403,8 +1403,15 @@ static int mmc_blk_reset(struct mmc_blk_data *md, struct mmc_host *host, md->reset_done |= type; err = mmc_hw_reset(host); + if (err && err != -EOPNOTSUPP) { + /* We failed to reset so we need to abort the request */ + pr_err("%s: %s: failed to reset %d\n", mmc_hostname(host), + __func__, err); + return -ENODEV; + } + /* Ensure we switch back to the correct partition */ - if (err != -EOPNOTSUPP) { + if (host->card) { struct mmc_blk_data *main_md = dev_get_drvdata(&host->card->dev); int part_err;