asus-laptop: fix asus_input_init error path
authorAxel Lin <axel.lin@gmail.com>
Mon, 5 Jul 2010 07:29:00 +0000 (15:29 +0800)
committerMatthew Garrett <mjg@redhat.com>
Tue, 3 Aug 2010 13:49:11 +0000 (09:49 -0400)
This patch includes below fixes:
1. return -ENOMEM instead of 0 if input_allocate_device fail.
2. fix wrong goto if sparse_keymap_setup fail.
3. fix wrong goto if input_register_device fail.

Signed-off-by: Axel Lin <axel.lin@gmail.com>
Signed-off-by: Matthew Garrett <mjg@redhat.com>
drivers/platform/x86/asus-laptop.c

index 6aba246..b756e07 100644 (file)
@@ -1124,7 +1124,7 @@ static int asus_input_init(struct asus_laptop *asus)
        input = input_allocate_device();
        if (!input) {
                pr_info("Unable to allocate input device\n");
-               return 0;
+               return -ENOMEM;
        }
        input->name = "Asus Laptop extra buttons";
        input->phys = ASUS_LAPTOP_FILE "/input0";
@@ -1135,20 +1135,20 @@ static int asus_input_init(struct asus_laptop *asus)
        error = sparse_keymap_setup(input, asus_keymap, NULL);
        if (error) {
                pr_err("Unable to setup input device keymap\n");
-               goto err_keymap;
+               goto err_free_dev;
        }
        error = input_register_device(input);
        if (error) {
                pr_info("Unable to register input device\n");
-               goto err_device;
+               goto err_free_keymap;
        }
 
        asus->inputdev = input;
        return 0;
 
-err_keymap:
+err_free_keymap:
        sparse_keymap_free(input);
-err_device:
+err_free_dev:
        input_free_device(input);
        return error;
 }