Commit 6b50d8b8 authored by Dmitry Torokhov's avatar Dmitry Torokhov

Input: elo - handle input_register_device() failures

Signed-off-by: default avatarDmitry Torokhov <dtor@mail.ru>
parent f287caee
...@@ -187,11 +187,13 @@ static irqreturn_t elo_interrupt(struct serio *serio, ...@@ -187,11 +187,13 @@ static irqreturn_t elo_interrupt(struct serio *serio,
static void elo_disconnect(struct serio *serio) static void elo_disconnect(struct serio *serio)
{ {
struct elo* elo = serio_get_drvdata(serio); struct elo *elo = serio_get_drvdata(serio);
input_get_device(elo->dev);
input_unregister_device(elo->dev); input_unregister_device(elo->dev);
serio_close(serio); serio_close(serio);
serio_set_drvdata(serio, NULL); serio_set_drvdata(serio, NULL);
input_put_device(elo->dev);
kfree(elo); kfree(elo);
} }
...@@ -211,7 +213,7 @@ static int elo_connect(struct serio *serio, struct serio_driver *drv) ...@@ -211,7 +213,7 @@ static int elo_connect(struct serio *serio, struct serio_driver *drv)
input_dev = input_allocate_device(); input_dev = input_allocate_device();
if (!elo || !input_dev) { if (!elo || !input_dev) {
err = -ENOMEM; err = -ENOMEM;
goto fail; goto fail1;
} }
elo->serio = serio; elo->serio = serio;
...@@ -257,13 +259,17 @@ static int elo_connect(struct serio *serio, struct serio_driver *drv) ...@@ -257,13 +259,17 @@ static int elo_connect(struct serio *serio, struct serio_driver *drv)
err = serio_open(serio, drv); err = serio_open(serio, drv);
if (err) if (err)
goto fail; goto fail2;
err = input_register_device(elo->dev);
if (err)
goto fail3;
input_register_device(elo->dev);
return 0; return 0;
fail: serio_set_drvdata(serio, NULL); fail3: serio_close(serio);
input_free_device(input_dev); fail2: serio_set_drvdata(serio, NULL);
fail1: input_free_device(input_dev);
kfree(elo); kfree(elo);
return err; return err;
} }
......
Markdown is supported
0%
or
You are about to add 0 people to the discussion. Proceed with caution.
Finish editing this message first!
Please register or to comment