Everything is technically correct but the posing could be tweaked a bit.
1. Have the subject smile or close they're mouth completely. A partially open mouth usually doesn't look good IMO.
2. Never have someone lock they're knee back, it destroys posture.
3. Have your subject look to the ground, then back at you and take the photo at that moment. This will allow a true expression of the person.
Hope this helps.