[PATCH v2] video: fbdev: pxafb: zero allocated memory for video modes

When parsing the video modes from DT properties, make sure to zero out
memory befor using it. This is important because not all fields in the mode
struct are explicitly initialized, even though they are used later on.

Fixes: 420a488278e86 (video: fbdev: pxafb: initial devicetree conversion)
Cc: Robert Jarzmik <robert.jarzmik@free.fr>
Signed-off-by: Daniel Mack <daniel@zonque.org>
---
v2: use kcalloc() instead of open-coding the same through
    kmalloc_array(__GFP_ZERO)

 drivers/video/fbdev/pxafb.c | 4 ++--
 1 file changed, 2 insertions(+), 2 deletions(-)

diff --git a/drivers/video/fbdev/pxafb.c b/drivers/video/fbdev/pxafb.c
index c2c2e2790079..4752b218fac4 100644
--- a/drivers/video/fbdev/pxafb.c
+++ b/drivers/video/fbdev/pxafb.c

@@ -2130,8 +2130,8 @@ static int of_get_pxafb_display(struct device *dev, struct device_node *disp,
 		return -EINVAL;
 
 	ret = -ENOMEM;
-	info->modes = kmalloc_array(timings->num_timings,
-				    sizeof(info->modes[0]), GFP_KERNEL);
+	info->modes = kcalloc(timings->num_timings, sizeof(info->modes[0]),
+			      GFP_KERNEL);
 	if (!info->modes)
 		goto out;
 	info->num_modes = timings->num_timings;

-- 
2.17.1

Daniel Mack [off-list ref] writes:

When parsing the video modes from DT properties, make sure to zero out
memory befor using it. This is important because not all fields in the mode

s/befor/before/

This taken care of ...
Reviewed-by: Robert Jarzmik <robert.jarzmik@free.fr>

Cheers.

--
Robert

On Tuesday, June 19, 2018 10:11 PM, Robert Jarzmik wrote:

Daniel Mack [off-list ref] writes:

quoted

When parsing the video modes from DT properties, make sure to zero out
memory befor using it. This is important because not all fields in the mode

s/befor/before/

Yeah, seen this too after sending. Should I resend, or can this be 
amended when applying?

 > This taken care of ...
 > Reviewed-by: Robert Jarzmik [off-list ref]

Thanks,
Daniel

Daniel Mack [off-list ref] writes:

On Tuesday, June 19, 2018 10:11 PM, Robert Jarzmik wrote:

quoted

Daniel Mack [off-list ref] writes:

quoted

When parsing the video modes from DT properties, make sure to zero out
memory befor using it. This is important because not all fields in the mode

s/befor/before/

Yeah, seen this too after sending. Should I resend, or can this be amended when
applying?

I think this question is for Bartlomiej, as fbdev is his tree ...

quoted

This taken care of ...
Reviewed-by: Robert Jarzmik <robert.jarzmik@free.fr>

Cheers.

-- 
Robert

On Thursday, June 21, 2018 11:22:35 AM Robert Jarzmik wrote:

Daniel Mack [off-list ref] writes:

quoted

On Tuesday, June 19, 2018 10:11 PM, Robert Jarzmik wrote:

quoted

Daniel Mack [off-list ref] writes:

quoted

When parsing the video modes from DT properties, make sure to zero out
memory befor using it. This is important because not all fields in the mode

s/befor/before/

Yeah, seen this too after sending. Should I resend, or can this be amended when
applying?

I think this question is for Bartlomiej, as fbdev is his tree ...

I will fix this typo while applying, no need for resend..

quoted

quoted

This taken care of ...
Reviewed-by: Robert Jarzmik <robert.jarzmik@free.fr>

Best regards,
--
Bartlomiej Zolnierkiewicz
Samsung R&D Institute Poland
Samsung Electronics

On Thursday, June 21, 2018 12:39 PM, Bartlomiej Zolnierkiewicz wrote:

On Thursday, June 21, 2018 11:22:35 AM Robert Jarzmik wrote:

quoted

Daniel Mack [off-list ref] writes:

quoted

On Tuesday, June 19, 2018 10:11 PM, Robert Jarzmik wrote:

quoted

Daniel Mack [off-list ref] writes:

quoted

When parsing the video modes from DT properties, make sure to zero out
memory befor using it. This is important because not all fields in the mode

s/befor/before/

Yeah, seen this too after sending. Should I resend, or can this be amended when
applying?

I think this question is for Bartlomiej, as fbdev is his tree ...

I will fix this typo while applying, no need for resend..

Hang on. I have some more patches on top, so I'll resend this one as 
part of a bigger series soon.


Thanks,
Daniel