From d2df4e4d0f5e7ca657d07aee84b5292d13161763 Mon Sep 17 00:00:00 2001 From: Patrick Georgi Date: Sun, 22 Feb 2015 22:50:32 +0100 Subject: libpayload: Don't try to free individual xhci device slots `di` points to a single item in xhci->dev[], which is malloc'd collectively. Trying to free() leads to pain. Change-Id: Ibd99eda905d43cbf2d2c111dfd0186ed6b119329 Signed-off-by: Patrick Georgi Found-by: Coverity Scan Reviewed-on: http://review.coreboot.org/8515 Reviewed-by: Paul Menzel Tested-by: build bot (Jenkins) Reviewed-by: Stefan Reinauer --- payloads/libpayload/drivers/usb/xhci_devconf.c | 1 - 1 file changed, 1 deletion(-) (limited to 'payloads/libpayload/drivers') diff --git a/payloads/libpayload/drivers/usb/xhci_devconf.c b/payloads/libpayload/drivers/usb/xhci_devconf.c index 5699499f26..012f610287 100644 --- a/payloads/libpayload/drivers/usb/xhci_devconf.c +++ b/payloads/libpayload/drivers/usb/xhci_devconf.c @@ -218,7 +218,6 @@ _free_return: free(tr); if (di) free(di->ctx.raw); - free((void *)di); _free_ic_return: if (ic) free(ic->raw); -- cgit v1.2.3