fix return from non-void