diff options
author | Stewart Smith <stewart@linux.vnet.ibm.com> | 2014-12-05 11:28:55 +1100 |
---|---|---|
committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2014-12-05 11:28:55 +1100 |
commit | 3681a9d62f59e6d4f29d8d723b78f449667179d3 (patch) | |
tree | 52168f9b00063b24b323476c8de71d8477c71a14 | |
parent | 4a13d92b88c0b58f6e209114ec3acbbabe0bb352 (diff) | |
download | skiboot-3681a9d62f59e6d4f29d8d723b78f449667179d3.zip skiboot-3681a9d62f59e6d4f29d8d723b78f449667179d3.tar.gz skiboot-3681a9d62f59e6d4f29d8d723b78f449667179d3.tar.bz2 |
Fix unbounded stack usage warning in core/test/run-mem_region_release_unused_noalloc.c
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
-rw-r--r-- | core/test/run-mem_region_release_unused_noalloc.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/core/test/run-mem_region_release_unused_noalloc.c b/core/test/run-mem_region_release_unused_noalloc.c index 818e272..8dadddb 100644 --- a/core/test/run-mem_region_release_unused_noalloc.c +++ b/core/test/run-mem_region_release_unused_noalloc.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))) |