caf0311c06
[ Upstream commit 90f3feb24172185f1832636264943e8b5e289245 ] priv->pdev pointer was set after being used in fsl_asoc_card_audmux_init(). Move this assignment at the start of the probe function, so sub-functions can correctly use pdev through priv. fsl_asoc_card_audmux_init() dereferences priv->pdev to get access to the dev struct, used with dev_err macros. As priv is zero-initialised, there would be a NULL pointer dereference. Note that if priv->dev is dereferenced before assignment but never used, for example if there is no error to be printed, the driver won't crash probably due to compiler optimisations. Fixes: 708b4351f08c ("ASoC: fsl: Add Freescale Generic ASoC Sound Card with ASRC support") Signed-off-by: Elinor Montmasson <elinor.montmasson@savoirfairelinux.com> Link: https://patch.msgid.link/20240620132511.4291-2-elinor.montmasson@savoirfairelinux.com Signed-off-by: Mark Brown <broonie@kernel.org> Signed-off-by: Sasha Levin <sashal@kernel.org> |
||
---|---|---|
.. | ||
efika-audio-fabric.c | ||
eukrea-tlv320.c | ||
fsl-asoc-card.c | ||
fsl_asrc.c | ||
fsl_asrc.h | ||
fsl_asrc_common.h | ||
fsl_asrc_dma.c | ||
fsl_audmix.c | ||
fsl_audmix.h | ||
fsl_dma.c | ||
fsl_dma.h | ||
fsl_easrc.c | ||
fsl_easrc.h | ||
fsl_esai.c | ||
fsl_esai.h | ||
fsl_micfil.c | ||
fsl_micfil.h | ||
fsl_mqs.c | ||
fsl_sai.c | ||
fsl_sai.h | ||
fsl_spdif.c | ||
fsl_spdif.h | ||
fsl_ssi.c | ||
fsl_ssi.h | ||
fsl_ssi_dbg.c | ||
fsl_utils.c | ||
fsl_utils.h | ||
imx-audmix.c | ||
imx-audmux.c | ||
imx-audmux.h | ||
imx-es8328.c | ||
imx-mc13783.c | ||
imx-pcm-dma.c | ||
imx-pcm-fiq.c | ||
imx-pcm.h | ||
imx-sgtl5000.c | ||
imx-spdif.c | ||
imx-ssi.c | ||
imx-ssi.h | ||
Kconfig | ||
Makefile | ||
mpc5200_dma.c | ||
mpc5200_dma.h | ||
mpc5200_psc_ac97.c | ||
mpc5200_psc_i2s.c | ||
mpc8610_hpcd.c | ||
mx27vis-aic32x4.c | ||
p1022_ds.c | ||
p1022_rdk.c | ||
pcm030-audio-fabric.c | ||
phycore-ac97.c | ||
wm1133-ev1.c |