Are you absolutely sure you is not NULL? Try put an error("test"); in the if clause and see where exactly the crash occurs. It's usually a call to a parameter in an empty pointer (you.skill1 while you is NULL) or a division by zero.