Don't you need to increment the counter here also?