mirror of
https://github.com/dgibson/dtc.git
synced 2025-10-13 16:27:39 -04:00
If fdt_add_property or fdt_property_placeholder fail after allocating a string for the name, they return without freeing that string. This does not change the structure of the tree, but in very specific cases it could lead to undesirable space consumption. Fix this by rolling back the string allocation in this situation. Signed-off-by: Nicholas Piggin <npiggin@gmail.com> Message-Id: <20190509094122.834-2-npiggin@gmail.com> Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
100 lines
2.9 KiB
Text
100 lines
2.9 KiB
Text
LIB_TESTS_L = get_mem_rsv \
|
|
root_node find_property subnode_offset path_offset \
|
|
get_name getprop get_prop_offset get_phandle \
|
|
get_path supernode_atdepth_offset parent_offset \
|
|
node_offset_by_prop_value node_offset_by_phandle \
|
|
node_check_compatible node_offset_by_compatible \
|
|
get_alias \
|
|
char_literal \
|
|
sized_cells \
|
|
notfound \
|
|
addr_size_cells \
|
|
addr_size_cells2 \
|
|
appendprop_addrrange \
|
|
stringlist \
|
|
setprop_inplace nop_property nop_node \
|
|
sw_tree1 sw_states \
|
|
move_and_save mangle-layout nopulate \
|
|
open_pack rw_tree1 rw_oom set_name setprop del_property del_node \
|
|
appendprop1 appendprop2 propname_escapes \
|
|
string_escapes references path-references phandle_format \
|
|
boot-cpuid incbin \
|
|
extra-terminating-null \
|
|
dtbs_equal_ordered \
|
|
dtb_reverse dtbs_equal_unordered \
|
|
add_subnode_with_nops path_offset_aliases \
|
|
utilfdt_test \
|
|
integer-expressions \
|
|
property_iterate \
|
|
subnode_iterate \
|
|
overlay overlay_bad_fixup \
|
|
check_path check_header check_full \
|
|
fs_tree1
|
|
LIB_TESTS = $(LIB_TESTS_L:%=$(TESTS_PREFIX)%)
|
|
|
|
LIBTREE_TESTS_L = truncated_property truncated_string truncated_memrsv
|
|
LIBTREE_TESTS = $(LIBTREE_TESTS_L:%=$(TESTS_PREFIX)%)
|
|
|
|
DL_LIB_TESTS_L = asm_tree_dump value-labels
|
|
DL_LIB_TESTS = $(DL_LIB_TESTS_L:%=$(TESTS_PREFIX)%)
|
|
|
|
TESTS = $(LIB_TESTS) $(LIBTREE_TESTS) $(DL_LIB_TESTS)
|
|
|
|
TESTS_TREES_L = test_tree1.dtb
|
|
TESTS_TREES = $(TESTS_TREES_L:%=$(TESTS_PREFIX)%)
|
|
|
|
TESTS_TARGETS = $(TESTS) $(TESTS_TREES)
|
|
|
|
TESTS_DEPFILES = $(TESTS:%=%.d) \
|
|
$(addprefix $(TESTS_PREFIX),testutils.d trees.d dumptrees.d)
|
|
|
|
TESTS_CLEANFILES_L = $(STD_CLEANFILES) \
|
|
*.dtb *.test.dts *.test.dt.yaml *.dtsv1 tmp.* \
|
|
dumptrees
|
|
TESTS_CLEANFILES = $(TESTS) $(TESTS_CLEANFILES_L:%=$(TESTS_PREFIX)%)
|
|
TESTS_CLEANDIRS_L = fs
|
|
TESTS_CLEANDIRS = $(TESTS_CLEANDIRS_L:%=$(TESTS_PREFIX)%)
|
|
|
|
.PHONY: tests
|
|
tests: $(TESTS) $(TESTS_TREES)
|
|
|
|
$(LIB_TESTS): %: $(TESTS_PREFIX)testutils.o util.o $(LIBFDT_lib)
|
|
|
|
# Not necessary on all platforms; allow -ldl to be excluded instead of forcing
|
|
# other platforms to patch it out.
|
|
LIBDL = -ldl
|
|
$(DL_LIB_TESTS): %: %.o $(TESTS_PREFIX)testutils.o util.o $(LIBFDT_lib)
|
|
@$(VECHO) LD [libdl] $@
|
|
$(LINK.c) -o $@ $^ $(LIBDL)
|
|
|
|
$(LIBTREE_TESTS): %: $(TESTS_PREFIX)testutils.o $(TESTS_PREFIX)trees.o \
|
|
util.o $(LIBFDT_lib)
|
|
|
|
$(TESTS_PREFIX)dumptrees: $(TESTS_PREFIX)trees.o
|
|
|
|
$(TESTS_TREES): $(TESTS_PREFIX)dumptrees
|
|
@$(VECHO) DUMPTREES
|
|
cd $(TESTS_PREFIX); ./dumptrees >/dev/null
|
|
|
|
tests_clean:
|
|
@$(VECHO) CLEAN "(tests)"
|
|
rm -f $(TESTS_CLEANFILES)
|
|
rm -rf $(TESTS_CLEANDIRS)
|
|
|
|
check: tests ${TESTS_BIN} $(TESTS_PYLIBFDT)
|
|
cd $(TESTS_PREFIX); PYTHON=$(PYTHON) ./run_tests.sh
|
|
|
|
ifeq ($(NO_VALGRIND),1)
|
|
checkm:
|
|
@echo "make checkm requires valgrind, but NO_VALGRIND=1"
|
|
else
|
|
checkm: tests ${TESTS_BIN} $(TESTS_PYLIBFDT)
|
|
cd $(TESTS_PREFIX); PYTHON=$(PYTHON) ./run_tests.sh -m
|
|
endif
|
|
|
|
checkv: tests ${TESTS_BIN} $(TESTS_PYLIBFDT)
|
|
cd $(TESTS_PREFIX); PYTHON=$(PYTHON) ./run_tests.sh -v
|
|
|
|
ifneq ($(DEPTARGETS),)
|
|
-include $(TESTS_DEPFILES)
|
|
endif
|