diff options
| author | Stewart Smith <stewart@linux.vnet.ibm.com> | 2015-09-09 15:04:45 +1000 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2015-09-09 15:51:23 +1000 |
| commit | ade4a4b3aedf5fbb0832b825dd2c88ffd42ba22b (patch) | |
| tree | 97b0ec1afe9ad90878a8ea60135a6b0da09d9e8e /core/mem_region.c | |
| parent | 18cc99db5867438b2b9ba8dca4e78455c2a98abf (diff) | |
| download | skiboot-ade4a4b3aedf5fbb0832b825dd2c88ffd42ba22b.zip skiboot-ade4a4b3aedf5fbb0832b825dd2c88ffd42ba22b.tar.gz skiboot-ade4a4b3aedf5fbb0832b825dd2c88ffd42ba22b.tar.bz2 | |
Assert that memory allocation succeeds in mem_region_add_dt_reserved_node()
If we fail to allocate memory at this point in boot, we should just
assert, there's really no coming back from not being able to reserve
our reserved memory.
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'core/mem_region.c')
| -rw-r--r-- | core/mem_region.c | 2 |
1 files changed, 2 insertions, 0 deletions
diff --git a/core/mem_region.c b/core/mem_region.c index a742fe4..f6a52a0 100644 --- a/core/mem_region.c +++ b/core/mem_region.c @@ -1078,6 +1078,7 @@ static void mem_region_add_dt_reserved_node(struct dt_node *parent, } name = strdup(region->name); + assert(name); /* remove any cell addresses in the region name; we have our own cell * addresses here */ @@ -1086,6 +1087,7 @@ static void mem_region_add_dt_reserved_node(struct dt_node *parent, *p = '\0'; region->node = dt_new_addr(parent, name, region->start); + assert(region->node); dt_add_property_u64s(region->node, "reg", region->start, region->len); free(name); } |