diff options
author | Bjorn Andersson <bjorn.andersson@linaro.org> | 2017-02-13 03:44:27 -0500 |
---|---|---|
committer | Bjorn Andersson <bjorn.andersson@linaro.org> | 2017-02-13 03:52:54 -0500 |
commit | fa04b769f2832bf1aeb2e04b8865f6b688a752fd (patch) | |
tree | e2b55e83225bd8851d386ab91e813caa1850c57d /drivers/rpmsg | |
parent | 88e30752dd47f0a6398cd014af82332f1b9873ea (diff) |
rpmsg: rpmsg_create_ept() returns NULL on error
The parameter validation incorrectly returned an ERR_PTR(), which is not
handled by the callers to rpmsg_create_ept(), per the definition NULL
should be returned.
Fixes: 93e9324431c9 ("rpmsg: Handle invalid parameters in public API")
Reported-by: Dan Carpenter <dan.carpenter@oracle.com>
Signed-off-by: Bjorn Andersson <bjorn.andersson@linaro.org>
Diffstat (limited to 'drivers/rpmsg')
-rw-r--r-- | drivers/rpmsg/rpmsg_core.c | 2 |
1 files changed, 1 insertions, 1 deletions
diff --git a/drivers/rpmsg/rpmsg_core.c b/drivers/rpmsg/rpmsg_core.c index 3bf1418683b1..600f5f9f7431 100644 --- a/drivers/rpmsg/rpmsg_core.c +++ b/drivers/rpmsg/rpmsg_core.c | |||
@@ -72,7 +72,7 @@ struct rpmsg_endpoint *rpmsg_create_ept(struct rpmsg_device *rpdev, | |||
72 | struct rpmsg_channel_info chinfo) | 72 | struct rpmsg_channel_info chinfo) |
73 | { | 73 | { |
74 | if (WARN_ON(!rpdev)) | 74 | if (WARN_ON(!rpdev)) |
75 | return ERR_PTR(-EINVAL); | 75 | return NULL; |
76 | 76 | ||
77 | return rpdev->ops->create_ept(rpdev, cb, priv, chinfo); | 77 | return rpdev->ops->create_ept(rpdev, cb, priv, chinfo); |
78 | } | 78 | } |