diff options
author | Gustavo A. R. Silva <garsilva@embeddedor.com> | 2017-07-06 16:58:33 -0500 |
---|---|---|
committer | Sean Paul <seanpaul@chromium.org> | 2017-07-10 14:13:00 -0400 |
commit | 6f6e0b217a93011f8e11b9a2d5521a08fcf36990 (patch) | |
tree | b71e3e38c1c6272438190805a85a471f4f515ca7 | |
parent | 4086d90cffb8f48400d51fbab575fe50458512e3 (diff) |
drm/rockchip: fix NULL check on devm_kzalloc() return valuedrm-misc-next-fixes-2017-07-10
The right variable to check here is port, not dp.
This issue was detected using Coccinelle and the following semantic patch:
@@
expression x;
identifier fld;
@@
* x = devm_kzalloc(...);
... when != x == NULL
x->fld
Signed-off-by: Gustavo A. R. Silva <garsilva@embeddedor.com>
Acked-by: Mark Yao <mark.yao@rock-chips.com>
Signed-off-by: Sean Paul <seanpaul@chromium.org>
Link: http://patchwork.freedesktop.org/patch/msgid/20170706215833.GA25411@embeddedgus
-rw-r--r-- | drivers/gpu/drm/rockchip/cdn-dp-core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/gpu/drm/rockchip/cdn-dp-core.c b/drivers/gpu/drm/rockchip/cdn-dp-core.c index 14fa1f8351e8..9b0b0588bbed 100644 --- a/drivers/gpu/drm/rockchip/cdn-dp-core.c +++ b/drivers/gpu/drm/rockchip/cdn-dp-core.c @@ -1195,7 +1195,7 @@ static int cdn_dp_probe(struct platform_device *pdev) continue; port = devm_kzalloc(dev, sizeof(*port), GFP_KERNEL); - if (!dp) + if (!port) return -ENOMEM; port->extcon = extcon; |