greybus: module: fix double freeing of module structure
authorViresh Kumar <viresh.kumar@linaro.org>
Wed, 21 Oct 2015 09:56:45 +0000 (15:26 +0530)
committerGreg Kroah-Hartman <gregkh@google.com>
Wed, 21 Oct 2015 23:17:05 +0000 (16:17 -0700)
The module will be released by gb_module_release() once all references
for the module are dropped. And so there is no need to free it in the
error path specially.

Reviewed-by: Johan Hovold <johan@hovoldconsulting.com>
Signed-off-by: Viresh Kumar <viresh.kumar@linaro.org>
Signed-off-by: Greg Kroah-Hartman <gregkh@google.com>
drivers/staging/greybus/module.c

index d770654..43e8bab 100644 (file)
@@ -151,7 +151,6 @@ struct gb_module *gb_module_create(struct device *parent, u8 module_id)
                pr_err("failed to add module device for id 0x%02hhx\n",
                        module_id);
                put_device(&module->dev);
-               kfree(module);
                return NULL;
        }