Skip to content

Commit

Permalink
Input: db9 - handle errors from input_register_device()
Browse files Browse the repository at this point in the history
Also db9_remove shouldn't be marked __exit as it is also called from
__init code.

Signed-off-by: Dmitry Torokhov <dtor@mail.ru>
  • Loading branch information
Dmitry Torokhov committed Jan 30, 2006
1 parent 84c6189 commit 2e9d675
Showing 1 changed file with 10 additions and 5 deletions.
15 changes: 10 additions & 5 deletions drivers/input/joystick/db9.c
Original file line number Diff line number Diff line change
Expand Up @@ -616,7 +616,7 @@ static struct db9 __init *db9_probe(int parport, int mode)
if (!input_dev) {
printk(KERN_ERR "db9.c: Not enough memory for input device\n");
err = -ENOMEM;
goto err_free_devs;
goto err_unreg_devs;
}

sprintf(db9->phys[i], "%s/input%d", db9->pd->port->name, i);
Expand All @@ -642,13 +642,17 @@ static struct db9 __init *db9_probe(int parport, int mode)
input_set_abs_params(input_dev, db9_abs[j], 1, 255, 0, 0);
}

input_register_device(input_dev);
err = input_register_device(input_dev);
if (err)
goto err_free_dev;
}

parport_put_port(pp);
return db9;

err_free_devs:
err_free_dev:
input_free_device(db9->dev[i]);
err_unreg_devs:
while (--i >= 0)
input_unregister_device(db9->dev[i]);
kfree(db9);
Expand All @@ -660,7 +664,7 @@ static struct db9 __init *db9_probe(int parport, int mode)
return ERR_PTR(err);
}

static void __exit db9_remove(struct db9 *db9)
static void db9_remove(struct db9 *db9)
{
int i;

Expand Down Expand Up @@ -698,7 +702,8 @@ static int __init db9_init(void)

if (err) {
while (--i >= 0)
db9_remove(db9_base[i]);
if (db9_base[i])
db9_remove(db9_base[i]);
return err;
}

Expand Down

0 comments on commit 2e9d675

Please sign in to comment.