diff options
author | Srinivas Kandagatla <srinivas.kandagatla@linaro.org> | 2019-05-22 17:24:43 +0100 |
---|---|---|
committer | Vinod Koul <vkoul@kernel.org> | 2019-05-27 10:34:12 +0530 |
commit | 03ecad90d3798be11b033248bbd4bbff4425a1c7 (patch) | |
tree | 96a116c3d313bc0712e21f41c9f7d55f8bd630d1 /drivers/soundwire | |
parent | a188339ca5a396acc588e5851ed7e19f66b0ebd9 (diff) | |
download | linux-03ecad90d3798be11b033248bbd4bbff4425a1c7.tar.gz linux-03ecad90d3798be11b033248bbd4bbff4425a1c7.tar.bz2 linux-03ecad90d3798be11b033248bbd4bbff4425a1c7.zip |
soundwire: stream: fix out of boundary access on port properties
Assigning local iterator to array element and using it again for
indexing would cross the array boundary.
Fix this by directly referring array element without using the local
variable.
Signed-off-by: Srinivas Kandagatla <srinivas.kandagatla@linaro.org>
Acked-by: Pierre-Louis Bossart <pierre-louis.bossart@linux.intel.com>
Signed-off-by: Vinod Koul <vkoul@kernel.org>
Diffstat (limited to 'drivers/soundwire')
-rw-r--r-- | drivers/soundwire/stream.c | 4 |
1 files changed, 1 insertions, 3 deletions
diff --git a/drivers/soundwire/stream.c b/drivers/soundwire/stream.c index d01060dbee96..544925ff0b40 100644 --- a/drivers/soundwire/stream.c +++ b/drivers/soundwire/stream.c @@ -1406,9 +1406,7 @@ struct sdw_dpn_prop *sdw_get_slave_dpn_prop(struct sdw_slave *slave, } for (i = 0; i < num_ports; i++) { - dpn_prop = &dpn_prop[i]; - - if (dpn_prop->num == port_num) + if (dpn_prop[i].num == port_num) return &dpn_prop[i]; } |