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>
This commit is contained in:
parent
4e6d561d51
commit
3ba5515bf4
|
@ -711,6 +711,8 @@ static ssize_t add_link_store(struct device_driver *drv,
|
||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
comp = match_component(comp_name);
|
comp = match_component(comp_name);
|
||||||
|
if (!comp)
|
||||||
|
return -ENODEV;
|
||||||
if (!comp_param || *comp_param == 0) {
|
if (!comp_param || *comp_param == 0) {
|
||||||
snprintf(devnod_buf, sizeof(devnod_buf), "%s-%s", mdev,
|
snprintf(devnod_buf, sizeof(devnod_buf), "%s-%s", mdev,
|
||||||
mdev_ch);
|
mdev_ch);
|
||||||
|
@ -754,6 +756,8 @@ static ssize_t remove_link_store(struct device_driver *drv,
|
||||||
if (ret)
|
if (ret)
|
||||||
return ret;
|
return ret;
|
||||||
comp = match_component(comp_name);
|
comp = match_component(comp_name);
|
||||||
|
if (!comp)
|
||||||
|
return -ENODEV;
|
||||||
c = get_channel(mdev, mdev_ch);
|
c = get_channel(mdev, mdev_ch);
|
||||||
if (!c)
|
if (!c)
|
||||||
return -ENODEV;
|
return -ENODEV;
|
||||||
|
|
Loading…
Reference in New Issue