fix return value
retval?