diff options
| author | Stewart Smith <stewart@linux.vnet.ibm.com> | 2014-12-05 11:52:22 +1100 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2014-12-05 11:52:22 +1100 |
| commit | 4e6b385fd50a74fd47396dc150b51a0bada2aa87 (patch) | |
| tree | 8c4a4ffa138c12190374d51df3d4a613bd3ea68c /core | |
| parent | 0e6167e3fee3adbd9f65fdb5cfe59d6e544bd4bb (diff) | |
| download | skiboot-4e6b385fd50a74fd47396dc150b51a0bada2aa87.zip skiboot-4e6b385fd50a74fd47396dc150b51a0bada2aa87.tar.gz skiboot-4e6b385fd50a74fd47396dc150b51a0bada2aa87.tar.bz2 | |
Fix unbounded stack usage warning in core/test/run-mem_region_release_unused.c
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'core')
| -rw-r--r-- | core/test/run-mem_region_release_unused.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/core/test/run-mem_region_release_unused.c b/core/test/run-mem_region_release_unused.c index e73cf25..b8acffd 100644 --- a/core/test/run-mem_region_release_unused.c +++ b/core/test/run-mem_region_release_unused.c @@ -77,7 +77,10 @@ static void add_mem_node(uint64_t start, uint64_t len) { struct dt_node *mem; u64 reg[2]; - char name[sizeof("memory@") + STR_MAX_CHARS(reg[0])]; + char *name; + + name = (char*)malloc(sizeof("memory@") + STR_MAX_CHARS(reg[0])); + assert(name); /* reg contains start and length */ reg[0] = cpu_to_be64(start); @@ -88,6 +91,7 @@ static void add_mem_node(uint64_t start, uint64_t len) mem = dt_new(dt_root, name); dt_add_property_string(mem, "device_type", "memory"); dt_add_property(mem, "reg", reg, sizeof(reg)); + free(name); } void add_chip_dev_associativity(struct dt_node *dev __attribute__((unused))) |