If "brightness" is greater than 255, it will not reach zero before the end of the loop. That could be it.