Is this creating an infinite loop?