greybus: bundle: fix double freeing of bundle structure
authorViresh Kumar <viresh.kumar@linaro.org>
Wed, 21 Oct 2015 09:56:43 +0000 (15:26 +0530)
committerGreg Kroah-Hartman <gregkh@google.com>
Wed, 21 Oct 2015 23:17:05 +0000 (16:17 -0700)
The bundle will be released by gb_bundle_release() once all references
for the bundle 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/bundle.c

index 5c09bcc..01745f4 100644 (file)
@@ -184,7 +184,6 @@ struct gb_bundle *gb_bundle_create(struct gb_interface *intf, u8 bundle_id,
                pr_err("failed to add bundle device for id 0x%02hhx\n",
                        bundle_id);
                put_device(&bundle->dev);
-               kfree(bundle);
                return NULL;
        }