diff options
| author | Cyril Bur <cyril.bur@au1.ibm.com> | 2017-08-03 16:45:46 +1000 |
|---|---|---|
| committer | Stewart Smith <stewart@linux.vnet.ibm.com> | 2017-08-15 16:37:14 +1000 |
| commit | fd0411b559a017df47b690c2f69f593fa2eece09 (patch) | |
| tree | 9c07b0616e4d0ac2b4bd1cd3fca0594f5f1045fb /core | |
| parent | 80ec3381e96629522793a0056f1a3e36d35d9315 (diff) | |
| download | skiboot-fd0411b559a017df47b690c2f69f593fa2eece09.zip skiboot-fd0411b559a017df47b690c2f69f593fa2eece09.tar.gz skiboot-fd0411b559a017df47b690c2f69f593fa2eece09.tar.bz2 | |
core/mem_region: Check return value of add_region() (CID 147251)
Just print an error if a region failed to add - at least then there will
be a trace somewhere about the problem.
Fixes: CID 147251
Signed-off-by: Cyril Bur <cyril.bur@au1.ibm.com>
Signed-off-by: Stewart Smith <stewart@linux.vnet.ibm.com>
Diffstat (limited to 'core')
| -rw-r--r-- | core/mem_region.c | 6 |
1 files changed, 5 insertions, 1 deletions
diff --git a/core/mem_region.c b/core/mem_region.c index a098685..3c93cd4 100644 --- a/core/mem_region.c +++ b/core/mem_region.c @@ -995,7 +995,11 @@ static bool mem_region_parse_reserved_nodes(const char *path) dt_get_number(reg->prop, 2), dt_get_number(reg->prop + sizeof(u64), 2), node, type); - add_region(region); + if (!add_region(region)) { + char *nodepath = dt_get_path(node); + prerror("node %s failed to add_region()\n", nodepath); + free(nodepath); + } } return true; |