Shouldn't this be checking the return value?