diff options
| author | Charles Keepax <ckeepax@opensource.wolfsonmicro.com> | 2013-01-21 10:02:31 +0100 |
|---|---|---|
| committer | Mark Brown <broonie@opensource.wolfsonmicro.com> | 2013-01-22 08:40:26 +0100 |
| commit | a4cdbec758d2491a86ba94263b847768fa004fde (patch) | |
| tree | 3f7af336f1102dfcc032eeb1d419ca846583ee22 /sound/soc/codecs/wm_adsp.c | |
| parent | ASoC: wm_adsp: Use GFP_DMA for things that may be DMAed (diff) | |
| download | linux-a4cdbec758d2491a86ba94263b847768fa004fde.tar.xz linux-a4cdbec758d2491a86ba94263b847768fa004fde.zip | |
ASoC: wm_adsp: Release firmware on error
This patch correctly releases the firmware if the magic string in the
firmware header does not match.
Signed-off-by: Charles Keepax <ckeepax@opensource.wolfsonmicro.com>
Signed-off-by: Mark Brown <broonie@opensource.wolfsonmicro.com>
Diffstat (limited to 'sound/soc/codecs/wm_adsp.c')
| -rw-r--r-- | sound/soc/codecs/wm_adsp.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/sound/soc/codecs/wm_adsp.c b/sound/soc/codecs/wm_adsp.c index 4196f2d54967..b6b654837585 100644 --- a/sound/soc/codecs/wm_adsp.c +++ b/sound/soc/codecs/wm_adsp.c @@ -396,7 +396,7 @@ static int wm_adsp_load_coeff(struct wm_adsp *dsp) hdr = (void*)&firmware->data[0]; if (memcmp(hdr->magic, "WMDR", 4) != 0) { adsp_err(dsp, "%s: invalid magic\n", file); - return -EINVAL; + goto out_fw; } adsp_dbg(dsp, "%s: v%d.%d.%d\n", file, |