diff options
| author |   Zijun Hu <quic_zijuhu@quicinc.com> | 2024-12-09 21:25:00 +0800 |
|---|---|---|
| committer |   Rob Herring (Arm) <robh@kernel.org> | 2024-12-10 10:50:26 -0600 |
| commit | 3a973ce1b9180a5263924fbcebf8ad7572cca1fc (patch) | |
| tree | 74a977266a542241cd6217834cf0702c3540cd6d | |
| parent | of: Simplify API of_find_node_with_property() implementation (diff) | |
| download | wireguard-linux-3a973ce1b9180a5263924fbcebf8ad7572cca1fc.tar.xz wireguard-linux-3a973ce1b9180a5263924fbcebf8ad7572cca1fc.zip | |
of/irq: Correct element count for array @dummy_imask in API of_irq_parse_raw()
Array @dummy_imask only needs MAX_PHANDLE_ARGS elements, but it actually
has (MAX_PHANDLE_ARGS + 1) elements. One extra element doesn't hurt
anything except for some stack usage.
Fix by using (MAX_PHANDLE_ARGS - 1) as max element index in initializer.
Signed-off-by: Zijun Hu <quic_zijuhu@quicinc.com>
Link: https://lore.kernel.org/r/20241209-of_irq_fix-v1-2-782f1419c8a1@quicinc.com
Signed-off-by: Rob Herring (Arm) <robh@kernel.org>
Diffstat (limited to '')
| -rw-r--r-- | drivers/of/irq.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/of/irq.c b/drivers/of/irq.c index 67fc0ceaa5f5..3fbaf8f8610a 100644 --- a/drivers/of/irq.c +++ b/drivers/of/irq.c @@ -170,7 +170,7 @@ int of_irq_parse_raw(const __be32 *addr, struct of_phandle_args *out_irq) struct device_node *ipar, *tnode, *old = NULL; __be32 initial_match_array[MAX_PHANDLE_ARGS]; const __be32 *match_array = initial_match_array; - const __be32 *tmp, dummy_imask[] = { [0 ... MAX_PHANDLE_ARGS] = cpu_to_be32(~0) }; + const __be32 *tmp, dummy_imask[] = { [0 ... (MAX_PHANDLE_ARGS - 1)] = cpu_to_be32(~0) }; u32 intsize = 1, addrsize; int i, rc = -EINVAL; |