diff options
| author | Stewart Smith <stewart@linux.vnet.ibm.com> | 2014-10-27 18:18:18 +1100 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2014-10-27 18:18:18 +1100 |
| commit | 62be0cb831e9ad497b77ae828e40a407cf97b72c (patch) | |
| tree | 9210c01049742d2d34920cad15047b939ab707fe | |
| parent | cdaf6b5b313c4ea5e0cdcc299a6890817f6b02c1 (diff) | |
| download | skiboot-62be0cb831e9ad497b77ae828e40a407cf97b72c.zip skiboot-62be0cb831e9ad497b77ae828e40a407cf97b72c.tar.gz skiboot-62be0cb831e9ad497b77ae828e40a407cf97b72c.tar.bz2 | |
Fix stack usage might be unbounded warning in core/test/run-mem_region_init
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
| -rw-r--r-- | core/test/run-mem_region_init.c | 5 |
1 files changed, 4 insertions, 1 deletions
diff --git a/core/test/run-mem_region_init.c b/core/test/run-mem_region_init.c index 75c04b9..65643ec 100644 --- a/core/test/run-mem_region_init.c +++ b/core/test/run-mem_region_init.c @@ -83,7 +83,9 @@ 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= (char*)malloc(sizeof("memory@") + STR_MAX_CHARS(reg[0])); + + assert(name); /* reg contains start and length */ reg[0] = cpu_to_be64(start); @@ -95,6 +97,7 @@ static void add_mem_node(uint64_t start, uint64_t len) assert(mem); 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))) |