Patrick Rudolph has uploaded this change for review. ( https://review.coreboot.org/27445
Change subject: Coverity: Fix CID1393979 ......................................................................
Coverity: Fix CID1393979
Fix resource leak in dt_set_bin_prop_by_path().
Change-Id: I1c4d7e01b25847a2091ad90d2d70711beae55905 Signed-off-by: Patrick Rudolph patrick.rudolph@9elements.com --- M src/lib/device_tree.c 1 file changed, 3 insertions(+), 0 deletions(-)
git pull ssh://review.coreboot.org:29418/coreboot refs/changes/45/27445/1
diff --git a/src/lib/device_tree.c b/src/lib/device_tree.c index 3d44f39..00b36fd 100644 --- a/src/lib/device_tree.c +++ b/src/lib/device_tree.c @@ -944,6 +944,7 @@
prop_name = strrchr(path_copy, '/'); if (!prop_name) { + free(path_copy); printk(BIOS_ERR, "Path %s does not include '/'\n", path); return 1; } @@ -956,10 +957,12 @@ if (!dt_node) { printk(BIOS_ERR, "Failed to %s %s in the device tree\n", create ? "create" : "find", path_copy); + free(path_copy); return 1; }
dt_add_bin_prop(dt_node, prop_name, data, data_size); + free(path_copy);
return 0; }