OSDN Git Service

soundwire: stream: fix out of boundary access on port properties
authorSrinivas Kandagatla <srinivas.kandagatla@linaro.org>
Wed, 22 May 2019 16:24:43 +0000 (17:24 +0100)
committerVinod Koul <vkoul@kernel.org>
Mon, 27 May 2019 05:04:12 +0000 (10:34 +0530)
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>
drivers/soundwire/stream.c

index d01060d..544925f 100644 (file)
@@ -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];
        }