xlat_v2: add a check on mm_cursor->size to avoid infinite loop
authorYann Gautier <yann.gautier@st.com>
Thu, 14 Jun 2018 12:36:20 +0000 (14:36 +0200)
committerYann Gautier <yann.gautier@st.com>
Thu, 14 Jun 2018 12:36:20 +0000 (14:36 +0200)
commit75df62699b9911de0ed0b9b4e95c5d682910cc0e
treee5cf462301a64f257198e140466be6c4c406e2cf
parented4cf49020b62fadd994dfa96f74151abb105cf4
xlat_v2: add a check on mm_cursor->size to avoid infinite loop

The issue can occur if end_va is equal to the max architecture address,
and when mm_cursor point to the last entry of mmap_region_t table: {0}.
The first line of the while will then be true, e.g. on AARCH32, we have:
mm_cursor->base_va (=0) + mm_cursor->size (=0) - 1 == end_va (=0xFFFFFFFF)
And the mm_cursor->size = 0 will be lesser than mm->size

A check on mm_cursor->size != 0 should be done as in the previous while,
to avoid such kind of infinite loop.

fixes arm-software/tf-issues#594

Signed-off-by: Yann Gautier <yann.gautier@st.com>
lib/xlat_tables_v2/xlat_tables_internal.c