[PATCH 0/6] ux500: Export SoC information and some platform clean-up · Lee Jones <hidden> · 2012-01-20
[PATCH 1/6] mach-ux500: pass parent pointer to each platform device · Lee Jones <hidden> · 2012-01-20
[PATCH 2/6] drivers/base: add bus for System-on-Chip devices · Lee Jones <hidden> · 2012-01-20
[PATCH 2/6] drivers/base: add bus for System-on-Chip devices · Greg KH <hidden> · 2012-01-20
[PATCH 2/6] drivers/base: add bus for System-on-Chip devices · Greg KH <hidden> · 2012-01-20
[PATCH 3/6] Documentation: add information for new sysfs soc bus functionality · Lee Jones <hidden> · 2012-01-20
[PATCH 4/6] mach-ux500: export System-on-Chip information ux500 via sysfs · Lee Jones <hidden> · 2012-01-20
[PATCH 5/6] mach-ux500: move top level platform devices in sysfs to /sys/devices/socX · Lee Jones <hidden> · 2012-01-20
[PATCH 6/6] mach-ux500: remove intermediary add_platform_device* functions · Lee Jones <hidden> · 2012-01-20

DORMANTno replies

Revision v1 of 12 in this series.

Revisions (12)

2011-10-17 v1 [diff vs current]
2012-01-20 v1 [diff vs current]
2012-01-20 v1 [diff vs current]
2012-01-20 v1 current
2012-01-21 v1 [diff vs current]
2012-01-28 v1 [diff vs current]
2012-01-30 v1 [diff vs current]
2012-01-30 v1 [diff vs current]
2012-02-01 v1 [diff vs current]
2012-02-01 v1 [diff vs current]
2012-02-01 v1 [diff vs current]
2012-02-06 v1 [diff vs current]

[PATCH 2/6] drivers/base: add bus for System-on-Chip devices

From: Greg KH <hidden>
Date: 2012-01-20 16:39:47

On Fri, Jan 20, 2012 at 04:10:25PM +0000, Lee Jones wrote:

+struct soc_device *soc_device_register(struct soc_device_attribute *soc_dev_attr)
+{
+	struct soc_device *soc_dev;
+	int ret;
+	int i, j = 0;
+
+	soc_dev = kzalloc(sizeof(*soc_dev), GFP_KERNEL);
+	if (!soc_dev) {
+	        ret = -ENOMEM;
+		goto out1;
+	}

<snip>

+out1:
+	kfree(soc_dev_attr);

You just freed the caller's memory, are you sure you wanted to do that?