From mboxrd@z Thu Jan  1 00:00:00 1970
From: greg@kroah.com (Greg KH)
Date: Fri, 20 Jan 2012 11:39:47 -0500
Subject: [PATCH 2/6] drivers/base: add bus for System-on-Chip devices
In-Reply-To: <1327075829-14346-3-git-send-email-lee.jones@linaro.org>
References: <1327075829-14346-1-git-send-email-lee.jones@linaro.org>
 <1327075829-14346-3-git-send-email-lee.jones@linaro.org>
Message-ID: <20120120163947.GB4900@kroah.com>
To: linux-arm-kernel@lists.infradead.org
List-Id: linux-arm-kernel.lists.infradead.org

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?