diff options
author | Axel Lin <axel.lin@ingics.com> | 2014-04-02 10:21:04 -0400 |
---|---|---|
committer | Mark Brown <broonie@linaro.org> | 2014-04-03 06:13:38 -0400 |
commit | 0a6d38795a405c49ea0012f04173613382def58c (patch) | |
tree | 41cca48cd41bdf3c9034b8f59ff5f4079f9bdaf2 /drivers/spi/spi-fsl-espi.c | |
parent | 455c6fdbd219161bd09b1165f11699d6d73de11c (diff) |
spi: Always check complete callback before calling it
Since commit 1e25cd4729bd "spi: Do not require a completion", this checking is
required to prevent NULL pointer dereference.
Signed-off-by: Axel Lin <axel.lin@ingics.com>
Signed-off-by: Mark Brown <broonie@linaro.org>
Diffstat (limited to 'drivers/spi/spi-fsl-espi.c')
-rw-r--r-- | drivers/spi/spi-fsl-espi.c | 3 |
1 files changed, 2 insertions, 1 deletions
diff --git a/drivers/spi/spi-fsl-espi.c b/drivers/spi/spi-fsl-espi.c index 428dc7a6b62e..e8e3d34bab07 100644 --- a/drivers/spi/spi-fsl-espi.c +++ b/drivers/spi/spi-fsl-espi.c | |||
@@ -446,7 +446,8 @@ static void fsl_espi_do_one_msg(struct spi_message *m) | |||
446 | 446 | ||
447 | m->actual_length = espi_trans.actual_length; | 447 | m->actual_length = espi_trans.actual_length; |
448 | m->status = espi_trans.status; | 448 | m->status = espi_trans.status; |
449 | m->complete(m->context); | 449 | if (m->complete) |
450 | m->complete(m->context); | ||
450 | } | 451 | } |
451 | 452 | ||
452 | static int fsl_espi_setup(struct spi_device *spi) | 453 | static int fsl_espi_setup(struct spi_device *spi) |