Correct
A charmed creature may attack any legal target. That is to to say if it can fly it can attack fliers and non fliers if it does not fly it can only attack things on the ground. But it may not attack the mage that charmed it.Hope that helps.
And why the card says "controller" instead "mage"?