From: Stephen Warren <swarren@nvidia.com>
Date: Fri, 12 Feb 2016 20:56:01 +0000 (-0700)
Subject: usb: eth: fix memalign() parameter order
X-Git-Tag: v2016.03-rc3~20^2~2
X-Git-Url: https://git.sur5r.net/?a=commitdiff_plain;h=53419bac4edc4d07225fa80ff1aedf0c301e7cc4;p=u-boot

usb: eth: fix memalign() parameter order

The alignment and size were swapped, leading to malloc heap corruption.

On my system, this sometimes caused U-Boot to crash during or after
certain USB Ethernet operations.

Fixes: c8c2797c3810 ("dm: usb: eth: Support driver model with USB Ethernet")
Signed-off-by: Stephen Warren <swarren@nvidia.com>
---

diff --git a/drivers/usb/eth/usb_ether.c b/drivers/usb/eth/usb_ether.c
index b9c9a8402e..36734e2e51 100644
--- a/drivers/usb/eth/usb_ether.c
+++ b/drivers/usb/eth/usb_ether.c
@@ -73,7 +73,7 @@ int usb_ether_register(struct udevice *dev, struct ueth_data *ueth, int rxsize)
 	}
 
 	ueth->rxsize = rxsize;
-	ueth->rxbuf = memalign(rxsize, ARCH_DMA_MINALIGN);
+	ueth->rxbuf = memalign(ARCH_DMA_MINALIGN, rxsize);
 	if (!ueth->rxbuf)
 		return -ENOMEM;