are we handling the error case?