]>
Commit | Line | Data |
---|---|---|
fee2093b GKH |
1 | From foo@baz Fri Mar 16 15:43:17 CET 2018 |
2 | From: "Gustavo A. R. Silva" <garsilva@embeddedor.com> | |
3 | Date: Wed, 22 Nov 2017 22:34:44 -0500 | |
4 | Subject: media: davinci: vpif_capture: add NULL check on devm_kzalloc return value | |
5 | ||
6 | From: "Gustavo A. R. Silva" <garsilva@embeddedor.com> | |
7 | ||
8 | ||
9 | [ Upstream commit 5a18c2434f8bfc8bc2fb0f8af3e44f7408d63e4f ] | |
10 | ||
11 | Check return value from call to devm_kzalloc() in order to prevent | |
12 | a NULL pointer dereference. | |
13 | ||
14 | This issue was detected with the help of Coccinelle. | |
15 | ||
16 | Fixes: 4a5f8ae50b66 ("[media] davinci: vpif_capture: get subdevs from DT when available") | |
17 | ||
18 | Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com> | |
19 | Signed-off-by: Hans Verkuil <hans.verkuil@cisco.com> | |
20 | Signed-off-by: Mauro Carvalho Chehab <mchehab@s-opensource.com> | |
21 | Signed-off-by: Sasha Levin <alexander.levin@microsoft.com> | |
22 | Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org> | |
23 | --- | |
24 | drivers/media/platform/davinci/vpif_capture.c | 2 ++ | |
25 | 1 file changed, 2 insertions(+) | |
26 | ||
27 | --- a/drivers/media/platform/davinci/vpif_capture.c | |
28 | +++ b/drivers/media/platform/davinci/vpif_capture.c | |
29 | @@ -1545,6 +1545,8 @@ vpif_capture_get_pdata(struct platform_d | |
30 | sizeof(*chan->inputs) * | |
31 | VPIF_CAPTURE_NUM_CHANNELS, | |
32 | GFP_KERNEL); | |
33 | + if (!chan->inputs) | |
34 | + return NULL; | |
35 | ||
36 | chan->input_count++; | |
37 | chan->inputs[i].input.type = V4L2_INPUT_TYPE_CAMERA; |