Message ID | 20220416181530.2311155-1-jorge@foundries.io |
---|---|
State | Accepted |
Commit | 9b31e109b2dab0e696659f77c057e620bfb62b5b |
Delegated to: | Michal Simek |
Headers | show |
Series | [v2] soc: xilinx: versal: fix out of bounds array access | expand |
On 16/04/22, Jorge Ramirez-Ortiz wrote: > The call to xilinx_pm_request requires an array of a larger size. > > Signed-off-by: Jorge Ramirez-Ortiz <jorge@foundries.io> > --- > drivers/soc/soc_xilinx_versal.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/soc/soc_xilinx_versal.c b/drivers/soc/soc_xilinx_versal.c > index f8bcd9ab40..3d8c25c19b 100644 > --- a/drivers/soc/soc_xilinx_versal.c > +++ b/drivers/soc/soc_xilinx_versal.c > @@ -45,7 +45,7 @@ static const struct soc_ops soc_xilinx_versal_ops = { > static int soc_xilinx_versal_probe(struct udevice *dev) > { > struct soc_xilinx_versal_priv *priv = dev_get_priv(dev); > - u32 ret_payload[4]; > + u32 ret_payload[PAYLOAD_ARG_CNT]; all good? can this be merged? > int ret; > > priv->family = versal_family; > -- > 2.34.1 >
On 4/16/22 20:15, Jorge Ramirez-Ortiz wrote: > The call to xilinx_pm_request requires an array of a larger size. > > Signed-off-by: Jorge Ramirez-Ortiz <jorge@foundries.io> > --- > drivers/soc/soc_xilinx_versal.c | 2 +- > 1 file changed, 1 insertion(+), 1 deletion(-) > > diff --git a/drivers/soc/soc_xilinx_versal.c b/drivers/soc/soc_xilinx_versal.c > index f8bcd9ab40..3d8c25c19b 100644 > --- a/drivers/soc/soc_xilinx_versal.c > +++ b/drivers/soc/soc_xilinx_versal.c > @@ -45,7 +45,7 @@ static const struct soc_ops soc_xilinx_versal_ops = { > static int soc_xilinx_versal_probe(struct udevice *dev) > { > struct soc_xilinx_versal_priv *priv = dev_get_priv(dev); > - u32 ret_payload[4]; > + u32 ret_payload[PAYLOAD_ARG_CNT]; > int ret; > > priv->family = versal_family; Applied. M
diff --git a/drivers/soc/soc_xilinx_versal.c b/drivers/soc/soc_xilinx_versal.c index f8bcd9ab40..3d8c25c19b 100644 --- a/drivers/soc/soc_xilinx_versal.c +++ b/drivers/soc/soc_xilinx_versal.c @@ -45,7 +45,7 @@ static const struct soc_ops soc_xilinx_versal_ops = { static int soc_xilinx_versal_probe(struct udevice *dev) { struct soc_xilinx_versal_priv *priv = dev_get_priv(dev); - u32 ret_payload[4]; + u32 ret_payload[PAYLOAD_ARG_CNT]; int ret; priv->family = versal_family;
The call to xilinx_pm_request requires an array of a larger size. Signed-off-by: Jorge Ramirez-Ortiz <jorge@foundries.io> --- drivers/soc/soc_xilinx_versal.c | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-)