diff options
| author | Jia Xianhua <jiaxianhua@gmail.com> | 2022-09-15 16:47:03 +0800 |
|---|---|---|
| committer | Sebastian Huber <sebastian.huber@embedded-brains.de> | 2022-12-22 08:02:57 +0100 |
| commit | b4d005e3fb44e1ffb006fb0ac2d0e890fdc2bd35 (patch) | |
| tree | ca993022948db13dee01ed8fe53de271164d30c4 | |
| parent | score: Fix _Processor_mask_To_uint32_t() (diff) | |
| download | rtems-b4d005e3fb44e1ffb006fb0ac2d0e890fdc2bd35.tar.bz2 | |
libdtc: remove duplicate judgments
There is no need to check the VALID_DTB repeatedly, and can be combined
into one if statement.
Signed-off-by: Jia Xianhua <jiaxianhua@gmail.com>
Signed-off-by: David Gibson <david@gibson.dropbear.id.au>
| -rw-r--r-- | cpukit/dtc/libfdt/fdt.c | 3 |
1 files changed, 0 insertions, 3 deletions
diff --git a/cpukit/dtc/libfdt/fdt.c b/cpukit/dtc/libfdt/fdt.c index 9fe7cf4b74..90a39e8fb7 100644 --- a/cpukit/dtc/libfdt/fdt.c +++ b/cpukit/dtc/libfdt/fdt.c @@ -106,7 +106,6 @@ int fdt_check_header(const void *fdt) } hdrsize = fdt_header_size(fdt); if (!can_assume(VALID_DTB)) { - if ((fdt_totalsize(fdt) < hdrsize) || (fdt_totalsize(fdt) > INT_MAX)) return -FDT_ERR_TRUNCATED; @@ -115,9 +114,7 @@ int fdt_check_header(const void *fdt) if (!check_off_(hdrsize, fdt_totalsize(fdt), fdt_off_mem_rsvmap(fdt))) return -FDT_ERR_TRUNCATED; - } - if (!can_assume(VALID_DTB)) { /* Bounds check structure block */ if (!can_assume(LATEST) && fdt_version(fdt) < 17) { if (!check_off_(hdrsize, fdt_totalsize(fdt), |