From: Christian Gromm <christian.gromm@microchip.com>
Date: Tue, 21 Nov 2017 14:05:17 +0000 (+0100)
Subject: staging: most: core: check value returned by match function
X-Git-Tag: v4.16-rc1~112^2~542
X-Git-Url: https://asedeno.scripts.mit.edu/gitweb/?a=commitdiff_plain;h=3ba5515bf4eecbec08fedde5e35bdabb153c83be;p=linux.git

staging: most: core: check value returned by match function

This patch adds a check for the pointer returned by the function
match_component. It is needed to prevent a NULL pointer dereference in
case the provided component name does not match any list entry.

Signed-off-by: Christian Gromm <christian.gromm@microchip.com>
Signed-off-by: Greg Kroah-Hartman <gregkh@linuxfoundation.org>
---

diff --git a/drivers/staging/most/core.c b/drivers/staging/most/core.c
index 839e6cf4fbc0..6e7dcb5999bd 100644
--- a/drivers/staging/most/core.c
+++ b/drivers/staging/most/core.c
@@ -711,6 +711,8 @@ static ssize_t add_link_store(struct device_driver *drv,
 	if (ret)
 		return ret;
 	comp = match_component(comp_name);
+	if (!comp)
+		return -ENODEV;
 	if (!comp_param || *comp_param == 0) {
 		snprintf(devnod_buf, sizeof(devnod_buf), "%s-%s", mdev,
 			 mdev_ch);
@@ -754,6 +756,8 @@ static ssize_t remove_link_store(struct device_driver *drv,
 	if (ret)
 		return ret;
 	comp = match_component(comp_name);
+	if (!comp)
+		return -ENODEV;
 	c = get_channel(mdev, mdev_ch);
 	if (!c)
 		return -ENODEV;