diff options
| author | [email protected] <[email protected]> | 2017-05-08 20:40:16 +0200 |
|---|---|---|
| committer | Tom Rini <[email protected]> | 2017-05-12 08:37:18 -0400 |
| commit | c42640c74813f5c9aa9d0d3bdad5ebd565bdfcdc (patch) | |
| tree | ec951c7937cd3094c91dc482113470e2aeabf390 | |
| parent | 902f5bcfbcbc8dce964a69e4c9fcf658dfb62998 (diff) | |
pci: avoid memory leak
strdup uses malloc to allocate memory for str.
If we cannot bind to the generic driver we should release
the memory.
The problem was indicated by clang scan-build.
Signed-off-by: Heinrich Schuchardt <[email protected]>
Reviewed-by: Bin Meng <[email protected]>
| -rw-r--r-- | drivers/pci/pci-uclass.c | 1 |
1 files changed, 1 insertions, 0 deletions
diff --git a/drivers/pci/pci-uclass.c b/drivers/pci/pci-uclass.c index 40f59c0c4c4..504d7e3bb13 100644 --- a/drivers/pci/pci-uclass.c +++ b/drivers/pci/pci-uclass.c @@ -660,6 +660,7 @@ static int pci_find_and_bind_driver(struct udevice *parent, ret = device_bind_driver(parent, drv, str, devp); if (ret) { debug("%s: Failed to bind generic driver: %d\n", __func__, ret); + free(str); return ret; } debug("%s: No match found: bound generic driver instead\n", __func__); |