i2c: slave: add sanity check when unregistering

Wolfram Sang · wsakernel · commit 8808981baf96 · 2020-07-28T18:37:17.000+02:00
Signed-off-by: Wolfram Sang &lt;wsa+renesas@sang-engineering.com&gt;
Reviewed-by: Alain Volmat &lt;alain.volmat@st.com&gt;
Signed-off-by: Wolfram Sang &lt;wsa@kernel.org&gt;
diff --git 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;

Original file line number	Diff line number	Diff line change
`@@ -58,6 +58,9 @@ int i2c_slave_unregister(struct i2c_client *client)`
`58`	`58`	`{`
`59`	`59`	`int ret;`
`60`	`60`
	`61`	`+ if (IS_ERR_OR_NULL(client))`
	`62`	`+ return -EINVAL;`
	`63`	`+`
`61`	`64`	`if (!client->adapter->algo->unreg_slave) {`
`62`	`65`	`dev_err(&client->dev, "%s: not supported by adapter\n", __func__);`
`63`	`66`	`return -EOPNOTSUPP;`