The scanf takes the 'name' which is already an address so you don't need the ampersand, and you are reading a string instead of a single character so the format specifier is 's', not 'c'.