fixed variable name and return value