Message ID | 20200725195053.14334-3-wsa+renesas@sang-engineering.com |
---|---|
State | Accepted |
Headers | show |
Series | i2c: slave: improve sanity checks when un-/registering | expand |
Hi Wolfram, Reviewed-by: Alain Volmat <alain.volmat@st.com> Alain On Sat, Jul 25, 2020 at 09:50:53PM +0200, Wolfram Sang wrote: > Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> > --- > drivers/i2c/i2c-core-slave.c | 3 +++ > 1 file changed, 3 insertions(+) > > diff --git a/drivers/i2c/i2c-core-slave.c b/drivers/i2c/i2c-core-slave.c > index 549751347e6c..1589179d5eb9 100644 > --- a/drivers/i2c/i2c-core-slave.c > +++ b/drivers/i2c/i2c-core-slave.c > @@ -58,6 +58,9 @@ int i2c_slave_unregister(struct i2c_client *client) > { > int ret; > > + if (IS_ERR_OR_NULL(client)) > + return -EINVAL; > + > if (!client->adapter->algo->unreg_slave) { > dev_err(&client->dev, "%s: not supported by adapter\n", __func__); > return -EOPNOTSUPP; > -- > 2.20.1 >
On Sat, Jul 25, 2020 at 09:50:53PM +0200, Wolfram Sang wrote:
> Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com>
Applied to for-current, thanks!
diff --git a/drivers/i2c/i2c-core-slave.c b/drivers/i2c/i2c-core-slave.c index 549751347e6c..1589179d5eb9 100644 --- a/drivers/i2c/i2c-core-slave.c +++ b/drivers/i2c/i2c-core-slave.c @@ -58,6 +58,9 @@ int i2c_slave_unregister(struct i2c_client *client) { int ret; + if (IS_ERR_OR_NULL(client)) + return -EINVAL; + if (!client->adapter->algo->unreg_slave) { dev_err(&client->dev, "%s: not supported by adapter\n", __func__); return -EOPNOTSUPP;
Signed-off-by: Wolfram Sang <wsa+renesas@sang-engineering.com> --- drivers/i2c/i2c-core-slave.c | 3 +++ 1 file changed, 3 insertions(+)