tree 9085c033cfbd6129cd8ed7c9174ab3a5f9c7324c
parent dd0490cfd2d7179960d2af6e4b6ac10054dec1af
author Greg Kaiser <gkaiser@google.com> 1551107608 -0800
committer Greg Kaiser <gkaiser@google.com> 1551107608 -0800

MemoryDealer: Avoid using invalid iterator

'pos' is invalidated by the call to erase(), so we no longer
want to use it.  Instead, we reassign 'pos' to the result of
erase(), which a valid iterator.  Since that result could be
mList.end(), and since we would otherwise perform a '++pos',
we add a new check against mList.end() to break out of our
do/while loop.  Since we'll invoke the "return freed" path,
we don't need to be worried about the outer 'for' loop
incrementing 'pos' beyond the end.

Test: TreeHugger
Change-Id: Iade7ed4077318840aca9da3a4196c268c9080e7c