fixed return value