fixing return values