rpmsg: rpmsg_create_ept() returns NULL on error
authorBjorn Andersson <bjorn.andersson@linaro.org>
Mon, 13 Feb 2017 08:44:27 +0000 (00:44 -0800)
committerBjorn Andersson <bjorn.andersson@linaro.org>
Mon, 13 Feb 2017 08:52:54 +0000 (00:52 -0800)
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>
drivers/rpmsg/rpmsg_core.c

index 3bf1418..600f5f9 100644 (file)
@@ -72,7 +72,7 @@ struct rpmsg_endpoint *rpmsg_create_ept(struct rpmsg_device *rpdev,
                                        struct rpmsg_channel_info chinfo)
 {
        if (WARN_ON(!rpdev))
-               return ERR_PTR(-EINVAL);
+               return NULL;
 
        return rpdev->ops->create_ept(rpdev, cb, priv, chinfo);
 }