fixed return values