Search:

I never thought about that. Clever, lol.

Most likely because you are modifying the size of the members list while looping it with an iterator.
Try this instead:

for(int i=0;i<members.size();i++) {
Membership membership =...