lists.openwall.net   lists  /  announce  owl-users  owl-dev  john-users  john-dev  passwdqc-users  yescrypt  popa3d-users  /  oss-security  kernel-hardening  musl  sabotage  tlsify  passwords  /  crypt-dev  xvendor  /  Bugtraq  Full-Disclosure  linux-kernel  linux-netdev  linux-ext4  linux-hardening  linux-cve-announce  PHC 
Open Source and information security mailing list archives
 
Hash Suite: Windows password security audit tool. GUI, reports in PDF.
[<prev] [next>] [thread-next>] [day] [month] [year] [list]
Date:	Tue, 10 Oct 2006 02:49:39 -0400
From:	Jeff Garzik <jeff@...zik.org>
To:	dmitry.torokhov@...il.com, Andrew Morton <akpm@...l.org>,
	LKML <linux-kernel@...r.kernel.org>
Subject: [PATCH] input: handle sysfs errors


Signed-off-by: Jeff Garzik <jeff@...zik.org>

---

 drivers/input/keyboard/atkbd.c     |   25 ++++++++++++++++++++-----
 drivers/input/mouse/psmouse-base.c |   11 +++++++++--
 drivers/input/mouse/trackpoint.c   |    7 ++++++-
 3 files changed, 35 insertions(+), 8 deletions(-)

diff --git a/drivers/input/keyboard/atkbd.c b/drivers/input/keyboard/atkbd.c
index b6ef9ea..4d5c790 100644
--- a/drivers/input/keyboard/atkbd.c
+++ b/drivers/input/keyboard/atkbd.c
@@ -961,11 +961,16 @@ static int atkbd_connect(struct serio *s
 	atkbd_set_keycode_table(atkbd);
 	atkbd_set_device_attrs(atkbd);
 
-	device_create_file(&serio->dev, &atkbd_attr_extra);
-	device_create_file(&serio->dev, &atkbd_attr_scroll);
-	device_create_file(&serio->dev, &atkbd_attr_set);
-	device_create_file(&serio->dev, &atkbd_attr_softrepeat);
-	device_create_file(&serio->dev, &atkbd_attr_softraw);
+	err = device_create_file(&serio->dev, &atkbd_attr_extra);
+	if (err) goto fail_serio;
+	err = device_create_file(&serio->dev, &atkbd_attr_scroll);
+	if (err) goto fail_extra;
+	err = device_create_file(&serio->dev, &atkbd_attr_set);
+	if (err) goto fail_scroll;
+	err = device_create_file(&serio->dev, &atkbd_attr_softrepeat);
+	if (err) goto fail_set;
+	err = device_create_file(&serio->dev, &atkbd_attr_softraw);
+	if (err) goto fail_softrep;
 
 	atkbd_enable(atkbd);
 
@@ -973,6 +978,16 @@ static int atkbd_connect(struct serio *s
 
 	return 0;
 
+ fail_softrep:
+	device_remove_file(&serio->dev, &atkbd_attr_softrepeat);
+ fail_set:
+	device_remove_file(&serio->dev, &atkbd_attr_set);
+ fail_scroll:
+	device_remove_file(&serio->dev, &atkbd_attr_scroll);
+ fail_extra:
+	device_remove_file(&serio->dev, &atkbd_attr_extra);
+ fail_serio:
+	serio_close(serio);
  fail:	serio_set_drvdata(serio, NULL);
 	input_free_device(dev);
 	kfree(atkbd);
diff --git a/drivers/input/mouse/psmouse-base.c b/drivers/input/mouse/psmouse-base.c
index 6f9b2c7..0bd349f 100644
--- a/drivers/input/mouse/psmouse-base.c
+++ b/drivers/input/mouse/psmouse-base.c
@@ -1152,11 +1152,13 @@ static int psmouse_connect(struct serio 
 
 	input_register_device(psmouse->dev);
 
+	retval = sysfs_create_group(&serio->dev.kobj, &psmouse_attribute_group);
+	if (retval)
+		goto err_out;
+
 	if (parent && parent->pt_activate)
 		parent->pt_activate(parent);
 
-	sysfs_create_group(&serio->dev.kobj, &psmouse_attribute_group);
-
 	psmouse_activate(psmouse);
 
 	retval = 0;
@@ -1174,6 +1176,11 @@ out:
 
 	mutex_unlock(&psmouse_mutex);
 	return retval;
+err_out:
+	input_unregister_device(psmouse->dev);
+	psmouse_set_state(psmouse, PSMOUSE_IGNORE);
+	serio_close(serio);
+	goto out;
 }
 
 
diff --git a/drivers/input/mouse/trackpoint.c b/drivers/input/mouse/trackpoint.c
index ae5871a..8d639ff 100644
--- a/drivers/input/mouse/trackpoint.c
+++ b/drivers/input/mouse/trackpoint.c
@@ -293,6 +293,7 @@ int trackpoint_detect(struct psmouse *ps
 	struct ps2dev *ps2dev = &psmouse->ps2dev;
 	unsigned char firmware_id;
 	unsigned char button_info;
+	int rc;
 
 	if (trackpoint_start_protocol(psmouse, &firmware_id))
 		return -1;
@@ -318,7 +319,11 @@ int trackpoint_detect(struct psmouse *ps
 	trackpoint_defaults(priv);
 	trackpoint_sync(psmouse);
 
-	sysfs_create_group(&ps2dev->serio->dev.kobj, &trackpoint_attr_group);
+	rc=sysfs_create_group(&ps2dev->serio->dev.kobj, &trackpoint_attr_group);
+	if (rc) {
+		kfree(priv);
+		return -1;
+	}
 
 	printk(KERN_INFO "IBM TrackPoint firmware: 0x%02x, buttons: %d/%d\n",
 		firmware_id, (button_info & 0xf0) >> 4, button_info & 0x0f);
-
To unsubscribe from this list: send the line "unsubscribe linux-kernel" in
the body of a message to majordomo@...r.kernel.org
More majordomo info at  http://vger.kernel.org/majordomo-info.html
Please read the FAQ at  http://www.tux.org/lkml/

Powered by blists - more mailing lists