Robert Jarzmik <robert.jarzmik@xxxxxxx> writes: > As seen by Julia, the initial allocation memory is not checked anymore > after commit "video: fbdev: pxafb: initial devicetree conversion". > Introduce back the removed test. > > Reported-by: Julia Lawall <julia.lawall@xxxxxxx> > Signed-off-by: Robert Jarzmik <robert.jarzmik@xxxxxxx> > --- > drivers/video/fbdev/pxafb.c | 2 ++ > 1 file changed, 2 insertions(+) > > diff --git a/drivers/video/fbdev/pxafb.c b/drivers/video/fbdev/pxafb.c > index 8588dd683105..54666424a720 100644 > --- a/drivers/video/fbdev/pxafb.c > +++ b/drivers/video/fbdev/pxafb.c > @@ -2252,6 +2252,8 @@ static int pxafb_probe(struct platform_device *dev) > ret = -ENOMEM; > pdata = dev_get_platdata(&dev->dev); > inf = devm_kmalloc(&dev->dev, sizeof(*inf), GFP_KERNEL); > + if (!inf) > + goto failed; > > if (pdata) { > *inf = *pdata; Tomi, Jean-Christophe, Can you carry out this fix please ? Cheers. -- Robert -- To unsubscribe from this list: send the line "unsubscribe linux-fbdev" in the body of a message to majordomo@xxxxxxxxxxxxxxx More majordomo info at http://vger.kernel.org/majordomo-info.html