Input: synaptics-rmi4 - fail probing if memory allocation for "phys" fails
authorKunwu Chan <chentao@kylinos.cn>
Thu, 18 Jan 2024 19:37:59 +0000 (11:37 -0800)
committerDmitry Torokhov <dmitry.torokhov@gmail.com>
Thu, 18 Jan 2024 23:21:34 +0000 (15:21 -0800)
While input core can work with input->phys set to NULL userspace might
depend on it, so better fail probing if allocation fails. The system must
be in a pretty bad shape for it to happen anyway.

Signed-off-by: Kunwu Chan <chentao@kylinos.cn>
Link: https://lore.kernel.org/r/20240117073124.143636-1-chentao@kylinos.cn
Signed-off-by: Dmitry Torokhov <dmitry.torokhov@gmail.com>
drivers/input/rmi4/rmi_driver.c

index 258d5fe..aa32371 100644 (file)
@@ -1196,7 +1196,11 @@ static int rmi_driver_probe(struct device *dev)
                }
                rmi_driver_set_input_params(rmi_dev, data->input);
                data->input->phys = devm_kasprintf(dev, GFP_KERNEL,
-                                               "%s/input0", dev_name(dev));
+                                                  "%s/input0", dev_name(dev));
+               if (!data->input->phys) {
+                       retval = -ENOMEM;
+                       goto err;
+               }
        }
 
        retval = rmi_init_functions(data);