Overflow danger. Also, you don't check if return value is zero.