diff options
| author | Sachin Kamat <sachin.kamat@linaro.org> | 2014-02-25 09:48:28 (GMT) |
|---|---|---|
| committer | Chris Ball <chris@printf.net> | 2014-02-25 20:43:27 (GMT) |
| commit | 17c8bc85f27227e073ed8e59da39ff32f1bee873 (patch) | |
| tree | 44445e683c5021e806fa9c4b68fb07e26a2f4d41 /drivers/mmc | |
| parent | 550459eeb21b8f3553283d506fdcb92c9147c1eb (diff) | |
| download | linux-17c8bc85f27227e073ed8e59da39ff32f1bee873.tar.xz | |
mmc: dw_mmc: Fix NULL pointer dereference
If mrq->sbc is not NULL but data->stop happens to be NULL,
it will lead to NULL pointer dereferencing. Avoid this by
having a NULL check for data->stop.
Signed-off-by: Sachin Kamat <sachin.kamat@linaro.org>
Acked-by: Seungwon Jeon <tgih.jun@samsung.com>
Signed-off-by: Chris Ball <chris@printf.net>
Diffstat (limited to 'drivers/mmc')
| -rw-r--r-- | drivers/mmc/host/dw_mmc.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/mmc/host/dw_mmc.c b/drivers/mmc/host/dw_mmc.c index 55cd110..0c56faa 100644 --- a/drivers/mmc/host/dw_mmc.c +++ b/drivers/mmc/host/dw_mmc.c @@ -1345,7 +1345,7 @@ static void dw_mci_tasklet_func(unsigned long priv) if (!err) { if (!data->stop || mrq->sbc) { - if (mrq->sbc) + if (mrq->sbc && data->stop) data->stop->error = 0; dw_mci_request_end(host, mrq); goto unlock; |