diff options
author | Chengfeng Ye <cyeaa@connect.ust.hk> | 2021-11-05 06:45:07 -0700 |
---|---|---|
committer | Sebastian Reichel <sebastian.reichel@collabora.com> | 2021-11-26 00:27:06 +0100 |
commit | a1ee1c08fcd5af03187dcd41dcab12fd5b379555 (patch) | |
tree | cc8ce98bbfdb6ec69c76fe2705f47392bf1734cb | |
parent | fa55b7dcdc43c1aa1ba12bca9d2dd4318c2a0dbf (diff) | |
download | linux-a1ee1c08fcd5af03187dcd41dcab12fd5b379555.tar.bz2 |
HSI: core: Fix return freed object in hsi_new_client
cl is freed on error of calling device_register, but this
object is return later, which will cause uaf issue. Fix it
by return NULL on error.
Signed-off-by: Chengfeng Ye <cyeaa@connect.ust.hk>
Signed-off-by: Sebastian Reichel <sebastian.reichel@collabora.com>
-rw-r--r-- | drivers/hsi/hsi_core.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/hsi/hsi_core.c b/drivers/hsi/hsi_core.c index ec90713564e3..884066109699 100644 --- a/drivers/hsi/hsi_core.c +++ b/drivers/hsi/hsi_core.c @@ -102,6 +102,7 @@ struct hsi_client *hsi_new_client(struct hsi_port *port, if (device_register(&cl->device) < 0) { pr_err("hsi: failed to register client: %s\n", info->name); put_device(&cl->device); + goto err; } return cl; |