diff options
| author | Patrick Georgi <patrick@georgi-clan.de> | 2015-02-22 22:50:32 +0100 |
|---|---|---|
| committer | Patrick Georgi <pgeorgi@google.com> | 2015-03-04 20:46:07 +0100 |
| commit | d2df4e4d0f5e7ca657d07aee84b5292d13161763 (patch) | |
| tree | 20490ea741063b1760a9109a0423b97d866d967d /payloads/libpayload | |
| parent | 48e78cf6fd04b6086f4bcb04703fad936c35f56c (diff) | |
| download | coreboot-d2df4e4d0f5e7ca657d07aee84b5292d13161763.tar.xz | |
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 <patrick@georgi-clan.de>
Found-by: Coverity Scan
Reviewed-on: http://review.coreboot.org/8515
Reviewed-by: Paul Menzel <paulepanter@users.sourceforge.net>
Tested-by: build bot (Jenkins)
Reviewed-by: Stefan Reinauer <stefan.reinauer@coreboot.org>
Diffstat (limited to 'payloads/libpayload')
| -rw-r--r-- | payloads/libpayload/drivers/usb/xhci_devconf.c | 1 |
1 files changed, 0 insertions, 1 deletions
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); |