diff options
| author | Zijun Hu <quic_zijuhu@quicinc.com> | 2025-01-09 21:26:57 +0800 |
|---|---|---|
| committer | Greg Kroah-Hartman <gregkh@linuxfoundation.org> | 2025-02-08 10:01:58 +0100 |
| commit | f391a037b61d2058594d3c697d36a2ff1347beec (patch) | |
| tree | 501dcd92394aea3e4180556b9c05f0fffb99d072 /drivers/of | |
| parent | 58a58e31eca6ec2d22b130f23b6dfff1dc09188f (diff) | |
| download | linux-f391a037b61d2058594d3c697d36a2ff1347beec.tar.gz linux-f391a037b61d2058594d3c697d36a2ff1347beec.tar.bz2 linux-f391a037b61d2058594d3c697d36a2ff1347beec.zip | |
of: property: Avoiding using uninitialized variable @imaplen in parse_interrupt_map()
[ Upstream commit f73780e772c06901e99b2ad114b7f0f3fbe73ad4 ]
parse_interrupt_map() will use uninitialized variable @imaplen if fails
to get property 'interrupt-map'.
Fix by using the variable after successfully getting the property.
Fixes: e7985f43609c ("of: property: Fix fw_devlink handling of interrupt-map")
Signed-off-by: Zijun Hu <quic_zijuhu@quicinc.com>
Reviewed-by: Krzysztof Kozlowski <krzk@kernel.org>
Link: https://lore.kernel.org/r/20250109-of_core_fix-v4-6-db8a72415b8c@quicinc.com
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Signed-off-by: Sasha Levin <sashal@kernel.org>
Diffstat (limited to 'drivers/of')
| -rw-r--r-- | drivers/of/property.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/of/property.c b/drivers/of/property.c index cfc8aea002e4..b0633f3589de 100644 --- a/drivers/of/property.c +++ b/drivers/of/property.c @@ -1390,9 +1390,9 @@ static struct device_node *parse_interrupt_map(struct device_node *np, addrcells = of_bus_n_addr_cells(np); imap = of_get_property(np, "interrupt-map", &imaplen); - imaplen /= sizeof(*imap); if (!imap) return NULL; + imaplen /= sizeof(*imap); imap_end = imap + imaplen; |