diff options
author | Lars-Peter Clausen <lars@metafoo.de> | 2014-11-19 12:29:02 -0500 |
---|---|---|
committer | Mark Brown <broonie@kernel.org> | 2014-11-20 04:55:29 -0500 |
commit | 50c0f21b42dd4cd02b51f82274f66912d9a7fa32 (patch) | |
tree | 129a7c3d73dc0a214762454fe0db9b262e2efa92 | |
parent | f114040e3ea6e07372334ade75d1ee0775c355e1 (diff) |
ASoC: sigmadsp: Refuse to load firmware files with a non-supported version
Make sure to check the version field of the firmware header to make sure to
not accidentally try to parse a firmware file with a different layout.
Trying to do so can result in loading invalid firmware code to the device.
Signed-off-by: Lars-Peter Clausen <lars@metafoo.de>
Signed-off-by: Mark Brown <broonie@kernel.org>
Cc: stable@vger.kernel.org
-rw-r--r-- | sound/soc/codecs/sigmadsp.c | 7 |
1 files changed, 7 insertions, 0 deletions
diff --git a/sound/soc/codecs/sigmadsp.c b/sound/soc/codecs/sigmadsp.c index f2de7e049bc6..81a38dd9af1f 100644 --- a/sound/soc/codecs/sigmadsp.c +++ b/sound/soc/codecs/sigmadsp.c | |||
@@ -159,6 +159,13 @@ int _process_sigma_firmware(struct device *dev, | |||
159 | goto done; | 159 | goto done; |
160 | } | 160 | } |
161 | 161 | ||
162 | if (ssfw_head->version != 1) { | ||
163 | dev_err(dev, | ||
164 | "Failed to load firmware: Invalid version %d. Supported firmware versions: 1\n", | ||
165 | ssfw_head->version); | ||
166 | goto done; | ||
167 | } | ||
168 | |||
162 | crc = crc32(0, fw->data + sizeof(*ssfw_head), | 169 | crc = crc32(0, fw->data + sizeof(*ssfw_head), |
163 | fw->size - sizeof(*ssfw_head)); | 170 | fw->size - sizeof(*ssfw_head)); |
164 | pr_debug("%s: crc=%x\n", __func__, crc); | 171 | pr_debug("%s: crc=%x\n", __func__, crc); |