On 04/23/2012 01:50 AM, Johannes Berg wrote:
On Mon, 2012-04-23 at 10:45 +0200, Florian Fainelli wrote:
} else if (regaddr == SDIO_CCCR_ABORT) {
+ sdfunc = kzalloc(sizeof(struct sdio_func), GFP_KERNEL);
You are not catching a kzalloc() possible failure here.
+ memcpy(sdfunc, sdiodev->func[0], sizeof(struct sdio_func));
and it should probably be kmemdup anyway?
johannes
Thx Florian and Johannes, I will send out a V2.
Franky
--
To unsubscribe from this list: send the line "unsubscribe linux-wireless" in
the body of a message to majordomo@xxxxxxxxxxxxxxx
More majordomo info at http://vger.kernel.org/majordomo-info.html
